mirror of
https://github.com/kdave/btrfs-progs
synced 2025-02-23 13:26:50 +00:00
btrfs-progs: btrfstune: fix multiple options error
when we use multiple options, error return status will be override by the last option status. example: btrfstune -S 1 -r /dev/loop0 when -S option fails and -r option succeeds, return value is 0, rather than 1, where 1 is the right return status. Reported-by: Chen Hanxiao <chenhanxiao@cn.fujitsu.com> Signed-off-by: Fan Chengniang <fancn.fnst@cn.fujitsu.com> Signed-off-by: David Sterba <dsterba@suse.cz>
This commit is contained in:
parent
27ceeed1a7
commit
6eafa191c0
@ -111,6 +111,7 @@ int main(int argc, char *argv[])
|
||||
{
|
||||
struct btrfs_root *root;
|
||||
int success = 0;
|
||||
int total = 0;
|
||||
int extrefs_flag = 0;
|
||||
int seeding_flag = 0;
|
||||
u64 seeding_value = 0;
|
||||
@ -188,19 +189,22 @@ int main(int argc, char *argv[])
|
||||
ret = update_seeding_flag(root, seeding_value);
|
||||
if (!ret)
|
||||
success++;
|
||||
total++;
|
||||
}
|
||||
|
||||
if (extrefs_flag) {
|
||||
enable_extrefs_flag(root);
|
||||
success++;
|
||||
total++;
|
||||
}
|
||||
|
||||
if (skinny_flag) {
|
||||
enable_skinny_metadata(root);
|
||||
success++;
|
||||
total++;
|
||||
}
|
||||
|
||||
if (success > 0) {
|
||||
if (success == total) {
|
||||
ret = 0;
|
||||
} else {
|
||||
root->fs_info->readonly = 1;
|
||||
|
Loading…
Reference in New Issue
Block a user