andy/glusterfs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

glusterd: Fixing wrong array comparison (Coverity fix)

Browse Source 
Coverity CID: 1256175

Change-Id: Ib29fc2eaa54a7ce8369918e68bf117d0f04ca94d
BUG: 789278
Signed-off-by: Nandaja Varma <nandaja.varma@gmail.com>
Reviewed-on: http://review.gluster.org/9679
Reviewed-by: Vijaikumar Mallikarjuna <vmallika@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Humble Devassy Chirammal <humble.devassy@gmail.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
This commit is contained in:
Nandaja Varma 2015-02-18 16:40:03 +05:30 committed by Vijay Bellur
parent cd3d34289c
commit d71418ac4e
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
xlators/mgmt/glusterd/src/glusterd-snapshot.c
View File

@ -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",