diff options
| author | Amar Tumballi <amar@gluster.com> | 2010-10-04 11:01:14 +0000 |
|---|---|---|
| committer | Vijay Bellur <vijay@dev.gluster.com> | 2010-10-04 09:48:47 -0700 |
| commit | bde5cb81e251032594e2bba57fb4a72c10e32207 (patch) | |
| tree | ead19da09126011c7581f5ac99d5e6ed94a92efa /xlators/mgmt | |
| parent | 98d51842863d52bab2674258c3ec12d668139539 (diff) | |
rpc: don't refer to 'req' after calling 'prog->actor()'
there are chances that a error reply is sent to client from actor,
in which case, 'req' would be free'd and accessing it would result
in error.
Signed-off-by: Amar Tumballi <amar@gluster.com>
Signed-off-by: Vijay Bellur <vijay@dev.gluster.com>
BUG: 1790 ()
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1790
Diffstat (limited to 'xlators/mgmt')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-rebalance.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-rebalance.c b/xlators/mgmt/glusterd/src/glusterd-rebalance.c index 6583dcb0d84..f24ff85c9b9 100644 --- a/xlators/mgmt/glusterd/src/glusterd-rebalance.c +++ b/xlators/mgmt/glusterd/src/glusterd-rebalance.c @@ -476,5 +476,6 @@ out: gf_xdr_serialize_cli_defrag_vol_rsp); if (cli_req.volname) free (cli_req.volname);//malloced by xdr - return ret; + + return 0; } |