Whamcloud - gitweb
ext2fs: don't ignore return value in ext2fs_count_blocks
[tools/e2fsprogs.git] / lib / ext2fs / extent.c
index fbb0a74..9e61103 100644 (file)
@@ -29,6 +29,8 @@
 #include "ext2fsP.h"
 #include "e2image.h"
 
+#undef DEBUG
+
 /*
  * Definitions to be dropped in lib/ext2fs/ext2fs.h
  */
@@ -58,6 +60,7 @@ struct ext2_extent_handle {
        int                     type;
        int                     level;
        int                     max_depth;
+       int                     max_paths;
        struct extent_path      *path;
 };
 
@@ -121,11 +124,39 @@ static void dbg_print_extent(char *desc, struct ext2fs_extent *extent)
 
 }
 
+static void dump_path(const char *tag, struct ext2_extent_handle *handle,
+                     struct extent_path *path)
+{
+       struct extent_path *ppp = path;
+       printf("%s: level=%d\n", tag, handle->level);
+
+       do {
+               printf("%s: path=%ld buf=%p entries=%d max_entries=%d left=%d "
+                      "visit_num=%d flags=0x%x end_blk=%llu curr=%p(%ld)\n",
+                      tag, (ppp - handle->path), ppp->buf, ppp->entries,
+                      ppp->max_entries, ppp->left, ppp->visit_num, ppp->flags,
+                      ppp->end_blk, ppp->curr, ppp->curr - (void *)ppp->buf);
+               printf("  ");
+               dbg_show_header((struct ext3_extent_header *)ppp->buf);
+               if (ppp->curr) {
+                       printf("  ");
+                       dbg_show_index(ppp->curr);
+                       printf("  ");
+                       dbg_show_extent(ppp->curr);
+               }
+               ppp--;
+       } while (ppp >= handle->path);
+       fflush(stdout);
+
+       return;
+}
+
 #else
 #define dbg_show_header(eh) do { } while (0)
 #define dbg_show_index(ix) do { } while (0)
 #define dbg_show_extent(ex) do { } while (0)
 #define dbg_print_extent(desc, ex) do { } while (0)
+#define dump_path(tag, handle, path) do { } while (0)
 #endif
 
 /*
@@ -160,7 +191,7 @@ errcode_t ext2fs_extent_header_verify(void *ptr, int size)
 /*
  * Begin functions to handle an inode's extent information
  */
