Changeset d0c9b4b in mainline

Timestamp:

2012-05-24T16:00:54Z (12 years ago)

Author:

Frantisek Princ <frantisek.princ@…>

Branches:

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

Children:

fb04cd90

Parents:

fa3704d

Message:

fixed bug in releasing extents

File:

• uspace/lib/ext4/libext4_extent.c (modified) (5 diffs)

uspace/lib/ext4/libext4_extent.c

@@ -666 +666 @@
 
         // If leaf node is empty, the whole tree must be checked and the node will be released
-        bool check_tree = false;
+        bool remove_parent_record = false;
 
         // Don't release root block (including inode data) !!!
@@ -674 +674 @@
                         goto cleanup;
                 }
-                check_tree = true;
+                remove_parent_record = true;
         }
 
@@ -688 +688 @@
 
                 // Correct entry because of changes in the previous iteration
-                if (check_tree) {
+                if (remove_parent_record) {
                         entries--;
-                        ext4_extent_header_set_entries_count(path_ptr->header, entries);
-                } else {
-                        // TODO check this condition
-                        break;
                 }
 
@@ -704 +700 @@
                         ++index;
                         --entries;
-                        ext4_extent_header_set_entries_count(path_ptr->header, entries);
-                }
+                }
+
+                ext4_extent_header_set_entries_count(path_ptr->header, entries);
 
                 path_ptr->block->dirty = true;
@@ -717 +714 @@
 
                         // Mark parent to be checked
-                        check_tree = true;
+                        remove_parent_record = true;
                 } else {
-                        check_tree = false;
+                        remove_parent_record = false;
                 }