From 555ae67390d88e9b2abc91b53f4f6426d4b328df Mon Sep 17 00:00:00 2001 From: Anand Jain Date: Fri, 5 Apr 2013 13:55:02 +0800 Subject: [PATCH] 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 --- btrfs-select-super.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/btrfs-select-super.c b/btrfs-select-super.c index 0c4f5c0..6a458b8 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; } -- 2.7.4