Changeset 86364e8 in rtems


Ignore:
Timestamp:
Dec 5, 2013, 1:52:42 AM (6 years ago)
Author:
Nick Withers <nick.withers@…>
Branches:
4.11, master
Children:
607c854
Parents:
2d5c486
git-author:
Nick Withers <nick.withers@…> (12/05/13 01:52:42)
git-committer:
Gedare Bloom <gedare@…> (12/24/14 03:49:26)
Message:

Correct error return mismatches

Closes #2139

Location:
cpukit/libfs/src/rfs
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • cpukit/libfs/src/rfs/rtems-rfs-buffer.h

    r2d5c486 r86364e8  
    6969  uint32_t               references;
    7070} rtems_rfs_buffer;
    71 #define rtems_rfs_buffer_io_request rtems_rfs_buffer_devceio_request
    72 #define rtems_rfs_buffer_io_release rtems_rfs_uffer_deviceio_release
     71#define rtems_rfs_buffer_io_request rtems_rfs_buffer_deviceio_request
     72#define rtems_rfs_buffer_io_release rtems_rfs_buffer_deviceio_release
    7373
    7474/**
  • cpukit/libfs/src/rfs/rtems-rfs-file-system.c

    r2d5c486 r86364e8  
    302302  }
    303303
    304   errno = 0;
    305304  return 0;
    306305}
  • cpukit/libfs/src/rfs/rtems-rfs-file-system.h

    r2d5c486 r86364e8  
    390390 *
    391391 * @retval 0 Successful operation.
    392  * @retval error_code An error occurred.
     392 * @retval -1 Error. See errno
    393393 */
    394394int rtems_rfs_fs_open (const char*             name,
     
    404404 *
    405405 * @retval 0 Successful operation.
    406  * @retval error_code An error occurred.
     406 * @retval -1 Error. See errno
    407407 */
    408408int rtems_rfs_fs_close (rtems_rfs_file_system* fs);
  • cpukit/libfs/src/rfs/rtems-rfs-format.c

    r2d5c486 r86364e8  
    468468                          RTEMS_RFS_FS_FORCE_OPEN | RTEMS_RFS_FS_NO_LOCAL_CACHE,
    469469                          0, &fs);
    470   if (rc < 0)
    471   {
     470  if (rc != 0)
     471  {
     472    rc = errno;
     473
    472474    printf ("rtems-rfs: format: file system open failed: %d: %s\n",
    473             errno, strerror (errno));
    474     return -1;
     475            rc, strerror (rc));
     476
     477    return rc;
    475478  }
    476479
    477480  rc = rtems_rfs_inode_alloc (fs, RTEMS_RFS_ROOT_INO, &ino);
    478   if (rc > 0)
     481  if (rc != 0)
    479482  {
    480483    printf ("rtems-rfs: format: inode allocation failed: %d: %s\n",
     
    488491    printf ("rtems-rfs: format: allocated inode not root ino: %" PRId32 "\n", ino);
    489492    rtems_rfs_fs_close (fs);
    490     return rc;
     493    return EINVAL;
    491494  }
    492495
    493496  rc = rtems_rfs_inode_open (fs, ino, &inode, true);
    494   if (rc > 0)
     497  if (rc != 0)
    495498  {
    496499    printf ("rtems-rfs: format: inode open failed: %d: %s\n",
     
    505508                                    RTEMS_RFS_S_IXGRP | RTEMS_RFS_S_IXOTH),
    506509                                   0, 0);
    507   if (rc > 0)
     510  if (rc != 0)
    508511    printf ("rtems-rfs: format: inode initialise failed: %d: %s\n",
    509512            rc, strerror (rc));
    510513
    511514  rc = rtems_rfs_dir_add_entry (fs, &inode, ".", 1, ino);
    512   if (rc > 0)
     515  if (rc != 0)
    513516    printf ("rtems-rfs: format: directory add failed: %d: %s\n",
    514517            rc, strerror (rc));
    515518
    516519  rc = rtems_rfs_inode_close (fs, &inode);
    517   if (rc > 0)
     520  if (rc != 0)
    518521    printf ("rtems-rfs: format: inode close failed: %d: %s\n",
    519522            rc, strerror (rc));
    520523
    521524  rc = rtems_rfs_fs_close (fs);
    522   if (rc < 0)
     525  if (rc != 0)
     526  {
     527    rc = errno;
     528
    523529    printf ("rtems-rfs: format: file system close failed: %d: %s\n",
    524             errno, strerror (errno));
    525 
    526   return rc;
     530            rc, strerror (rc));
     531
     532    return rc;
     533  }
     534
     535  return 0;
    527536}
    528537
     
    555564   */
    556565  rc = rtems_rfs_buffer_open (name, &fs);
    557   if (rc > 0)
     566  if (rc != 0)
    558567  {
    559568    printf ("rtems-rfs: format: buffer open failed: %d: %s\n",
    560569            rc, strerror (rc));
     570
     571    errno = rc;
    561572    return -1;
    562573  }
     
    569580    printf ("rtems-rfs: media block is invalid: %" PRIu32 "\n",
    570581            rtems_rfs_fs_media_block_size (&fs));
     582
     583    errno = EINVAL;
    571584    return -1;
    572585  }
     
    576589   */
    577590  if (!rtems_rfs_check_config (&fs, config))
     591  {
     592    errno = EINVAL;
    578593    return -1;
     594  }
    579595
    580596  if (config->verbose)
     
    605621
    606622  rc = rtems_rfs_buffer_setblksize (&fs, rtems_rfs_fs_block_size (&fs));
    607   if (rc > 0)
     623  if (rc != 0)
    608624  {
    609625    printf ("rtems-rfs: format: setting block size failed: %d: %s\n",
    610626            rc, strerror (rc));
     627
     628    errno = rc;
    611629    return -1;
    612630  }
     
    615633  {
    616634    printf ("rtems-rfs: format: superblock write failed\n");
     635
     636    errno = EIO;
    617637    return -1;
    618638  }
     
    621641    if (!rtems_rfs_write_group (&fs, group,
    622642                                config->initialise_inodes, config->verbose))
    623       return -1;
     643      {
     644        errno = EIO;
     645        return -1;
     646      }
    624647
    625648  if (config->verbose)
     
    627650
    628651  rc = rtems_rfs_buffer_close (&fs);
    629   if (rc > 0)
     652  if (rc != 0)
    630653  {
    631654    printf ("rtems-rfs: format: buffer close failed: %d: %s\n",
    632655            rc, strerror (rc));
     656
     657    errno = rc;
    633658    return -1;
    634659  }
    635660
    636661  rc = rtems_rfs_write_root_dir (name);
    637   if (rc > 0)
     662  if (rc != 0)
    638663  {
    639664    printf ("rtems-rfs: format: writing root dir failed: %d: %s\n",
    640665            rc, strerror (rc));
     666
     667    errno = rc;
    641668    return -1;
    642669  }
Note: See TracChangeset for help on using the changeset viewer.