Changeset e63ce679 in mainline for uspace/lib/ext4/libext4_directory_index.c

Timestamp:

2012-03-03T20:06:31Z (12 years ago)

Author:

Frantisek Princ <frantisek.princ@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

e8d054a

Parents:

7689590

Message:

Some TODOs solved (mostly in error handling)

File:

• uspace/lib/ext4/libext4_directory_index.c (modified) (16 diffs)

uspace/lib/ext4/libext4_directory_index.c

@@ -353 +353 @@
         // Hardcoded number 2 means maximum height of index tree !!!
         ext4_directory_dx_block_t dx_blocks[2];
-        ext4_directory_dx_block_t *dx_block;
+        ext4_directory_dx_block_t *dx_block, *tmp;
         rc = ext4_directory_dx_get_leaf(&hinfo, fs, inode_ref->inode, root_block, &dx_block, dx_blocks);
         if (rc != EOK) {
@@ -367 +367 @@
         rc = ext4_filesystem_get_inode_data_block_index(fs, inode_ref->inode, leaf_block_idx, &leaf_block_addr);
         if (rc != EOK) {
-                return EXT4_ERR_BAD_DX_DIR;
+                goto cleanup;
         }
 
@@ -373 +373 @@
                 rc = block_get(&leaf_block, fs->device, leaf_block_addr, BLOCK_FLAGS_NONE);
                 if (rc != EOK) {
-                        return EXT4_ERR_BAD_DX_DIR;
+                        goto cleanup;
                 }
 
                 ext4_directory_entry_ll_t *res_dentry;
                 rc = ext4_directory_find_in_block(leaf_block, fs->superblock, name_len, name, &res_dentry);
-
 
                 // Found => return it
@@ -387 +386 @@
                 }
 
+                // Not found, leave untouched
                 block_put(leaf_block);
 
-                // ERROR - corrupted index
-                if (rc == -1) {
-                        // TODO cleanup
-                        return EXT4_ERR_BAD_DX_DIR;
+                if (rc != ENOENT) {
+                        goto cleanup;
                 }
 
                 rc = ext4_directory_dx_next_block(fs, inode_ref->inode, hinfo.hash, dx_block, &dx_blocks[0]);
                 if (rc < 0) {
-                        // TODO cleanup
-                        return EXT4_ERR_BAD_DX_DIR;
+                        goto cleanup;
                 }
 
@@ -404 +401 @@
 
         return ENOENT;
+
+cleanup:
+
+        tmp = dx_blocks;
+        while (tmp <= dx_block) {
+                block_put(tmp->block);
+                ++tmp;
+        }
+        return rc;
 }
 
@@ -605 +611 @@
 {
         int rc = EOK;
-        int rc2;
+        int rc2 = EOK;
 
         // get direct block 0 (index root)
@@ -630 +636 @@
         // Hardcoded number 2 means maximum height of index tree !!!
         ext4_directory_dx_block_t dx_blocks[2];
-        ext4_directory_dx_block_t *dx_block;
+        ext4_directory_dx_block_t *dx_block, *dx_it;
         rc = ext4_directory_dx_get_leaf(&hinfo, fs, parent->inode, root_block, &dx_block, dx_blocks);
         if (rc != EOK) {
-                block_put(root_block);
-                return EXT4_ERR_BAD_DX_DIR;
+                rc = EXT4_ERR_BAD_DX_DIR;
+                goto release_index;
         }
 
@@ -643 +649 @@
         rc = ext4_filesystem_get_inode_data_block_index(fs, parent->inode, leaf_block_idx, &leaf_block_addr);
         if (rc != EOK) {
-                return EXT4_ERR_BAD_DX_DIR;
+                goto release_index;
         }
 
@@ -650 +656 @@
         rc = block_get(&target_block, fs->device, leaf_block_addr, BLOCK_FLAGS_NONE);
         if (rc != EOK) {
-                return EXT4_ERR_BAD_DX_DIR;
+                goto release_index;
         }
 
         rc = ext4_directory_try_insert_entry(fs->superblock, target_block, child, name, name_len);
         if (rc == EOK) {
-                goto cleanup;
-
+                goto release_target_index;
         }
 
@@ -679 +684 @@
                 if ((levels > 0) && (root_limit == root_count)) {
                         EXT4FS_DBG("Directory index is full");
-
-                        // ENOSPC - cleanup !!!
-                        return ENOSPC;
+                        rc = ENOSPC;
+                        goto release_target_index;
                 }
 
@@ -688 +692 @@
                 rc =  ext4_directory_append_block(fs, parent, &new_fblock, &new_iblock);
                 if (rc != EOK) {
-                        goto cleanup;
+                        goto release_target_index;
                 }
 
@@ -695 +699 @@
                 rc = block_get(&new_block, fs->device, new_fblock, BLOCK_FLAGS_NOREAD);
                 if (rc != EOK) {
-                        goto cleanup;
+                        goto release_target_index;
                 }
 
@@ -772 +776 @@
         rc = ext4_directory_dx_split_data(fs, parent, &hinfo, target_block, dx_block, &new_block);
         if (rc != EOK) {
-                // TODO error
+                rc2 = rc;
+                goto release_target_index;
         }
 
@@ -788 +793 @@
 
 
+// TODO check rc handling
 terminate:
         rc = block_put(new_block);
@@ -795 +801 @@
 
 
-cleanup:
+release_target_index:
 
         rc2 = rc;
@@ -804 +810 @@
         }
 
-        ext4_directory_dx_block_t *dx_it = dx_blocks;
+release_index:
+        dx_it = dx_blocks;
 
         while (dx_it <= dx_block) {