Fork us on GitHub Follow us on Facebook Follow us on Twitter

Changeset cfac897 in mainline


Ignore:
Timestamp:
2011-04-21T19:26:34Z (11 years ago)
Author:
Maurizio Lombardi <m.lombardi85@…>
Branches:
lfn, master
Children:
48de019
Parents:
adddd75
Message:

Use the on_error() macro where appropriated

Location:
uspace/srv/fs/minixfs
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • uspace/srv/fs/minixfs/mfs_inode.c

    radddd75 rcfac897  
    234234                                BLOCK_FLAGS_NONE);
    235235
    236         if (r != EOK)
    237                 goto out;
     236        on_error(r, goto out);
    238237
    239238        struct mfs_inode *ino = b->data;
     
    277276                                BLOCK_FLAGS_NONE);
    278277
    279         if (r != EOK)
    280                 goto out;
     278        on_error(r, goto out);
    281279
    282280        struct mfs2_inode *ino2 = b->data;
     
    338336
    339337                r = mfs_alloc_bit(mnode->instance, &new_zone, BMAP_ZONE);
    340                 if (r != EOK)
    341                         return r;
     338                on_error(r, return r);
    342339
    343340                mfsdebug("write_map = %d\n", (int) ((start_zone + i) * bs));
     
    346343                r = block_get(&b, mnode->instance->handle, new_zone,
    347344                                                BLOCK_FLAGS_NOREAD);
    348                 if (r != EOK)
    349                         return r;
     345                on_error(r, return r);
    350346
    351347                memset(b->data, 0, bs);
     
    355351                r = write_map(mnode, (start_zone + i) * bs,
    356352                                new_zone, &dummy);
    357                 if (r != EOK)
    358                         return r;
     353
     354                on_error(r, return r);
    359355
    360356                ino_i->i_size += bs;
  • uspace/srv/fs/minixfs/mfs_ops.c

    radddd75 rcfac897  
    265265
    266266        r = mfs_instance_get(handle, &inst);
    267         if (r != EOK)
    268                 return r;
     267        on_error(r, return r);
    269268
    270269        if (flags & L_DIRECTORY) {
     
    275274        /*Alloc a new inode*/
    276275        r = mfs_alloc_bit(inst, &inum, BMAP_INODE);
    277         if (r != EOK)
    278                 return r;
     276        on_error(r, return r);
    279277
    280278        struct mfs_ino_info *ino_i;
     
    318316
    319317        r = put_inode(mnode);
    320         if (r != EOK)
    321                 goto out_err_2;
     318        on_error(r, goto out_err_2);
    322319
    323320        fs_node_initialize(fsnode);
     
    353350        while (1) {
    354351                r = read_directory_entry(mnode, &d_info, i++);
    355                 if (r != EOK)
    356                         return r;
     352                on_error(r, return r);
    357353
    358354                if (!d_info) {
     
    395391void mfs_stat(ipc_callid_t rid, ipc_call_t *request)
    396392{
    397         mfsdebug("mfs_stat()\n");
    398393        libfs_stat(&mfs_libfs_ops, mfs_reg.fs_handle, rid, request);
    399394}
     
    406401
    407402        rc = mfs_instance_get(devmap_handle, &instance);
    408 
    409         if (rc != EOK)
    410                 return rc;
     403        on_error(rc, return rc);
    411404
    412405        return mfs_node_core_get(rfn, instance, index);
     
    477470
    478471        rc = get_inode(inst, &ino_i, index);
    479         if (rc != EOK)
    480                 goto out_err;
     472        on_error(rc, goto out_err);
    481473
    482474        ino_i->index = index;
     
    557549        while (1) {
    558550                r = read_directory_entry(mnode, &d_info, i++);
    559                 if (r != EOK)
    560                         return r;
     551                on_error(r, return r);
    561552
    562553                if (!d_info) {
     
    627618                while (1) {
    628619                        rc = read_directory_entry(mnode, &d_info, pos);
    629                         if (rc != EOK)
    630                                 goto out_error;
     620                        on_error(rc, goto out_error);
    631621
    632622                        if (!d_info) {
     
    670660
    671661                rc = read_map(&zone, mnode, pos);
    672                 if (rc != EOK)
    673                         goto out_error;
     662                on_error(rc, goto out_error);
    674663
    675664                if (zone == 0) {
     
    688677
    689678                rc = block_get(&b, handle, zone, BLOCK_FLAGS_NONE);
    690                 if (rc != EOK)
    691                         goto out_error;
     679                on_error(rc, goto out_error);
    692680
    693681                async_data_read_finalize(callid, b->data +
     
    738726                                mfs_version_t *version, bool *longfilenames)
    739727{
    740         bool rc = false;
     728        bool rc = true;
    741729        *longfilenames = false;
    742730
     
    744732                *native = magic == MFS_MAGIC_V1;
    745733                *version = MFS_VERSION_V1;
    746                 rc = true;
    747734        } else if (magic == MFS_MAGIC_V1L || magic == MFS_MAGIC_V1LR) {
    748735                *native = magic == MFS_MAGIC_V1L;
    749736                *version = MFS_VERSION_V1;
    750737                *longfilenames = true;
    751                 rc = true;
    752738        } else if (magic == MFS_MAGIC_V2 || magic == MFS_MAGIC_V2R) {
    753739                *native = magic == MFS_MAGIC_V2;
    754740                *version = MFS_VERSION_V2;
    755                 rc = true;
    756741        } else if (magic == MFS_MAGIC_V2L || magic == MFS_MAGIC_V2LR) {
    757742                *native = magic == MFS_MAGIC_V2L;
    758743                *version = MFS_VERSION_V2;
    759744                *longfilenames = true;
    760                 rc = true;
    761745        } else if (magic == MFS_MAGIC_V3 || magic == MFS_MAGIC_V3R) {
    762746                *native = magic == MFS_MAGIC_V3;
    763747                *version = MFS_VERSION_V3;
    764                 rc = true;
    765         }
     748        } else
     749                rc = false;
    766750
    767751        return rc;
Note: See TracChangeset for help on using the changeset viewer.