diff options
author | Kaleb S. KEITHLEY <kkeithle@redhat.com> | 2013-08-28 22:43:41 +0530 |
---|---|---|
committer | Anand Avati <avati@redhat.com> | 2013-08-28 16:58:43 -0700 |
commit | b880b6b2908ad4e4afc8e26613bd0db8f0b28750 (patch) | |
tree | 073f25549d4c3a023a7cc6776ac13559a5bb900e /xlators/cluster/stripe/src/stripe.c | |
parent | bbcdbd8c36c7756f39bb7464cd1c69b67e66cfaa (diff) |
stripe: remove unused param, handle mem alloc failure
Change-Id: I9c27b1edab111031ca8eea9cc49480ea01e39089
BUG: 1002207
Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com>
Reviewed-on: http://review.gluster.org/5716
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'xlators/cluster/stripe/src/stripe.c')
-rw-r--r-- | xlators/cluster/stripe/src/stripe.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xlators/cluster/stripe/src/stripe.c b/xlators/cluster/stripe/src/stripe.c index 056dea2eb59..a8b8f653088 100644 --- a/xlators/cluster/stripe/src/stripe.c +++ b/xlators/cluster/stripe/src/stripe.c @@ -5199,10 +5199,10 @@ stripe_vgetxattr_cbk (call_frame_t *frame, void *cookie, xattr->pos = cky; xattr->xattr_value = gf_memdup (xattr_val, - xattr->xattr_value, xattr->xattr_len); - local->xattr_total_len += xattr->xattr_len + 1; + if (xattr->xattr_value != NULL) + local->xattr_total_len += xattr->xattr_len + 1; } } out: |