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

api: free all dynamic memory returned from xdr_to_generic

Browse Source 
Change-Id: I01235ffca80683e69e99da09a9fc1a6d3ffdcb65
Signed-off-by: Kinglong Mee <mijinlong@open-fs.com>
Reviewed-on: https://review.gluster.org/17090
Tested-by: Poornima G <pgurusid@redhat.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Jeff Darcy <jeff@pl.atyp.us>
Smoke: Gluster Build System <jenkins@build.gluster.org>
This commit is contained in:
Kinglong Mee 2017-04-20 02:12:42 -04:00 committed by Jeff Darcy
parent b5ebf8adf6
commit 1e2f13fa89
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
api/src/glfs-mgmt.c
View File

@ -382,7 +382,7 @@ out:
if (rsp.dict.dict_val)
free (rsp.dict.dict_val);
if (rsp.op_errstr && *rsp.op_errstr)
if (rsp.op_errstr)
free (rsp.op_errstr);
gf_msg_debug (frame->this->name, 0, "Returning: %d", ret);
@ -677,6 +677,9 @@ out:
if (rsp.spec)
free (rsp.spec);
if (rsp.xdata.xdata_val)
free (rsp.xdata.xdata_val);
// Stop if server is running at an unsupported op-version
if (ENOTSUP == ret) {
gf_msg ("mgmt", GF_LOG_ERROR, ENOTSUP, API_MSG_WRONG_OPVERSION,