From ed05aac829498161289d4f2da4002b42fb54bc32 Mon Sep 17 00:00:00 2001 From: Michael McCracken Date: Fri, 19 Jan 2018 08:38:36 -0800 Subject: [PATCH] storage: treat return value from ops->destroy as int r->ops->destroy() returns an int, -1 on error. When assigned to a bool, this becomes true and hides errors. Signed-off-by: Michael McCracken --- src/lxc/storage/storage.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/lxc/storage/storage.c b/src/lxc/storage/storage.c index 98aa031b7..e080ad87a 100644 --- a/src/lxc/storage/storage.c +++ b/src/lxc/storage/storage.c @@ -603,13 +603,14 @@ bool storage_destroy(struct lxc_conf *conf) { struct lxc_storage *r; bool ret = false; + int destroy_rv = 0; r = storage_init(conf); if (!r) return ret; - ret = r->ops->destroy(r); - if (ret == 0) + destroy_rv = r->ops->destroy(r); + if (destroy_rv == 0) ret = true; storage_put(r); -- 2.47.2