diff options
| author | Raghavendra G <raghavendra@gluster.com> | 2012-03-16 09:11:58 +0530 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-03-17 23:39:07 -0700 |
| commit | 3823d7ee14fca54e513d45374e882c6de64fd2a7 (patch) | |
| tree | af0b8da0d50c564f00423f9999a5603378b695b4 | |
| parent | ca15758018b4da68da40c34f240de75b44cb49e9 (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.c | 5 | ||||
| -rw-r--r-- | xlators/protocol/server/src/server3_1-fops.c | 4 |
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; } |