From c078b9df8d8f44ca90e61566f83f83809ce5ac0f Mon Sep 17 00:00:00 2001 From: Pranith Kumar K Date: Thu, 29 Mar 2012 19:45:29 +0530 Subject: cluster/afr: Handle invalid inode in open_fd_fix Change-Id: Ie009fb4b8b7745ebd5b76f7a40287998be35eef3 BUG: 804914 Signed-off-by: Pranith Kumar K Reviewed-on: http://review.gluster.com/3045 Tested-by: Gluster Build System Reviewed-by: Jeff Darcy Reviewed-by: Vijay Bellur --- xlators/cluster/afr/src/afr-inode-write.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/xlators/cluster/afr/src/afr-inode-write.c b/xlators/cluster/afr/src/afr-inode-write.c index 368a68bd6..7fa6d096c 100644 --- a/xlators/cluster/afr/src/afr-inode-write.c +++ b/xlators/cluster/afr/src/afr-inode-write.c @@ -348,6 +348,7 @@ afr_open_fd_fix (call_frame_t *frame, xlator_t *this, gf_boolean_t pause_fop) int ret = 0; int i = 0; afr_fd_ctx_t *fd_ctx = NULL; + inode_t *inode = NULL; gf_boolean_t need_self_heal = _gf_false; int *need_open = NULL; int need_open_count = 0; @@ -360,7 +361,10 @@ afr_open_fd_fix (call_frame_t *frame, xlator_t *this, gf_boolean_t pause_fop) GF_ASSERT (local->fd); - if (fd_is_anonymous (local->fd)) { + inode = local->fd->inode; + //gfid is not set in rebalance, that case needs to be handled. + if (fd_is_anonymous (local->fd) || + !inode || uuid_is_null (inode->gfid)) { fop_continue = _gf_true; goto out; } -- cgit lue='release-3.9'>release-3.9
summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* build: make GLUSTERD_WORKDIR rely on localstatedirHarshavardhana2014-09-032-46/+46
| | | | | | | | | | | | | | | | | | | | | | | Backport from master branch - http://review.gluster.org/#/c/8246/ - Break-way from '/var/lib/glusterd' hard-coded previously, instead rely on 'configure' value from 'localstatedir' - Provide 's/lib/db' as default working directory for gluster management daemon for BSD and Darwin based installations - loff_t is really off_t on Darwin - fix-off the warnings generated by clang on FreeBSD/Darwin - Now 'tests/*' use GLUSTERD_WORKDIR a common variable for all platforms. - Define proper environment for running tests, define correct PATH and LD_LIBRARY_PATH when running tests, so that the desired version of glusterfs is used, regardless where it is installed. (Thanks to manu@netbsd.org for this additional work) Change-Id: I06e684ac4c26d1e74c9daf76753403ad15f79276 BUG: 1130308 Signed-off-by: Harshavardhana <harsha@harshavardhana.net> Reviewed-on: http://review.gluster.org/8486 Tested-by: Gluster Build System <jenkins@build.gluster.com>
* Made spelling changes to 19 filesAkshataDM2014-03-161-1/+1
| | | | | | | | | | Change-Id: If91cf44578fe0b5176ea01ae5c5962e31606f640 BUG: 1075417 Signed-off-by: AkshataDM <oxta28@gmail.com> Reviewed-on: http://review.gluster.org/7280 Reviewed-by: Varun Shastry <vshastry@redhat.com> Reviewed-by: Anand Avati <avati@redhat.com> Tested-by: Anand Avati <avati@redhat.com>
*