Changeset fbc76440 in rtems


Ignore:
Timestamp:
Mar 12, 2021, 5:14:47 PM (5 months ago)
Author:
Ryan Long <ryan.long@…>
Branches:
master
Children:
0b23660
Parents:
8383572
git-author:
Ryan Long <ryan.long@…> (03/12/21 17:14:47)
git-committer:
Joel Sherrill <joel@…> (04/08/21 22:58:19)
Message:

main_cp.c: Unused value (CID #1255344)

CID 1255344: Unused value in copy().

Closes #4339

File:
1 edited

Legend:

Unmodified
Added
Removed
  • cpukit/libmisc/shell/main_cp.c

    r8383572 rfbc76440  
    311311                        warnx("%s: %s",
    312312                            curr->fts_path, strerror(curr->fts_errno));
     313                #ifdef __rtems__
     314                /*
     315                 * Coverity spotted that badcp is set by each loop
     316                 * iteration so setting it right before continue
     317                 * results in the value being unused. See CID 1255344
     318                 *
     319                 * The current NetBSD source (v1.62) was checked and
     320                 * the same issue appears to apply although the
     321                 * variable names have changed since this was imported
     322                 * to RTEMS.
     323                 *
     324                 * This pattern exists in multiple places in this file.
     325                 */
     326                        rval = 1;
     327                #else
    313328                        badcp = rval = 1;
     329                #endif
    314330                        continue;
    315331                case FTS_DC:                    /* Warn, continue. */
    316332                        warnx("%s: directory causes a cycle", curr->fts_path);
     333                #ifdef __rtems__
     334                        rval = 1;
     335                #else
    317336                        badcp = rval = 1;
     337                #endif
    318338                        continue;
    319339                default:
     
    367387                                warnx("%s%s: name too long (not copied)",
    368388                                    to.p_path, p);
     389                        #ifdef __rtems__
     390                                rval = 1;
     391                        #else
    369392                                badcp = rval = 1;
     393                        #endif
    370394                                continue;
    371395                        }
     
    419443                                warnx("%s and %s are identical (not copied).",
    420444                                    to.p_path, curr->fts_path);
     445                        #ifdef __rtems__
     446                                rval = 1;
     447                        #else
    421448                                badcp = rval = 1;
     449                        #endif
    422450                                if (S_ISDIR(curr->fts_statp->st_mode))
    423451                                        (void)fts_set(ftsp, curr, FTS_SKIP);
     
    429457                                    "non-directory %s",
    430458                                    to.p_path, curr->fts_path);
     459                        #ifdef __rtems__
     460                                rval = 1;
     461                        #else
    431462                                badcp = rval = 1;
     463                        #endif
    432464                                continue;
    433465                        }
Note: See TracChangeset for help on using the changeset viewer.