diff options
| author | Jeff Darcy <jdarcy@redhat.com> | 2012-06-05 21:59:47 -0400 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-06-05 22:26:26 -0700 |
| commit | be1d2cd4d042bda2e53e3c45dd36e8ed9525ba32 (patch) | |
| tree | b4c15bc9e7d9bd9def09fce2ec46943cff62683e | |
| parent | a7370dc95ec2a875c937c682e6f0c8aa31c91b4a (diff) | |
rpc: avoid an invalid free of item on a list
If we actually "consumed" vol_opt by putting it on THIS->volume_options,
it's still in use and we shouldn't free it before returning.
Change-Id: I8ef3e4ce8a8b9f2552faa3345f1686e173d1aa10
BUG: 829104
Signed-off-by: Jeff Darcy <jdarcy@redhat.com>
Reviewed-on: http://review.gluster.com/3528
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-- | rpc/rpc-lib/src/rpc-transport.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/rpc/rpc-lib/src/rpc-transport.c b/rpc/rpc-lib/src/rpc-transport.c index d0e7834e8d7..4b693742449 100644 --- a/rpc/rpc-lib/src/rpc-transport.c +++ b/rpc/rpc-lib/src/rpc-transport.c @@ -305,6 +305,7 @@ rpc_transport_load (glusterfs_ctx_t *ctx, dict_t *options, char *trans_name) "volume option validation failed"); goto fail; } + vol_opt = NULL; } trans->options = options; |