diff options
| author | Anand Jain <anand.jain@oracle.com> | 2013-04-05 13:55:02 +0800 |
|---|---|---|
| committer | David Sterba <dsterba@suse.cz> | 2013-04-23 18:56:20 +0200 |
| commit | 555ae67390d88e9b2abc91b53f4f6426d4b328df (patch) | |
| tree | 109b380f3eae858575fd532ae712673399479b7a | |
| parent | 4b3c9136be2b72fe4df33c5dfa7a33cf1bce9d8d (diff) | |
btrfs-progs: btrfs-select-super output is confusing when it fails
Trivial patch:
./btrfs-progs/btrfs-select-super -s 0 /dev/sdc
using SB copy 0, bytenr 65536
No valid Btrfs found on /dev/sdc
Open ctree failed
The line 'using..' is confusing which gives an
indication that command is successful
This patch will avoid that when command fails
Signed-off-by: Anand Jain <anand.jain@oracle.com>
| -rw-r--r-- | btrfs-select-super.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/btrfs-select-super.c b/btrfs-select-super.c index 0c4f5c0a..6a458b8d 100644 --- a/btrfs-select-super.c +++ b/btrfs-select-super.c @@ -43,7 +43,7 @@ int main(int ac, char **av) { struct btrfs_root *root; int ret; - int num; + int num = 0; u64 bytenr = 0; while(1) { @@ -55,8 +55,6 @@ int main(int ac, char **av) case 's': num = atol(optarg); bytenr = btrfs_sb_offset(num); - printf("using SB copy %d, bytenr %llu\n", num, - (unsigned long long)bytenr); break; default: print_usage(); @@ -97,5 +95,7 @@ int main(int ac, char **av) * transaction commit. We just want the super copy we pulled off the * disk to overwrite all the other copies */ + printf("using SB copy %d, bytenr %llu\n", num, + (unsigned long long)bytenr); return ret; } |