From b37c742f579191a96737c519f6de1ff54a3076d7 Mon Sep 17 00:00:00 2001 From: Raghavendra G Date: Wed, 2 Dec 2009 01:52:15 +0000 Subject: performance/stat-prefetch: fix errors in handling memory allocation failures in sp_inode_ctx_process. - the variables that are used for making decisions are assigned before memory allocations and these variables were not reset to correct values in case of memory allocation failures. Signed-off-by: Raghavendra G Signed-off-by: Anand V. Avati BUG: 257 (Backport stat-prefetch to 2.0) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=257 --- xlators/performance/stat-prefetch/src/stat-prefetch.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'xlators') diff --git a/xlators/performance/stat-prefetch/src/stat-prefetch.c b/xlators/performance/stat-prefetch/src/stat-prefetch.c index 11821eb8a..b4e6b379c 100644 --- a/xlators/performance/stat-prefetch/src/stat-prefetch.c +++ b/xlators/performance/stat-prefetch/src/stat-prefetch.c @@ -108,9 +108,6 @@ sp_process_inode_ctx (call_frame_t *frame, xlator_t *this, loc_t *loc, } if (!(inode_ctx->looked_up || inode_ctx->lookup_in_progress)) { - *need_lookup = 1; - inode_ctx->lookup_in_progress = 1; - if (frame->local == NULL) { local = CALLOC (1, sizeof (*local)); GF_VALIDATE_OR_GOTO_WITH_ERROR (this->name, @@ -129,6 +126,9 @@ sp_process_inode_ctx (call_frame_t *frame, xlator_t *this, loc_t *loc, goto unlock; } } + + *need_lookup = 1; + inode_ctx->lookup_in_progress = 1; } if (inode_ctx->looked_up) { -- cgit