btrfs-progs: fix return value problem for btrfs sub show
authorGui Hecheng <guihc.fnst@cn.fujitsu.com>
Thu, 27 Nov 2014 02:01:34 +0000 (10:01 +0800)
committerDavid Sterba <dsterba@suse.cz>
Thu, 27 Nov 2014 17:36:55 +0000 (18:36 +0100)
commit05afee34683f5a57fba0b2f55375bef592bb3987
tree1abba30cd4e5051321928e23ee75f26578b75f97
parent15ed5e2a91ff8e02ae600044b3d38dad1917fc7c
btrfs-progs: fix return value problem for btrfs sub show

If you exec:
# btrfs sub show <dir> <== non-subvolume dir
The cmd print error messages as expected, but returns 0.
By convetion, it should return non-zero and we should explicitly
set it before it goto out.

With other pieces adopted:
1) removed a unnecessary return value set -EINVAL
2) fixed another code branch which may return 0 upon error.
3) with 2) applied, the ret = 0 follows can be removed

Signed-off-by: Gui Hecheng <guihc.fnst@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
cmds-subvolume.c