summaryrefslogtreecommitdiffstats
path: root/xlators/features/snapview-client
diff options
context:
space:
mode:
authorMohammed Rafi KC <rkavunga@redhat.com>2016-01-12 12:13:15 +0530
committerRajesh Joseph <rjoseph@redhat.com>2016-01-12 22:06:55 -0800
commite9d20f0e1a17e0ef2f148a9b49d1cf9596cbe609 (patch)
tree3277a7461fb7ec0cd124d0a130dbde4da7ef42d1 /xlators/features/snapview-client
parent682b19eec8e0a874f7dc8b0ec04b5d731be928e5 (diff)
snapview-client: remove check for parent inode type
In lookup call back path, we are checking for parent inode type to make sure parent inode was set properly, so that next op on the inode can be successfuly completed. More info can be found at bugid 1297311 Change-Id: Ifb1b17f472f855f9b12088c3dd8328389d895e77 BUG: 1297311 Signed-off-by: Mohammed Rafi KC <rkavunga@redhat.com> Reviewed-on: http://review.gluster.org/13227 Reviewed-by: Avra Sengupta <asengupt@redhat.com> Tested-by: NetBSD Build System <jenkins@build.gluster.org> Reviewed-by: Rajesh Joseph <rjoseph@redhat.com>
Diffstat (limited to 'xlators/features/snapview-client')
-rw-r--r--xlators/features/snapview-client/src/snapview-client.c26
1 files changed, 0 insertions, 26 deletions
diff --git a/xlators/features/snapview-client/src/snapview-client.c b/xlators/features/snapview-client/src/snapview-client.c
index 4a2bdfbd301..dec462f1943 100644
--- a/xlators/features/snapview-client/src/snapview-client.c
+++ b/xlators/features/snapview-client/src/snapview-client.c
@@ -264,11 +264,9 @@ svc_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
struct iatt *buf, dict_t *xdata, struct iatt *postparent)
{
svc_local_t *local = NULL;
- inode_t *parent = NULL;
xlator_t *subvolume = NULL;
gf_boolean_t do_unwind = _gf_true;
int inode_type = -1;
- int parent_type = -1;
int ret = -1;
local = frame->local;
@@ -330,27 +328,6 @@ svc_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this,
goto out;
}
- if (local->loc.parent)
- parent = inode_ref (local->loc.parent);
- else {
- parent = inode_parent (inode, NULL, NULL);
- if (!parent && !gf_uuid_is_null (local->loc.pargfid)) {
- parent = inode_find (inode->table,
- local->loc.pargfid);
- }
- }
-
- if (!__is_root_gfid (buf->ia_gfid) && parent) {
- ret = svc_inode_ctx_get (this, parent, &parent_type);
- if (ret < 0) {
- op_ret = -1;
- op_errno = EINVAL;
- gf_log (this->name, GF_LOG_WARNING,
- "Error fetching parent context");
- goto out;
- }
- }
-
if (subvolume == FIRST_CHILD (this))
inode_type = NORMAL_INODE;
else
@@ -367,9 +344,6 @@ out:
xdata, postparent);
}
- if (parent)
- inode_unref (parent);
-
return 0;
}