In restore_metadump(), we call stat() but never uses the result get. This call site is left by some code refactor, as the stat() call is now moved into fixup_device_size().
So we can safely remove the call. Signed-off-by: Qu Wenruo <w...@suse.com> --- image/main.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/image/main.c b/image/main.c index 48070e52c21f..24393188e5e3 100644 --- a/image/main.c +++ b/image/main.c @@ -2690,7 +2690,6 @@ static int restore_metadump(const char *input, FILE *out, int old_restore, if (!ret && !multi_devices && !old_restore && btrfs_super_num_devices(mdrestore.original_super) != 1) { struct btrfs_root *root; - struct stat st; root = open_ctree_fd(fileno(out), target, 0, OPEN_CTREE_PARTIAL | @@ -2703,13 +2702,6 @@ static int restore_metadump(const char *input, FILE *out, int old_restore, } info = root->fs_info; - if (stat(target, &st)) { - error("stat %s failed: %m", target); - close_ctree(info->chunk_root); - free(cluster); - return 1; - } - ret = fixup_chunks_and_devices(info, &mdrestore, fileno(out)); close_ctree(info->chunk_root); if (ret) -- 2.30.1