Btrfs-progs: fix magic return value in cmds-qgroup.c

Signed-off-by: Wang Shilong <wangsl.fnst@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
master
Wang Shilong 2013-09-04 23:22:25 +08:00 committed by Chris Mason
parent 7d16b6ca8f
commit c941ac9b8a
1 changed files with 12 additions and 14 deletions

View File

@ -54,12 +54,12 @@ static int qgroup_assign(int assign, int argc, char **argv)
*/ */
if ((args.src >> 48) >= (args.dst >> 48)) { if ((args.src >> 48) >= (args.dst >> 48)) {
fprintf(stderr, "ERROR: bad relation requested '%s'\n", path); fprintf(stderr, "ERROR: bad relation requested '%s'\n", path);
return 12; return 1;
} }
fd = open_file_or_dir(path, &dirstream); fd = open_file_or_dir(path, &dirstream);
if (fd < 0) { if (fd < 0) {
fprintf(stderr, "ERROR: can't access '%s'\n", path); fprintf(stderr, "ERROR: can't access '%s'\n", path);
return 12; return 1;
} }
ret = ioctl(fd, BTRFS_IOC_QGROUP_ASSIGN, &args); ret = ioctl(fd, BTRFS_IOC_QGROUP_ASSIGN, &args);
@ -68,7 +68,7 @@ static int qgroup_assign(int assign, int argc, char **argv)
if (ret < 0) { if (ret < 0) {
fprintf(stderr, "ERROR: unable to assign quota group: %s\n", fprintf(stderr, "ERROR: unable to assign quota group: %s\n",
strerror(e)); strerror(e));
return 30; return 1;
} }
return 0; return 0;
} }
@ -92,7 +92,7 @@ static int qgroup_create(int create, int argc, char **argv)
fd = open_file_or_dir(path, &dirstream); fd = open_file_or_dir(path, &dirstream);
if (fd < 0) { if (fd < 0) {
fprintf(stderr, "ERROR: can't access '%s'\n", path); fprintf(stderr, "ERROR: can't access '%s'\n", path);
return 12; return 1;
} }
ret = ioctl(fd, BTRFS_IOC_QGROUP_CREATE, &args); ret = ioctl(fd, BTRFS_IOC_QGROUP_CREATE, &args);
@ -101,7 +101,7 @@ static int qgroup_create(int create, int argc, char **argv)
if (ret < 0) { if (ret < 0) {
fprintf(stderr, "ERROR: unable to create quota group: %s\n", fprintf(stderr, "ERROR: unable to create quota group: %s\n",
strerror(e)); strerror(e));
return 30; return 1;
} }
return 0; return 0;
} }
@ -310,19 +310,17 @@ static int cmd_qgroup_show(int argc, char **argv)
fd = open_file_or_dir(path, &dirstream); fd = open_file_or_dir(path, &dirstream);
if (fd < 0) { if (fd < 0) {
fprintf(stderr, "ERROR: can't access '%s'\n", path); fprintf(stderr, "ERROR: can't access '%s'\n", path);
return 12; return 1;
} }
ret = list_qgroups(fd); ret = list_qgroups(fd);
e = errno; e = errno;
close_file_or_dir(fd, dirstream); close_file_or_dir(fd, dirstream);
if (ret < 0) { if (ret < 0)
fprintf(stderr, "ERROR: can't list qgroups: %s\n", fprintf(stderr, "ERROR: can't list qgroups: %s\n",
strerror(e)); strerror(e));
return 30;
}
return ret; return !!ret;
} }
static const char * const cmd_qgroup_limit_usage[] = { static const char * const cmd_qgroup_limit_usage[] = {
@ -392,12 +390,12 @@ static int cmd_qgroup_limit(int argc, char **argv)
ret = test_issubvolume(path); ret = test_issubvolume(path);
if (ret < 0) { if (ret < 0) {
fprintf(stderr, "ERROR: error accessing '%s'\n", path); fprintf(stderr, "ERROR: error accessing '%s'\n", path);
return 12; return 1;
} }
if (!ret) { if (!ret) {
fprintf(stderr, "ERROR: '%s' is not a subvolume\n", fprintf(stderr, "ERROR: '%s' is not a subvolume\n",
path); path);
return 13; return 1;
} }
/* /*
* keep qgroupid at 0, this indicates that the subvolume the * keep qgroupid at 0, this indicates that the subvolume the
@ -412,7 +410,7 @@ static int cmd_qgroup_limit(int argc, char **argv)
fd = open_file_or_dir(path, &dirstream); fd = open_file_or_dir(path, &dirstream);
if (fd < 0) { if (fd < 0) {
fprintf(stderr, "ERROR: can't access '%s'\n", path); fprintf(stderr, "ERROR: can't access '%s'\n", path);
return 12; return 1;
} }
ret = ioctl(fd, BTRFS_IOC_QGROUP_LIMIT, &args); ret = ioctl(fd, BTRFS_IOC_QGROUP_LIMIT, &args);
@ -421,7 +419,7 @@ static int cmd_qgroup_limit(int argc, char **argv)
if (ret < 0) { if (ret < 0) {
fprintf(stderr, "ERROR: unable to limit requested quota group: " fprintf(stderr, "ERROR: unable to limit requested quota group: "
"%s\n", strerror(e)); "%s\n", strerror(e));
return 30; return 1;
} }
return 0; return 0;
} }