Unverified Commit 7d4188ce by Christian Brauner Committed by GitHub

Merge pull request #2998 from rikardfalkeborn/fix-returning-non-bool

Fix returning -1 in functions with return type bool
parents fa9aa1fa cdcaad48
...@@ -1273,7 +1273,6 @@ static bool do_dump(struct lxc_container *c, char *mode, struct migrate_opts *op ...@@ -1273,7 +1273,6 @@ static bool do_dump(struct lxc_container *c, char *mode, struct migrate_opts *op
if (!cgroup_ops) { if (!cgroup_ops) {
ERROR("failed to cgroup_init()"); ERROR("failed to cgroup_init()");
_exit(EXIT_FAILURE); _exit(EXIT_FAILURE);
return -1;
} }
os.pipefd = criuout[1]; os.pipefd = criuout[1];
......
...@@ -535,13 +535,13 @@ bool lvm_create_clone(struct lxc_conf *conf, struct lxc_storage *orig, ...@@ -535,13 +535,13 @@ bool lvm_create_clone(struct lxc_conf *conf, struct lxc_storage *orig,
if (!newsize && blk_getsize(orig, &size) < 0) { if (!newsize && blk_getsize(orig, &size) < 0) {
ERROR("Failed to detect size of logical volume \"%s\"", ERROR("Failed to detect size of logical volume \"%s\"",
orig->src); orig->src);
return -1; return false;
} }
/* detect filesystem */ /* detect filesystem */
if (detect_fs(orig, fstype, 100) < 0) { if (detect_fs(orig, fstype, 100) < 0) {
INFO("Failed to detect filesystem type for \"%s\"", orig->src); INFO("Failed to detect filesystem type for \"%s\"", orig->src);
return -1; return false;
} }
} else if (!newsize) { } else if (!newsize) {
size = DEFAULT_FS_SIZE; size = DEFAULT_FS_SIZE;
...@@ -553,7 +553,7 @@ bool lvm_create_clone(struct lxc_conf *conf, struct lxc_storage *orig, ...@@ -553,7 +553,7 @@ bool lvm_create_clone(struct lxc_conf *conf, struct lxc_storage *orig,
ret = do_lvm_create(src, size, thinpool); ret = do_lvm_create(src, size, thinpool);
if (ret < 0) { if (ret < 0) {
ERROR("Failed to create lvm storage volume \"%s\"", src); ERROR("Failed to create lvm storage volume \"%s\"", src);
return -1; return false;
} }
cmd_args[0] = fstype; cmd_args[0] = fstype;
...@@ -563,7 +563,7 @@ bool lvm_create_clone(struct lxc_conf *conf, struct lxc_storage *orig, ...@@ -563,7 +563,7 @@ bool lvm_create_clone(struct lxc_conf *conf, struct lxc_storage *orig,
if (ret < 0) { if (ret < 0) {
ERROR("Failed to create new filesystem \"%s\" for lvm storage " ERROR("Failed to create new filesystem \"%s\" for lvm storage "
"volume \"%s\": %s", fstype, src, cmd_output); "volume \"%s\": %s", fstype, src, cmd_output);
return -1; return false;
} }
data.orig = orig; data.orig = orig;
......
...@@ -427,7 +427,7 @@ bool zfs_snapshot(struct lxc_conf *conf, struct lxc_storage *orig, ...@@ -427,7 +427,7 @@ bool zfs_snapshot(struct lxc_conf *conf, struct lxc_storage *orig,
if (ret < 0 || ret >= PATH_MAX) { if (ret < 0 || ret >= PATH_MAX) {
ERROR("Failed to create string"); ERROR("Failed to create string");
free(snapshot); free(snapshot);
return -1; return false;
} }
cmd_args.dataset = lxc_storage_get_path(new->src, new->type); cmd_args.dataset = lxc_storage_get_path(new->src, new->type);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment