diff options
author | Qu Wenruo <quwenruo@cn.fujitsu.com> | 2015-01-14 14:50:40 +0800 |
---|---|---|
committer | David Sterba <dsterba@suse.cz> | 2015-01-14 15:15:06 +0100 |
commit | e015a83e6d300a3ee972c6ba295e860efd6aa0be (patch) | |
tree | 4ca2b8468009cfc9db4c17e7802612d79d6641e1 | |
parent | 54498f9a14ea34f2c9a1ce82d412fca7a05b05fc (diff) |
btrfs-progs: Fix wrong return value when executing 'fi show' on umounted device.
When executing 'btrfs fi show' on unmounted device, even no problem
happens, the return value is still 1 not 0.
The problem lies in search_umounted_fs_uuids(), where when it finds the
given uuid, it should return 1, but later uuid copy overwrites the return
value, causing it always return 0 under that case.
Fix it by pass found as pointer, and return value only indicates
whether anything wrong happens, whether found or not is stored in the
new parameter.
Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
-rw-r--r-- | cmds-filesystem.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/cmds-filesystem.c b/cmds-filesystem.c index a3cf114f..30a7cc44 100644 --- a/cmds-filesystem.c +++ b/cmds-filesystem.c @@ -700,7 +700,7 @@ static int has_seed_devices(struct btrfs_fs_devices *fs_devices) } static int search_umounted_fs_uuids(struct list_head *all_uuids, - char *search) + char *search, int *found) { struct btrfs_fs_devices *cur_fs, *fs_copy; struct list_head *fs_uuids; @@ -717,7 +717,8 @@ static int search_umounted_fs_uuids(struct list_head *all_uuids, if (search) { if (uuid_search(cur_fs, search) == 0) continue; - ret = 1; + if (found) + *found = 1; } /* skip all fs already shown as mounted fs */ @@ -922,8 +923,8 @@ devs_only: return 1; } - found = search_umounted_fs_uuids(&all_uuids, search); - if (found < 0) { + ret = search_umounted_fs_uuids(&all_uuids, search, &found); + if (ret < 0) { fprintf(stderr, "ERROR: %d while searching target device\n", ret); return 1; |