From 2ec4b22bb02f572f74157df5264b247095fece77 Mon Sep 17 00:00:00 2001 From: Raghavendra Talur Date: Thu, 29 Aug 2013 19:04:40 +0530 Subject: api: Fill loc.path and loc.gfid before syncop_open. syncop_open was crashing because of NULL dereference. This fixes that. Change-Id: I4bc48fac2a6c0b15c806cbbb4ae45c67891ab7ed BUG: 1002577 Signed-off-by: Raghavendra Talur Reviewed-on: http://review.gluster.org/5734 Tested-by: Gluster Build System Reviewed-by: Anand Avati --- api/src/glfs-resolve.c | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'api') diff --git a/api/src/glfs-resolve.c b/api/src/glfs-resolve.c index beb270322..ec8890b2d 100644 --- a/api/src/glfs-resolve.c +++ b/api/src/glfs-resolve.c @@ -583,6 +583,15 @@ glfs_migrate_fd_safe (struct glfs *fs, xlator_t *newsubvol, fd_t *oldfd) loc.inode = inode_ref (newinode); + ret = inode_path (oldfd->inode, NULL, (char **)&loc.path); + if (ret < 0) { + gf_log (fs->volname, GF_LOG_INFO, "inode_path failed"); + goto out; + } + + uuid_copy (loc.gfid, oldinode->gfid); + + if (IA_ISDIR (oldinode->ia_type)) ret = syncop_opendir (newsubvol, &loc, newfd); else -- cgit