From 71844788a1da0249201b27109e3ea00599569b93 Mon Sep 17 00:00:00 2001 From: Xavier Hernandez Date: Fri, 8 Aug 2014 09:47:10 +0200 Subject: protocol/server: Fix possible memory leaks of xdata There was a memory leak of xdata argument in fops (f)setxattr, (f)xattrop and statfs. This leak was minor because xdata is rarely, or even not used at all, in this fops. Change-Id: Ie1083ae227755a11cf72f8d89595520a5afb3042 BUG: 1127653 Signed-off-by: Xavier Hernandez Reviewed-on: http://review.gluster.org/8434 Reviewed-by: Niels de Vos Tested-by: Gluster Build System --- xlators/protocol/server/src/server-rpc-fops.c | 17 +++++++++++++---- 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/xlators/protocol/server/src/server-rpc-fops.c b/xlators/protocol/server/src/server-rpc-fops.c index 7288fab5374..902a8e5f6b8 100644 --- a/xlators/protocol/server/src/server-rpc-fops.c +++ b/xlators/protocol/server/src/server-rpc-fops.c @@ -4321,7 +4321,9 @@ server3_3_setxattr (rpcsvc_request_t *req) ret = 0; resolve_and_resume (frame, server_setxattr_resume); - return ret; + /* 'dict' will be destroyed later when 'state' is not needed anymore */ + dict = NULL; + out: free (args.xdata.xdata_val); @@ -4395,7 +4397,9 @@ server3_3_fsetxattr (rpcsvc_request_t *req) ret = 0; resolve_and_resume (frame, server_fsetxattr_resume); - return ret; + /* 'dict' will be destroyed later when 'state' is not needed anymore */ + dict = NULL; + out: free (args.xdata.xdata_val); @@ -4469,7 +4473,8 @@ server3_3_fxattrop (rpcsvc_request_t *req) ret = 0; resolve_and_resume (frame, server_fxattrop_resume); - return ret; + /* 'dict' will be destroyed later when 'state' is not needed anymore */ + dict = NULL; out: free (args.xdata.xdata_val); @@ -4544,7 +4549,9 @@ server3_3_xattrop (rpcsvc_request_t *req) ret = 0; resolve_and_resume (frame, server_xattrop_resume); - return ret; + /* 'dict' will be destroyed later when 'state' is not needed anymore */ + dict = NULL; + out: free (args.xdata.xdata_val); @@ -6101,6 +6108,8 @@ server3_3_statfs (rpcsvc_request_t *req) ret = 0; resolve_and_resume (frame, server_statfs_resume); out: + free (args.xdata.xdata_val); + if (op_errno) SERVER_REQ_SET_ERROR (req, ret); -- cgit