From 1e2f13fa89a5aef64cc9e5a229216820ed6e7556 Mon Sep 17 00:00:00 2001 From: Kinglong Mee Date: Thu, 20 Apr 2017 02:12:42 -0400 Subject: api: free all dynamic memory returned from xdr_to_generic Change-Id: I01235ffca80683e69e99da09a9fc1a6d3ffdcb65 Signed-off-by: Kinglong Mee Reviewed-on: https://review.gluster.org/17090 Tested-by: Poornima G NetBSD-regression: NetBSD Build System CentOS-regression: Gluster Build System Reviewed-by: Jeff Darcy Smoke: Gluster Build System --- api/src/glfs-mgmt.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'api') diff --git a/api/src/glfs-mgmt.c b/api/src/glfs-mgmt.c index 1bfebcf2d03..b2728477ef7 100644 --- a/api/src/glfs-mgmt.c +++ b/api/src/glfs-mgmt.c @@ -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, -- cgit