diff options
| author | Pranith Kumar K <pranithk@gluster.com> | 2012-09-12 14:06:53 +0530 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-09-17 19:27:11 -0700 |
| commit | b3c6ee00f9fdc80330979e62fbd7f9bec9440b35 (patch) | |
| tree | 0965c34aaaeeba304a1d84d7b90911d0fca92daa /libglusterfs/src/fd.c | |
| parent | 8f9e94c65516662ff49926203a73b3a0166c087e (diff) | |
features/quota: Avoid mem_put on null local
Problem:
QUOTA_STACK_UNWIND does not check if local is NULL before mem_put.
This leads to a callingfn log for every setxattr in quota.
Fix:
Made sure the mem_put happens only on non-null local in quota.
Additional changes:
Removed unused QUOTA_STACK_DESTROY macro.
Tests:
Peformed setfattr on the mount point and the Invalid argument
logs don't come anymore. Checked that the change does not introduce
any memory leak in quota.
Change-Id: Ie4534c7af0fff80e35d8510311055f930262594f
BUG: 856541
Signed-off-by: Pranith Kumar K <pranithk@gluster.com>
Reviewed-on: http://review.gluster.org/3936
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Amar Tumballi <amarts@redhat.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'libglusterfs/src/fd.c')
0 files changed, 0 insertions, 0 deletions