diff options
| author | Vijay Bellur <vbellur@redhat.com> | 2015-06-03 23:43:55 +0530 |
|---|---|---|
| committer | Raghavendra G <rgowdapp@redhat.com> | 2015-06-04 22:37:49 -0700 |
| commit | d2d7afefd861943f26b290d214fb5213d24b6a02 (patch) | |
| tree | 91883409eff61bdf4ad254ec3038d0ce268fde76 /xlators/features/marker | |
| parent | f1c20897cc18fb3b257880e7a098c05eae28fd0f (diff) | |
features/marker: Cleanup loc in case of errors
Missing loc_wipe() for error paths in mq_readdir_cbk() can
cause memory leaks. loc_wipe() is now done for both happy
and unhappy paths.
Change-Id: I882aa5dcca06e25b56a828767fb2b91a1efaf83b
BUG: 1227904
Signed-off-by: Vijay Bellur <vbellur@redhat.com>
Reviewed-on: http://review.gluster.org/11074
Reviewed-by: Sachin Pandit <spandit@redhat.com>
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijaikumar Mallikarjuna <vmallika@redhat.com>
Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
Tested-by: Raghavendra G <rgowdapp@redhat.com>
Diffstat (limited to 'xlators/features/marker')
| -rw-r--r-- | xlators/features/marker/src/marker-quota.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/xlators/features/marker/src/marker-quota.c b/xlators/features/marker/src/marker-quota.c index 97039440eb6..6d0830cde17 100644 --- a/xlators/features/marker/src/marker-quota.c +++ b/xlators/features/marker/src/marker-quota.c @@ -660,12 +660,10 @@ mq_readdir_cbk (call_frame_t *frame, &loc, dict); offset = entry->d_off; - - loc_wipe (&loc); - newframe = NULL; out: + loc_wipe (&loc); if (dict) { dict_unref (dict); dict = NULL; |