diff options
author | Pranith Kumar K <pkarampu@redhat.com> | 2015-03-06 12:37:15 +0530 |
---|---|---|
committer | Raghavendra G <rgowdapp@redhat.com> | 2015-03-09 03:15:06 -0700 |
commit | b5e50cd714d105ff7aa1544182f07fb60320710d (patch) | |
tree | 34a7411e442cf2d5ea6935cb9b03f7e111a677f2 /rpc | |
parent | 5ab4a17a63acf7215e9dc98276e7547ae527a5a7 (diff) |
libxlator: Make sure marker_xattr is valid
Problem:
marker_xattr is allocated only when the op_ret is 0. If the final response is
a failure, then the marker time is dict_set with key as NULL. this will be
changed to ref:<address-of-value> by dict_set, so the value won't appear on the
marker-key when the getxattr_cbk is in dht. So dht unwinds with failure EINVAL.
Fix:
Always populate marker_xattr. Fixed dict mem-leak as well.
Change-Id: I1752f277a8852c47b0a2ccce9fd72ee88456ac02
BUG: 1199406
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/9817
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
Tested-by: Raghavendra G <rgowdapp@redhat.com>
Diffstat (limited to 'rpc')
0 files changed, 0 insertions, 0 deletions