Changeset 8f73af85 in rtems


Ignore:
Timestamp:
Nov 20, 2014, 11:19:13 PM (5 years ago)
Author:
Joel Sherrill <joel.sherrill@…>
Branches:
4.11, master
Children:
e8abdfb
Parents:
5dff7425
git-author:
Joel Sherrill <joel.sherrill@…> (11/20/14 23:19:13)
git-committer:
Joel Sherrill <joel.sherrill@…> (11/21/14 19:13:21)
Message:

dosfs/msdos_format.c: Dead code removal (Coverity ID 1255325)

Coverity identified that ret_val was never set except to be initialized
to 0. Thus the code could not be executed.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • cpukit/libfs/src/dosfs/msdos_format.c

    r5dff7425 r8f73af85  
    218218   */
    219219  while ((ret_val == 0) &&
    220         (sector_cnt > 0)) {
     220        (sector_cnt > 0)) {
    221221    int percent = (sector_cnt * 100) / total_sectors;
    222222    if (percent != last_percent) {
     
    311311
    312312  bool     finished = false;
    313   int      ret_val = 0;
    314313  uint32_t fatdata_cluster_cnt;
    315314  uint32_t fat_capacity;
     
    351350
    352351    sectors_per_fat = ((fat_capacity
    353                         + (bytes_per_sector - 1))
    354                        / bytes_per_sector);
     352                        + (bytes_per_sector - 1))
     353                       / bytes_per_sector);
    355354
    356355    fat_sectors_cnt = loc_align_object (sectors_per_fat * fat_num,
     
    359358
    360359    *data_cluster_cnt = (fatdata_cluster_cnt -
    361                         ((fat_sectors_cnt
    362                           + (sectors_per_cluster - 1))
    363                         / sectors_per_cluster));
     360                        ((fat_sectors_cnt
     361                          + (sectors_per_cluster - 1))
     362                        / sectors_per_cluster));
    364363    /*
    365364     * data cluster count too big? Then make clusters bigger
     
    380379      }
    381380    } else if ((sectors_per_cluster * bytes_per_sector)
    382         > MS_BYTES_PER_CLUSTER_LIMIT) {
     381        > MS_BYTES_PER_CLUSTER_LIMIT) {
    383382      finished = true;
    384383    }
     
    388387  *sectors_per_fat_ptr     = fat_sectors_cnt / fat_num;
    389388
    390   if (ret_val != 0) {
    391     errno = EINVAL;
    392   }
    393 
    394   return ret_val;
     389  return 0;
    395390}
    396391
     
    526521   */
    527522  if (ret_val == 0) {
    528     if ((rqdata == NULL) ||
    529         (rqdata->fat_num == 0)) {
     523    if ((rqdata == NULL) || (rqdata->fat_num == 0)) {
    530524      fmt_params->fat_num = 2;
    531525    }
     
    683677
    684678        ret_val = msdos_format_eval_sectors_per_cluster(fmt_params->fattype,
    685                                                         fmt_params->bytes_per_sector,
    686                                                         fmt_params->totl_sector_cnt,
    687                                                         fmt_params->rsvd_sector_cnt,
    688                                                         fmt_params->root_dir_sectors,
    689                                                         fmt_params->fat_num,
    690                                                         fmt_params->sectors_per_cluster,
    691                                                         fmt_params->skip_alignment,
    692                                                         &sectors_per_cluster_adj,
    693                                                         &fmt_params->sectors_per_fat,
    694                                                         &data_clusters_cnt);
     679                     fmt_params->bytes_per_sector,
     680                     fmt_params->totl_sector_cnt,
     681                     fmt_params->rsvd_sector_cnt,
     682                     fmt_params->root_dir_sectors,
     683                     fmt_params->fat_num,
     684                     fmt_params->sectors_per_cluster,
     685                     fmt_params->skip_alignment,
     686                     &sectors_per_cluster_adj,
     687                     &fmt_params->sectors_per_fat,
     688                     &data_clusters_cnt);
    695689        fmt_params->sectors_per_cluster = sectors_per_cluster_adj;
    696690        fat_type = fmt_params->fattype;
     
    744738   */
    745739  if (ret_val == 0) {
    746     if ((rqdata != NULL) &&
    747         (rqdata->media != 0)) {
     740    if ((rqdata != NULL) && (rqdata->media != 0)) {
    748741      const char valid_media_codes[] =
    749         {0xF0,0xF8,0xF9,0xFA,0xFB,0xFC,0xFD,0xFE,0xFF};
     742        {0xF0,0xF8,0xF9,0xFA,0xFB,0xFC,0xFD,0xFE,0xFF};
    750743      if (NULL==memchr(valid_media_codes,
    751                        rqdata->media,
    752                        sizeof(valid_media_codes))) {
    753         ret_val = -1;
    754         errno = EINVAL;
     744                       rqdata->media,
     745                       sizeof(valid_media_codes))) {
     746        ret_val = -1;
     747        errno = EINVAL;
    755748      }
    756749      else {
    757         fmt_params->media_code = rqdata->media;
     750        fmt_params->media_code = rqdata->media;
    758751      }
    759752    }
     
    789782      int          cnt;
    790783      from = "RTEMS"; /* default: make "from" point to OS Name */
    791     if ((rqdata != NULL) &&
    792         (rqdata->OEMName != NULL)) {
     784    if ((rqdata != NULL) && (rqdata->OEMName != NULL)) {
    793785      from = rqdata->OEMName;
    794786    }
    795     for (cnt = 0;
    796          cnt < (sizeof(fmt_params->OEMName)-1);
    797          cnt++) {
     787    for (cnt = 0; cnt < (sizeof(fmt_params->OEMName)-1); cnt++) {
    798788      if (isprint((unsigned char)*from)) {
    799         *to++ = *from++;
     789        *to++ = *from++;
    800790      }
    801791      else {
    802         /*
    803         * non-printable character in given name, so keep stuck
    804         * at that character and replace all following characters
    805         * with a ' '
    806         */
    807         *to++=' ';
     792        /*
     793        * non-printable character in given name, so keep stuck
     794        * at that character and replace all following characters
     795        * with a ' '
     796        */
     797        *to++=' ';
    808798      }
    809799      *to = '\0';
     
    820810      from = ""; /* default: make "from" point to empty string */
    821811    if ((rqdata != NULL) &&
    822         (rqdata->VolLabel != NULL)) {
     812        (rqdata->VolLabel != NULL)) {
    823813      from = rqdata->VolLabel;
    824814      fmt_params->VolLabel_present = true;
    825815    }
    826     for (cnt = 0;
    827          cnt < (sizeof(fmt_params->VolLabel)-1);
    828          cnt++) {
     816    for (cnt = 0; cnt < (sizeof(fmt_params->VolLabel)-1); cnt++) {
    829817      if (isprint((unsigned char)*from)) {
    830         *to++ = *from++;
     818        *to++ = *from++;
    831819      }
    832820      else {
    833         /*
    834         * non-printable character in given name, so keep stuck
    835         * at that character and replace all following characters
    836         * with a ' '
    837         */
    838         *to++=' ';
     821        /*
     822        * non-printable character in given name, so keep stuck
     823        * at that character and replace all following characters
     824        * with a ' '
     825        */
     826        *to++=' ';
    839827      }
    840828      *to = '\0';
     
    901889   */
    902890  memcpy(FAT_GET_ADDR_BR_OEMNAME(mbr),
    903         fmt_params->OEMName,
    904         FAT_BR_OEMNAME_SIZE);
     891        fmt_params->OEMName,
     892        FAT_BR_OEMNAME_SIZE);
    905893  FAT_SET_BR_BYTES_PER_SECTOR(mbr    , fmt_params->bytes_per_sector);
    906894  FAT_SET_BR_SECTORS_PER_CLUSTER(mbr , fmt_params->sectors_per_cluster);
     
    925913    FAT_SET_BR_VOLID(mbr             , fmt_params->vol_id); /* volume id */
    926914  memcpy(FAT_GET_ADDR_BR_VOLLAB(mbr),
    927         fmt_params->VolLabel,
    928         FAT_BR_VOLLAB_SIZE);
     915        fmt_params->VolLabel,
     916        FAT_BR_VOLLAB_SIZE);
    929917    memcpy(FAT_GET_ADDR_BR_FILSYSTYPE(mbr),
    930            (fmt_params->fattype == FAT_FAT12)
    931            ? "FAT12   "
    932            : "FAT16   ",
    933            FAT_BR_FILSYSTYPE_SIZE);
     918           (fmt_params->fattype == FAT_FAT12)
     919           ? "FAT12   "
     920           : "FAT16   ",
     921           FAT_BR_FILSYSTYPE_SIZE);
    934922  }
    935923  else {
     
    948936    memset(FAT_GET_ADDR_BR_FAT32_VOLLAB(mbr)   ,0,FAT_BR_VOLLAB_SIZE);
    949937    memcpy(FAT_GET_ADDR_BR_FAT32_FILSYSTYPE(mbr),
    950            "FAT32   ",
    951            FAT_BR_FILSYSTYPE_SIZE);
     938           "FAT32   ",
     939           FAT_BR_FILSYSTYPE_SIZE);
    952940  }
    953941  /*
     
    1003991   * write "empty" values for free cluster count and next cluster number
    1004992   */
    1005   FAT_SET_FSINFO_FREE_CLUSTER_COUNT(fsinfo+FAT_FSI_INFO,
    1006                                     0xffffffff);
    1007   FAT_SET_FSINFO_NEXT_FREE_CLUSTER (fsinfo+FAT_FSI_INFO,
    1008                                     0xffffffff);
     993  FAT_SET_FSINFO_FREE_CLUSTER_COUNT(fsinfo+FAT_FSI_INFO, 0xffffffff);
     994  FAT_SET_FSINFO_NEXT_FREE_CLUSTER (fsinfo+FAT_FSI_INFO, 0xffffffff);
    1009995  return 0;
    1010996}
     
    11421128      (fmt_params.fsinfo_sec != 0)) {
    11431129    ret_val = msdos_format_write_sec(fd,
    1144                                      fmt_params.fsinfo_sec,
    1145                                      fmt_params.bytes_per_sector,
    1146                                      tmp_sec);
     1130                                     fmt_params.fsinfo_sec,
     1131                                     fmt_params.bytes_per_sector,
     1132                                     tmp_sec);
    11471133  }
    11481134  /*
     
    12401226                                     fmt_params.skip_alignment);
    12411227    for (i = 0;
    1242         (i < fmt_params.fat_num) && (ret_val == 0);
    1243         i++) {
     1228        (i < fmt_params.fat_num) && (ret_val == 0);
     1229        i++) {
    12441230      ret_val = msdos_format_write_sec
    12451231        (fd,
Note: See TracChangeset for help on using the changeset viewer.