diff options
| author | Zhou Zhengping <johnzzpcrystal@gmail.com> | 2016-07-10 09:10:49 -0400 |
|---|---|---|
| committer | Jeff Darcy <jdarcy@redhat.com> | 2016-07-14 14:14:28 -0700 |
| commit | ab56ec1e99cd76d45b52d841e6b0009f94a3268b (patch) | |
| tree | 9b6170a831ddfd96fbed11c455ebb7f8d66a1e5f /libglusterfs | |
| parent | 8d3ef3322aa7fccff842e508a1bfd5be71e18849 (diff) | |
rpc: fix several problems in failure handle logic
Once dynstr is set into a dict by function dict_set_dynstr, its free
operation will be called by this dict when the dict is destroyed.
Signed-off-by: Zhou Zhengping <johnzzpcrystal@gmail.com>
Change-Id: Idd2bd19a041bcb477e1c897428ca1740fb75c5f3
BUG: 1354141
Reviewed-on: http://review.gluster.org/14882
Tested-by: Zhou Zhengping <johnzzpcrystal@gmail.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Smoke: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'libglusterfs')
| -rw-r--r-- | libglusterfs/src/dict.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/libglusterfs/src/dict.c b/libglusterfs/src/dict.c index 25ddff0d8c4..6f7adb51589 100644 --- a/libglusterfs/src/dict.c +++ b/libglusterfs/src/dict.c @@ -258,8 +258,6 @@ data_copy (data_t *old) return newdata; err_out: - - FREE (newdata->data); mem_put (newdata); return NULL; |