summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRaghavendra G <raghavendra@gluster.com>2012-03-16 09:11:58 +0530
committerAnand Avati <avati@redhat.com>2012-03-17 23:39:07 -0700
commit3823d7ee14fca54e513d45374e882c6de64fd2a7 (patch)
treeaf0b8da0d50c564f00423f9999a5603378b695b4
parentca15758018b4da68da40c34f240de75b44cb49e9 (diff)
protocol/server: memory leak fixes.
Change-Id: I203832d9d52373f068f90e30dc7672329d65bbea BUG: 803675 Signed-off-by: Raghavendra G <raghavendra@gluster.com> Reviewed-on: http://review.gluster.com/2954 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Amar Tumballi <amarts@redhat.com> Reviewed-by: Anand Avati <avati@redhat.com>
-rw-r--r--xlators/protocol/server/src/server-handshake.c5
-rw-r--r--xlators/protocol/server/src/server3_1-fops.c4
2 files changed, 9 insertions, 0 deletions
diff --git a/xlators/protocol/server/src/server-handshake.c b/xlators/protocol/server/src/server-handshake.c
index ba80185fe..1698a3f5c 100644
--- a/xlators/protocol/server/src/server-handshake.c
+++ b/xlators/protocol/server/src/server-handshake.c
@@ -733,6 +733,11 @@ fail:
rsp.op_errno = op_errno;
server_submit_reply (NULL, req, &rsp, NULL, 0, NULL,
(xdrproc_t)xdr_gf_set_lk_ver_rsp);
+
+ if (args.uid != NULL) {
+ free (args.uid);
+ }
+
return 0;
}
diff --git a/xlators/protocol/server/src/server3_1-fops.c b/xlators/protocol/server/src/server3_1-fops.c
index 3cf7eb2f8..d44ee1cf7 100644
--- a/xlators/protocol/server/src/server3_1-fops.c
+++ b/xlators/protocol/server/src/server3_1-fops.c
@@ -3919,6 +3919,10 @@ out:
if (op_errno)
req->rpc_err = GARBAGE_ARGS;
+ if (args.dict.dict_val != NULL) {
+ free (args.dict.dict_val);
+ }
+
return ret;
}