From d71418ac4e2da989fc49eece4e2acbb9288b71f9 Mon Sep 17 00:00:00 2001 From: Nandaja Varma Date: Wed, 18 Feb 2015 16:40:03 +0530 Subject: glusterd: Fixing wrong array comparison (Coverity fix) Coverity CID: 1256175 Change-Id: Ib29fc2eaa54a7ce8369918e68bf117d0f04ca94d BUG: 789278 Signed-off-by: Nandaja Varma Reviewed-on: http://review.gluster.org/9679 Reviewed-by: Vijaikumar Mallikarjuna Tested-by: Gluster Build System Reviewed-by: Humble Devassy Chirammal Reviewed-by: Kaleb KEITHLEY Reviewed-by: Vijay Bellur --- xlators/mgmt/glusterd/src/glusterd-snapshot.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xlators/mgmt/glusterd/src/glusterd-snapshot.c b/xlators/mgmt/glusterd/src/glusterd-snapshot.c index 761115d22ed..7359040f18c 100644 --- a/xlators/mgmt/glusterd/src/glusterd-snapshot.c +++ b/xlators/mgmt/glusterd/src/glusterd-snapshot.c @@ -2597,7 +2597,7 @@ glusterd_lvm_snapshot_remove (dict_t *rsp_dict, glusterd_volinfo_t *snap_vol) } /* Check if the brick has a LV associated with it */ - if (!brickinfo->device_path) { + if (strlen(brickinfo->device_path) == 0) { gf_log (this->name, GF_LOG_DEBUG, "Brick (%s:%s) does not have a LV " "associated with it. Removing the brick path", -- cgit