diff options
author | Poornima G <pgurusid@redhat.com> | 2017-05-16 19:25:20 +0530 |
---|---|---|
committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2017-05-22 12:39:59 +0000 |
commit | 284cd8851bfe60984d2f11b5c52fe3204ff43b06 (patch) | |
tree | 452d90a4da2a733460995afd3e3af9e0eb355101 /xlators/performance | |
parent | e43b40296956d132c70ffa3aa07b0078733b39d4 (diff) |
nl-cache: In case of nameless operations do not cache
Issue:
In nameless lookup/other fops, parent inode will be NULL, when we try
to add the cache to the NULL inode, it causes a crash.
Hence handle the scenario of nameless fops, and do not cache/serve
the nameless fops.
Change-Id: I3b90f882ac89e6aaf3419db89e6f890797f37700
BUG: 1451588
Signed-off-by: Poornima G <pgurusid@redhat.com>
Reviewed-on: https://review.gluster.org/17316
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'xlators/performance')
-rw-r--r-- | xlators/performance/nl-cache/src/nl-cache.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/xlators/performance/nl-cache/src/nl-cache.c b/xlators/performance/nl-cache/src/nl-cache.c index 26c09d7d89b..8a130dc2355 100644 --- a/xlators/performance/nl-cache/src/nl-cache.c +++ b/xlators/performance/nl-cache/src/nl-cache.c @@ -198,6 +198,8 @@ nlc_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this, local = frame->local; conf = this->private; + GF_VALIDATE_OR_GOTO (this->name, local, out); + /* Donot add to pe, this may lead to duplicate entry and * requires search before adding if list of strings */ if (op_ret < 0 && op_errno == ENOENT) { @@ -205,8 +207,9 @@ nlc_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this, GF_ATOMIC_INC (conf->nlc_counter.nlc_miss); } +out: NLC_STACK_UNWIND (lookup, frame, op_ret, op_errno, inode, buf, xdata, - postparent); + postparent); return 0; } @@ -218,15 +221,15 @@ nlc_lookup (call_frame_t *frame, xlator_t *this, loc_t *loc, dict_t *xdata) nlc_conf_t *conf = NULL; inode_t *inode = NULL; + if (loc_is_nameless (loc)) + goto wind; + local = nlc_local_init (frame, this, GF_FOP_LOOKUP, loc, NULL); if (!local) goto err; conf = this->private; - if ((!loc->parent && gf_uuid_is_null (loc->pargfid)) || !loc->name) - goto wind; - inode = inode_grep (loc->inode->table, loc->parent, loc->name); if (inode) { inode_unref (inode); |