-extern void ext2fs_extent_free(ext2_extent_handle_t handle)
+void ext2fs_extent_free(ext2_extent_handle_t handle)
 {
        int                     i;
 
@@ -168,7 +199,7 @@ extern void ext2fs_extent_free(ext2_extent_handle_t handle)
                return;
 
        if (handle->path) {
-               for (i=1; i <= handle->max_depth; i++) {
+               for (i = 1; i < handle->max_paths; i++) {
                        if (handle->path[i].buf)
                                ext2fs_free_mem(&handle->path[i].buf);
                }
@@ -177,13 +208,13 @@ extern void ext2fs_extent_free(ext2_extent_handle_t handle)
        ext2fs_free_mem(&handle);
 }
 
-extern errcode_t ext2fs_extent_open(ext2_filsys fs, ext2_ino_t ino,
+errcode_t ext2fs_extent_open(ext2_filsys fs, ext2_ino_t ino,
                                    ext2_extent_handle_t *ret_handle)
 {
        return ext2fs_extent_open2(fs, ino, NULL, ret_handle);
 }
 
-extern errcode_t ext2fs_extent_open2(ext2_filsys fs, ext2_ino_t ino,
+errcode_t ext2fs_extent_open2(ext2_filsys fs, ext2_ino_t ino,
                                    struct ext2_inode *inode,
                                    ext2_extent_handle_t *ret_handle)
 {
@@ -242,11 +273,10 @@ extern errcode_t ext2fs_extent_open2(ext2_filsys fs, ext2_ino_t ino,
        handle->max_depth = ext2fs_le16_to_cpu(eh->eh_depth);
        handle->type = ext2fs_le16_to_cpu(eh->eh_magic);
 
-       retval = ext2fs_get_mem(((handle->max_depth+1) *
-                                sizeof(struct extent_path)),
-                               &handle->path);
-       memset(handle->path, 0,
-              (handle->max_depth+1) * sizeof(struct extent_path));
+       handle->max_paths = handle->max_depth + 1;
+       retval = ext2fs_get_memzero(handle->max_paths *
+                                   sizeof(struct extent_path),
+                                   &handle->path);
        handle->path[0].buf = (char *) handle->inode->i_block;
 
        handle->path[0].left = handle->path[0].entries =
@@ -283,6 +313,7 @@ errcode_t ext2fs_extent_get(ext2_extent_handle_t handle,
        blk64_t                         blk;
        blk64_t                         end_blk;
        int                             orig_op, op;
+       int                             failed_csum = 0;
 
        EXT2_CHECK_MAGIC(handle, EXT2_ET_MAGIC_EXTENT_HANDLE);
 
@@ -457,10 +488,8 @@ retry:
 
                if (!(handle->fs->flags & EXT2_FLAG_IGNORE_CSUM_ERRORS) &&
                    !ext2fs_extent_block_csum_verify(handle->fs, handle->ino,
-                                                    eh)) {
-                       handle->level--;
-                       return EXT2_ET_EXTENT_CSUM_INVALID;
-               }
+                                                    eh))
+                       failed_csum = 1;
 
                newpath->left = newpath->entries =
                        ext2fs_le16_to_cpu(eh->eh_entries);
@@ -540,6 +569,9 @@ retry:
             (path->left != 0)))
                goto retry;
 
+       if (failed_csum)
+               return EXT2_ET_EXTENT_CSUM_INVALID;
+
        return 0;
 }
 
@@ -720,7 +752,14 @@ errcode_t ext2fs_extent_goto(ext2_extent_handle_t handle,
  * and so on.
  *
  * Safe to call for any position in node; if not at the first entry,
- * will  simply return.
+ * it will simply return.
+ *
+ * Note a subtlety of this function -- if there happen to be two extents
+ * mapping the same lblk and someone calls fix_parents on the second of the two
+ * extents, the position of the extent handle after the call will be the second
+ * extent if nothing happened, or the first extent if something did.  A caller
+ * in this situation must use ext2fs_extent_goto() after calling this function.
+ * Or simply don't map the same lblk with two extents, ever.
  */
 errcode_t ext2fs_extent_fix_parents(ext2_extent_handle_t handle)
 {
@@ -830,12 +869,31 @@ errcode_t ext2fs_extent_replace(ext2_extent_handle_t handle,
        return 0;
 }
 
+static int splitting_at_eof(struct ext2_extent_handle *handle,
+                           struct extent_path *path)
+{
+       struct extent_path *ppp = path;
+       dump_path(__func__, handle, path);
+
+       if (handle->level == 0)
+               return 0;
+
+       do {
+               if (ppp->left)
+                       return 0;
+               ppp--;
+       } while (ppp >= handle->path);
+
+       return 1;
+}
+
 /*
  * allocate a new block, move half the current node to it, and update parent
  *
  * handle will be left pointing at original record.
  */
-errcode_t ext2fs_extent_node_split(ext2_extent_handle_t handle)
+static errcode_t extent_node_split(ext2_extent_handle_t handle,
+                                  int expand_allowed)
 {
        errcode_t                       retval = 0;
        blk64_t                         new_node_pblk;
@@ -850,6 +908,7 @@ errcode_t ext2fs_extent_node_split(ext2_extent_handle_t handle)
        int                             tocopy;
        int                             new_root = 0;
        struct ext2_extent_info         info;
+       int                             no_balance;
 
        /* basic sanity */
        EXT2_CHECK_MAGIC(handle, EXT2_ET_MAGIC_EXTENT_HANDLE);
@@ -875,6 +934,25 @@ errcode_t ext2fs_extent_node_split(ext2_extent_handle_t handle)
        orig_height = info.max_depth - info.curr_level;
        orig_lblk = extent.e_lblk;
 
+       /* Try to put the index block before the first extent */
+       path = handle->path + handle->level;
+       eh = (struct ext3_extent_header *) path->buf;
+       if (handle->level == handle->max_depth) {
+               struct ext3_extent      *ex;
+
+               ex = EXT_FIRST_EXTENT(eh);
+               goal_blk = ext2fs_le32_to_cpu(ex->ee_start) +
+                       ((__u64) ext2fs_le16_to_cpu(ex->ee_start_hi) << 32);
+       } else {
+               struct ext3_extent_idx  *ix;
+
+               ix = EXT_FIRST_INDEX(eh);
+               goal_blk = ext2fs_le32_to_cpu(ix->ei_leaf) +
+                       ((__u64) ext2fs_le16_to_cpu(ix->ei_leaf_hi) << 32);
+       }
+       goal_blk -= EXT2FS_CLUSTER_RATIO(handle->fs);
+       goal_blk &= ~EXT2FS_CLUSTER_MASK(handle->fs);
+
        /* Is there room in the parent for a new entry? */
        if (handle->level &&
                        (handle->path[handle->level - 1].entries >=
@@ -888,9 +966,8 @@ errcode_t ext2fs_extent_node_split(ext2_extent_handle_t handle)
                retval = ext2fs_extent_get(handle, EXT2_EXTENT_UP, &extent);
                if (retval)
                        goto done;
-               goal_blk = extent.e_pblk;
 
-               retval = ext2fs_extent_node_split(handle);
+               retval = extent_node_split(handle, expand_allowed);
                if (retval)
                        goto done;
 
@@ -905,6 +982,14 @@ errcode_t ext2fs_extent_node_split(ext2_extent_handle_t handle)
        if (!path->curr)
                return EXT2_ET_NO_CURRENT_NODE;
 
+       /*
+        * Normally, we try to split a full node in half.  This doesn't turn
+        * out so well if we're tacking extents on the end of the file because
+        * then we're stuck with a tree of half-full extent blocks.  This of
+        * course doesn't apply to the root level.
+        */
+       no_balance = expand_allowed ? splitting_at_eof(handle, path) : 0;
+
        /* extent header of the current node we'll split */
        eh = (struct ext3_extent_header *)path->buf;
 
@@ -912,15 +997,16 @@ errcode_t ext2fs_extent_node_split(ext2_extent_handle_t handle)
        if (handle->level == 0) {
                new_root = 1;
                tocopy = ext2fs_le16_to_cpu(eh->eh_entries);
-               retval = ext2fs_get_mem(((handle->max_depth+2) *
-                                        sizeof(struct extent_path)),
-                                       &newpath);
+               retval = ext2fs_get_memzero((handle->max_paths + 1) *
+                                           sizeof(struct extent_path),
+                                           &newpath);
                if (retval)
                        goto done;
-               memset(newpath, 0,
-                      ((handle->max_depth+2) * sizeof(struct extent_path)));
        } else {
-               tocopy = ext2fs_le16_to_cpu(eh->eh_entries) / 2;
+               if (no_balance)
+                       tocopy = 1;
+               else
+                       tocopy = ext2fs_le16_to_cpu(eh->eh_entries) / 2;
        }
 
 #ifdef DEBUG
@@ -929,7 +1015,7 @@ errcode_t ext2fs_extent_node_split(ext2_extent_handle_t handle)
                                handle->level);
 #endif
 
-       if (!tocopy) {
+       if (!tocopy && !no_balance) {
 #ifdef DEBUG
                printf("Nothing to copy to new block!\n");
 #endif
@@ -944,14 +1030,9 @@ errcode_t ext2fs_extent_node_split(ext2_extent_handle_t handle)
                goto done;
        }
 
-       if (!goal_blk) {
-               dgrp_t  group = ext2fs_group_of_ino(handle->fs, handle->ino);
-               __u8    log_flex = handle->fs->super->s_log_groups_per_flex;
-
-               if (log_flex)
-                       group = group & ~((1 << (log_flex)) - 1);
-               goal_blk = ext2fs_group_first_block2(handle->fs, group);
-       }
+       if (!goal_blk)
+               goal_blk = ext2fs_find_inode_goal(handle->fs, handle->ino,
+                                                 handle->inode, 0);
        retval = ext2fs_alloc_block2(handle->fs, goal_blk, block_buf,
                                    &new_node_pblk);
        if (retval)
@@ -996,13 +1077,14 @@ errcode_t ext2fs_extent_node_split(ext2_extent_handle_t handle)
        /* current path now has fewer active entries, we copied some out */
        if (handle->level == 0) {
                memcpy(newpath, path,
-                      sizeof(struct extent_path) * (handle->max_depth+1));
+                      sizeof(struct extent_path) * handle->max_paths);
                handle->path = newpath;
                newpath = path;
                path = handle->path;
                path->entries = 1;
                path->left = path->max_entries - 1;
                handle->max_depth++;
+               handle->max_paths++;
                eh->eh_depth = ext2fs_cpu_to_le16(handle->max_depth);
        } else {
                path->entries -= tocopy;
@@ -1053,8 +1135,7 @@ errcode_t ext2fs_extent_node_split(ext2_extent_handle_t handle)
                goto done;
 
        /* new node hooked in, so update inode block count (do this here?) */
-       handle->inode->i_blocks += (handle->fs->blocksize *
-                                   EXT2FS_CLUSTER_RATIO(handle->fs)) / 512;
+       ext2fs_iblk_add_blocks(handle->fs, handle->inode, 1);
        retval = ext2fs_write_inode(handle->fs, handle->ino,
                                    handle->inode);
        if (retval)
@@ -1068,6 +1149,11 @@ done:
        return retval;
 }
 
+errcode_t ext2fs_extent_node_split(ext2_extent_handle_t handle)
+{
+       return extent_node_split(handle, 0);
+}
+
 errcode_t ext2fs_extent_insert(ext2_extent_handle_t handle, int flags,
                                      struct ext2fs_extent *extent)
 {
@@ -1099,7 +1185,7 @@ errcode_t ext2fs_extent_insert(ext2_extent_handle_t handle, int flags,
                        printf("node full (level %d) - splitting\n",
                                   handle->level);
 #endif
-                       retval = ext2fs_extent_node_split(handle);
+                       retval = extent_node_split(handle, 1);
                        if (retval)
                                return retval;
                        path = handle->path + handle->level;
@@ -1113,8 +1199,10 @@ errcode_t ext2fs_extent_insert(ext2_extent_handle_t handle, int flags,
                        ix++;
                        path->left--;
                }
-       } else
+       } else {
                ix = EXT_FIRST_INDEX(eh);
+               path->left = -1;
+       }
 
        path->curr = ix;
 
@@ -1378,17 +1466,25 @@ errcode_t ext2fs_extent_set_bmap(ext2_extent_handle_t handle,
                                                               &next_extent);
                                if (retval)
                                        goto done;
-                               retval = ext2fs_extent_fix_parents(handle);
-                               if (retval)
-                                       goto done;
                        } else
                                retval = ext2fs_extent_insert(handle,
                                      EXT2_EXTENT_INSERT_AFTER, &newextent);
                        if (retval)
                                goto done;
-                       /* Now pointing at inserted extent; move back to prev */
+                       retval = ext2fs_extent_fix_parents(handle);
+                       if (retval)
+                               goto done;
+                       /*
+                        * Now pointing at inserted extent; move back to prev.
+                        *
+                        * We cannot use EXT2_EXTENT_PREV to go back; note the
+                        * subtlety in the comment for fix_parents().
+                        */
+                       retval = ext2fs_extent_goto(handle, logical);
+                       if (retval)
+                               goto done;
                        retval = ext2fs_extent_get(handle,
-                                                  EXT2_EXTENT_PREV_LEAF,
+                                                  EXT2_EXTENT_CURRENT,
                                                   &extent);
                        if (retval)
                                goto done;
@@ -1421,6 +1517,9 @@ errcode_t ext2fs_extent_set_bmap(ext2_extent_handle_t handle,
                                                              0, &newextent);
                        if (retval)
                                goto done;
+                       retval = ext2fs_extent_fix_parents(handle);
+                       if (retval)
+                               goto done;
                        retval = ext2fs_extent_get(handle,
                                                   EXT2_EXTENT_NEXT_LEAF,
                                                   &extent);
@@ -1437,14 +1536,18 @@ errcode_t ext2fs_extent_set_bmap(ext2_extent_handle_t handle,
                if (retval)
                        goto done;
        } else {
-               __u32   orig_length;
+               __u32   save_length;
+               blk64_t save_lblk;
+               struct ext2fs_extent save_extent;
+               errcode_t r2;
 
 #ifdef DEBUG
                printf("(re/un)mapping in middle of extent\n");
 #endif
                /* need to split this extent; later */
-
-               orig_length = extent.e_len;
+               save_lblk = extent.e_lblk;
+               save_length = extent.e_len;
+               save_extent = extent;
 
                /* shorten pre-split extent */
                extent.e_len = (logical - extent.e_lblk);
@@ -1456,17 +1559,33 @@ errcode_t ext2fs_extent_set_bmap(ext2_extent_handle_t handle,
                        /* insert new extent after current */
                        retval = ext2fs_extent_insert(handle,
                                        EXT2_EXTENT_INSERT_AFTER, &newextent);
-                       if (retval)
+                       if (retval) {
+                               r2 = ext2fs_extent_goto(handle, save_lblk);
+                               if (r2 == 0)
+                                       (void)ext2fs_extent_replace(handle, 0,
+                                                             &save_extent);
                                goto done;
+                       }
                }
                /* add post-split extent */
                extent.e_pblk += extent.e_len + 1;
                extent.e_lblk += extent.e_len + 1;
-               extent.e_len = orig_length - extent.e_len - 1;
+               extent.e_len = save_length - extent.e_len - 1;
                retval = ext2fs_extent_insert(handle,
                                EXT2_EXTENT_INSERT_AFTER, &extent);
-               if (retval)
+               if (retval) {
+                       if (physical) {
+                               r2 = ext2fs_extent_goto(handle,
+                                                       newextent.e_lblk);
+                               if (r2 == 0)
+                                       (void)ext2fs_extent_delete(handle, 0);
+                       }
+                       r2 = ext2fs_extent_goto(handle, save_lblk);
+                       if (r2 == 0)
+                               (void)ext2fs_extent_replace(handle, 0,
+                                                           &save_extent);
                        goto done;
+               }
        }
 
 done:
@@ -1545,8 +1664,10 @@ errcode_t ext2fs_extent_delete(ext2_extent_handle_t handle, int flags)
        } else {
                eh = (struct ext3_extent_header *) path->buf;
                eh->eh_entries = ext2fs_cpu_to_le16(path->entries);
-               if ((path->entries == 0) && (handle->level == 0))
-                       eh->eh_depth = handle->max_depth = 0;
+               if ((path->entries == 0) && (handle->level == 0)) {
+                       eh->eh_depth = 0;
+                       handle->max_depth = 0;
+               }
                retval = update_path(handle);
        }
        return retval;
@@ -1576,10 +1697,157 @@ errcode_t ext2fs_extent_get_info(ext2_extent_handle_t handle,
 
        info->curr_level = handle->level;
        info->max_depth = handle->max_depth;
-       info->max_lblk = ((__u64) 1 << 32) - 1;
-       info->max_pblk = ((__u64) 1 << 48) - 1;
-       info->max_len = (1UL << 15);
-       info->max_uninit_len = (1UL << 15) - 1;
+       info->max_lblk = EXT_MAX_EXTENT_LBLK;
+       info->max_pblk = EXT_MAX_EXTENT_PBLK;
+       info->max_len = EXT_INIT_MAX_LEN;
+       info->max_uninit_len = EXT_UNINIT_MAX_LEN;
+
+       return 0;
+}
+
+static int ul_log2(unsigned long arg)
+{
+       int     l = 0;
+
+       arg >>= 1;
+       while (arg) {
+               l++;
+               arg >>= 1;
+       }
+       return l;
+}
+
+size_t ext2fs_max_extent_depth(ext2_extent_handle_t handle)
+{
+       size_t iblock_sz = sizeof(((struct ext2_inode *)NULL)->i_block);
+       size_t iblock_extents = (iblock_sz - sizeof(struct ext3_extent_header)) /
+                               sizeof(struct ext3_extent);
+       size_t extents_per_block = (handle->fs->blocksize -
+                                   sizeof(struct ext3_extent_header)) /
+                                  sizeof(struct ext3_extent);
+       static unsigned int last_blocksize = 0;
+       static size_t last_result = 0;
+
+       if (last_blocksize && last_blocksize == handle->fs->blocksize)
+               return last_result;
+
+       last_result = 1 + ((ul_log2(EXT_MAX_EXTENT_LBLK) - ul_log2(iblock_extents)) /
+                   ul_log2(extents_per_block));
+       last_blocksize = handle->fs->blocksize;
+       return last_result;
+}
+
+errcode_t ext2fs_fix_extents_checksums(ext2_filsys fs, ext2_ino_t ino,
+                                      struct ext2_inode *inode)
+{
+       ext2_extent_handle_t    handle;
+       struct ext2fs_extent    extent;
+       errcode_t               errcode;
+       int                     save_flags = fs->flags;
+
+       if (!ext2fs_has_feature_metadata_csum(fs->super) ||
+           (inode && !(inode->i_flags & EXT4_EXTENTS_FL)))
+               return 0;
+
+       errcode = ext2fs_extent_open2(fs, ino, inode, &handle);
+       if (errcode) {
+               if (errcode == EXT2_ET_INODE_NOT_EXTENT)
+                       errcode = 0;
+               return errcode;
+       }
+
+       fs->flags &= ~EXT2_FLAG_IGNORE_CSUM_ERRORS;
+       errcode = ext2fs_extent_get(handle, EXT2_EXTENT_ROOT, &extent);
+       if (errcode)
+               goto out;
+
+       do {
+               /* Skip to the end of a block of leaf nodes */
+               if (extent.e_flags & EXT2_EXTENT_FLAGS_LEAF) {
+                       errcode = ext2fs_extent_get(handle,
+                                                   EXT2_EXTENT_LAST_SIB,
+                                                   &extent);
+                       if (errcode)
+                               break;
+               }
+
+               errcode = ext2fs_extent_get(handle, EXT2_EXTENT_NEXT, &extent);
+               if (errcode == EXT2_ET_EXTENT_CSUM_INVALID)
+                       errcode = update_path(handle);
+       } while (errcode == 0);
+
+out:
+       /* Ok if we run off the end */
+       if (errcode == EXT2_ET_EXTENT_NO_NEXT)
+               errcode = 0;
+       ext2fs_extent_free(handle);
+       fs->flags = save_flags;
+       return errcode;
+}
+
+errcode_t ext2fs_decode_extent(struct ext2fs_extent *to, void *addr, int len)
+{
+       struct ext3_extent *from = (struct ext3_extent *)addr;
+
+       if (len != sizeof(struct ext3_extent))
+               return EXT2_ET_INVALID_ARGUMENT;
+
+       to->e_pblk = ext2fs_le32_to_cpu(from->ee_start) +
+               ((__u64) ext2fs_le16_to_cpu(from->ee_start_hi)
+                       << 32);
+       to->e_lblk = ext2fs_le32_to_cpu(from->ee_block);
+       to->e_len = ext2fs_le16_to_cpu(from->ee_len);
+       to->e_flags |= EXT2_EXTENT_FLAGS_LEAF;
+       if (to->e_len > EXT_INIT_MAX_LEN) {
+               to->e_len -= EXT_INIT_MAX_LEN;
+               to->e_flags |= EXT2_EXTENT_FLAGS_UNINIT;
+       }
+
+       return 0;
+}
+
+errcode_t ext2fs_count_blocks(ext2_filsys fs, ext2_ino_t ino,
+                             struct ext2_inode *inode, blk64_t *ret_count)
+{
+       ext2_extent_handle_t    handle = NULL;
+       struct ext2fs_extent    extent;
+       errcode_t               errcode;
+       int                     i;
+       blk64_t                 blkcount = 0;
+       blk64_t                 *intermediate_nodes;
+
+       errcode = ext2fs_extent_open2(fs, ino, inode, &handle);
+       if (errcode)
+               goto out;
+
+       errcode = ext2fs_extent_get(handle, EXT2_EXTENT_ROOT, &extent);
+       if (errcode)
+               goto out;
+
+       errcode = ext2fs_get_array(handle->max_depth, sizeof(blk64_t),
+                                  &intermediate_nodes);
+       if (errcode)
+               goto out;
+
+       blkcount = handle->level;
+       while (!errcode) {
+               if (extent.e_flags & EXT2_EXTENT_FLAGS_LEAF) {
+                       blkcount += extent.e_len;
+                       for (i = 0; i < handle->level; i++) {
+                               if (intermediate_nodes[i] !=
+                                       handle->path[i].end_blk) {
+                                       blkcount++;
+                                       intermediate_nodes[i] =
+                                               handle->path[i].end_blk;
+                               }
+                       }
+               }
+               errcode = ext2fs_extent_get(handle, EXT2_EXTENT_NEXT, &extent);
+       }
+       ext2fs_free_mem(&intermediate_nodes);
+out:
+       *ret_count = blkcount;
+       ext2fs_extent_free(handle);
 
        return 0;
 }