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

Ressources used by va_start must be freed with va_end

Browse Source 
Coverty warn of the defect.

Change-Id: Ie86684520e1d5b41237ab8d3247c24564a1a8639
BUG: 1424802
Signed-off-by: Michael Scherer <misc@redhat.com>
Reviewed-on: https://review.gluster.org/16673
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
Tested-by: Raghavendra G <rgowdapp@redhat.com>
Reviewed-by: Susant Palai <spalai@redhat.com>
Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
This commit is contained in:
Michael Scherer
2017-02-19 16:16:00 +01:00
committed by Shyamsundar Ranganathan
parent b4da61c9fc
commit 4a1740d3e8
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
xlators/cluster/dht/src/dht-common.c
View File

@ -9038,8 +9038,10 @@ dht_notify (xlator_t *this, int event, void *data, ...)
ret = dict_get_int32 (dict, "rebalance-command",
(int32_t*)&cmd);
if (ret)
if (ret) {
va_end (ap);
return ret;
}
LOCK (&defrag->lock);
{
if (defrag->is_exiting)
@ -9064,6 +9066,7 @@ dht_notify (xlator_t *this, int event, void *data, ...)
}
unlock:
UNLOCK (&defrag->lock);
va_end (ap);
return ret;
break;
}