summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRaghavendra Bhat <raghavendrabhat@gluster.com>2011-03-01 00:49:19 +0000
committerAnand V. Avati <avati@dev.gluster.com>2011-02-28 23:40:46 -0800
commit156e1e26d76d4b62ad1fae15938a1350c9bc892a (patch)
tree574fb7b86b6814cf6ae855b415d159377472674a
parentba1cea55eedec2f7f9fbbc10e581da2c9bcd8038 (diff)
check for inode and path being NULL before copying them
Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com> Signed-off-by: Anand V. Avati <avati@dev.gluster.com> BUG: 2463 ([7b07d444a77526f27f860210930bf1d4c7fbea9b]: ./autogen.sh crashes glusterfs in fuse_getattr or fuse_setattr) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2463
-rw-r--r--xlators/mount/fuse/src/fuse-bridge.c52
1 files changed, 28 insertions, 24 deletions
diff --git a/xlators/mount/fuse/src/fuse-bridge.c b/xlators/mount/fuse/src/fuse-bridge.c
index f988c0102ce..f4e142f0543 100644
--- a/xlators/mount/fuse/src/fuse-bridge.c
+++ b/xlators/mount/fuse/src/fuse-bridge.c
@@ -272,7 +272,7 @@ fuse_lookup (xlator_t *this, fuse_in_header_t *finh, void *msg)
"%"PRIu64": LOOKUP %s(%"PRId64")", finh->unique,
state->loc.path, state->loc.inode->ino);
state->is_revalidate = 1;
- memcpy (state->resolve.gfid, state->loc.inode->gfid, 16);
+ uuid_copy (state->resolve.gfid, state->loc.inode->gfid);
} else {
gf_log ("glusterfs-fuse", GF_LOG_TRACE,
"%"PRIu64": LOOKUP %s", finh->unique,
@@ -280,7 +280,7 @@ fuse_lookup (xlator_t *this, fuse_in_header_t *finh, void *msg)
uuid_generate (state->gfid);
}
- memcpy (state->resolve.pargfid, state->loc.parent->gfid, 16);
+ uuid_copy (state->resolve.pargfid, state->loc.parent->gfid);
state->resolve.bname = gf_strdup (name);
state->resolve.path = gf_strdup (state->loc.path);
@@ -517,8 +517,10 @@ fuse_getattr (xlator_t *this, fuse_in_header_t *finh, void *msg)
state->fd = NULL;
}
- memcpy (state->resolve.gfid, state->loc.inode->gfid, 16);
- state->resolve.path = gf_strdup (state->loc.path);
+ uuid_copy (state->resolve.gfid, state->loc.inode->gfid);
+ if (state->loc.path)
+ state->resolve.path = gf_strdup (state->loc.path);
+
fuse_resolve_and_resume (state, fuse_getattr_resume);
}
@@ -810,8 +812,10 @@ fuse_setattr (xlator_t *this, fuse_in_header_t *finh, void *msg)
state->size = fsi->size;
}
- memcpy (state->resolve.gfid, state->loc.inode->gfid, 16);
- state->resolve.path = gf_strdup (state->loc.path);
+ if (!state->fd) {
+ uuid_copy (state->resolve.gfid, state->loc.inode->gfid);
+ state->resolve.path = gf_strdup (state->loc.path);
+ }
fuse_resolve_and_resume (state, fuse_setattr_resume);
}
@@ -945,7 +949,7 @@ fuse_access (xlator_t *this, fuse_in_header_t *finh, void *msg)
state->mask = fai->mask;
- memcpy (state->resolve.gfid, state->loc.inode->gfid, 16);
+ uuid_copy (state->resolve.gfid, state->loc.inode->gfid);
state->resolve.path = gf_strdup (state->loc.path);
fuse_resolve_and_resume (state, fuse_access_resume);
@@ -1016,7 +1020,7 @@ fuse_readlink (xlator_t *this, fuse_in_header_t *finh, void *msg)
return;
}
- memcpy (state->resolve.gfid, state->loc.inode->gfid, 16);
+ uuid_copy (state->resolve.gfid, state->loc.inode->gfid);
state->resolve.path = gf_strdup (state->loc.path);
fuse_resolve_and_resume (state, fuse_readlink_resume);
@@ -1084,7 +1088,7 @@ fuse_mknod (xlator_t *this, fuse_in_header_t *finh, void *msg)
state->mode = fmi->mode;
state->rdev = fmi->rdev;
- memcpy (state->resolve.pargfid, state->loc.parent->gfid, 16);
+ uuid_copy (state->resolve.pargfid, state->loc.parent->gfid);
state->resolve.bname = gf_strdup (name);
state->resolve.path = gf_strdup (state->loc.path);
@@ -1144,7 +1148,7 @@ fuse_mkdir (xlator_t *this, fuse_in_header_t *finh, void *msg)
state->mode = fmi->mode;
- memcpy (state->resolve.pargfid, state->loc.parent->gfid, 16);
+ uuid_copy (state->resolve.pargfid, state->loc.parent->gfid);
state->resolve.bname = gf_strdup (name);
state->resolve.path = gf_strdup (state->loc.path);
@@ -1190,7 +1194,7 @@ fuse_unlink (xlator_t *this, fuse_in_header_t *finh, void *msg)
return;
}
- memcpy (state->resolve.pargfid, state->loc.parent->gfid, 16);
+ uuid_copy (state->resolve.pargfid, state->loc.parent->gfid);
state->resolve.bname = gf_strdup (name);
state->resolve.path = gf_strdup (state->loc.path);
@@ -1236,7 +1240,7 @@ fuse_rmdir (xlator_t *this, fuse_in_header_t *finh, void *msg)
return;
}
- memcpy (state->resolve.pargfid, state->loc.parent->gfid, 16);
+ uuid_copy (state->resolve.pargfid, state->loc.parent->gfid);
state->resolve.bname = gf_strdup (name);
state->resolve.path = gf_strdup (state->loc.path);
@@ -1295,7 +1299,7 @@ fuse_symlink (xlator_t *this, fuse_in_header_t *finh, void *msg)
state->name = gf_strdup (linkname);
- memcpy (state->resolve.pargfid, state->loc.parent->gfid, 16);
+ uuid_copy (state->resolve.pargfid, state->loc.parent->gfid);
state->resolve.bname = gf_strdup (name);
state->resolve.path = gf_strdup (state->loc.path);
@@ -1404,11 +1408,11 @@ fuse_rename (xlator_t *this, fuse_in_header_t *finh, void *msg)
return;
}
- memcpy (state->resolve.pargfid, state->loc.parent->gfid, 16);
+ uuid_copy (state->resolve.pargfid, state->loc.parent->gfid);
state->resolve.bname = gf_strdup (oldname);
state->resolve.path = gf_strdup (state->loc.path);
- memcpy (state->resolve2.pargfid, state->loc2.parent->gfid, 16);
+ uuid_copy (state->resolve2.pargfid, state->loc2.parent->gfid);
state->resolve2.bname = gf_strdup (newname);
state->resolve2.path = gf_strdup (state->loc2.path);
@@ -1461,11 +1465,11 @@ fuse_link (xlator_t *this, fuse_in_header_t *finh, void *msg)
return;
}
- memcpy (state->resolve.pargfid, state->loc.parent->gfid, 16);
+ uuid_copy (state->resolve.pargfid, state->loc.parent->gfid);
state->resolve.bname = gf_strdup (name);
state->resolve.path = gf_strdup (state->loc.path);
- memcpy (state->resolve2.gfid, state->loc2.inode->gfid, 16);
+ uuid_copy (state->resolve2.gfid, state->loc2.inode->gfid);
state->resolve2.path = gf_strdup (state->loc2.path);
fuse_resolve_and_resume (state, fuse_link_resume);
@@ -1641,7 +1645,7 @@ fuse_create (xlator_t *this, fuse_in_header_t *finh, void *msg)
state->mode = fci->mode;
state->flags = fci->flags;
- memcpy (state->resolve.pargfid, state->loc.parent->gfid, 16);
+ uuid_copy (state->resolve.pargfid, state->loc.parent->gfid);
state->resolve.bname = gf_strdup (name);
state->resolve.path = gf_strdup (state->loc.path);
@@ -1699,7 +1703,7 @@ fuse_open (xlator_t *this, fuse_in_header_t *finh, void *msg)
state->flags = foi->flags;
- memcpy (state->resolve.gfid, state->loc.inode->gfid, 16);
+ uuid_copy (state->resolve.gfid, state->loc.inode->gfid);
state->resolve.path = gf_strdup (state->loc.path);
fuse_resolve_and_resume (state, fuse_open_resume);
@@ -2024,7 +2028,7 @@ fuse_opendir (xlator_t *this, fuse_in_header_t *finh, void *msg)
return;
}
- memcpy (state->resolve.gfid, state->loc.inode->gfid, 16);
+ uuid_copy (state->resolve.gfid, state->loc.inode->gfid);
state->resolve.path = gf_strdup (state->loc.path);
fuse_resolve_and_resume (state, fuse_opendir_resume);
@@ -2391,7 +2395,7 @@ fuse_setxattr (xlator_t *this, fuse_in_header_t *finh, void *msg)
state->flags = fsi->flags;
state->name = gf_strdup (name);
- memcpy (state->resolve.gfid, state->loc.inode->gfid, 16);
+ uuid_copy (state->resolve.gfid, state->loc.inode->gfid);
state->resolve.path = gf_strdup (state->loc.path);
fuse_resolve_and_resume (state, fuse_setxattr_resume);
@@ -2580,7 +2584,7 @@ fuse_getxattr (xlator_t *this, fuse_in_header_t *finh, void *msg)
state->size = fgxi->size;
state->name = gf_strdup (name);
- memcpy (state->resolve.gfid, state->loc.inode->gfid, 16);
+ uuid_copy (state->resolve.gfid, state->loc.inode->gfid);
state->resolve.path = gf_strdup (state->loc.path);
fuse_resolve_and_resume (state, fuse_getxattr_resume);
@@ -2621,7 +2625,7 @@ fuse_listxattr (xlator_t *this, fuse_in_header_t *finh, void *msg)
}
state->size = fgxi->size;
- memcpy (state->resolve.gfid, state->loc.inode->gfid, 16);
+ uuid_copy (state->resolve.gfid, state->loc.inode->gfid);
state->resolve.path = gf_strdup (state->loc.path);
fuse_resolve_and_resume (state, fuse_listxattr_resume);
@@ -2663,7 +2667,7 @@ fuse_removexattr (xlator_t *this, fuse_in_header_t *finh, void *msg)
}
state->name = gf_strdup (name);
- memcpy (state->resolve.gfid, state->loc.inode->gfid, 16);
+ uuid_copy (state->resolve.gfid, state->loc.inode->gfid);
state->resolve.path = gf_strdup (state->loc.path);
fuse_resolve_and_resume (state, fuse_removexattr_resume);