From 720b63c24b07ee64e1338db28de602b9abbef0a1 Mon Sep 17 00:00:00 2001 From: Jiffin Tony Thottan Date: Thu, 17 Mar 2016 18:53:13 +0530 Subject: glusterd : skip non directories inside /var/lib/glusterd/vols Right now glusterd won't come up if vols directory contains an invalid entry. Instead of doing that with this change a message will be logged and then skip that entry Change-Id: I665b5c35291b059cf054622da0eec4db44ec5f68 BUG: 1318591 Signed-off-by: Jiffin Tony Thottan Reviewed-on: http://review.gluster.org/13764 Reviewed-by: Prashanth Pai Reviewed-by: Atin Mukherjee Smoke: Gluster Build System CentOS-regression: Gluster Build System NetBSD-regression: NetBSD Build System --- xlators/mgmt/glusterd/src/glusterd-store.c | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) (limited to 'xlators') diff --git a/xlators/mgmt/glusterd/src/glusterd-store.c b/xlators/mgmt/glusterd/src/glusterd-store.c index deaa0892afe..ba02bfac12d 100644 --- a/xlators/mgmt/glusterd/src/glusterd-store.c +++ b/xlators/mgmt/glusterd/src/glusterd-store.c @@ -3115,6 +3115,8 @@ glusterd_store_retrieve_volumes (xlator_t *this, glusterd_snap_t *snap) struct dirent *entry = NULL; struct dirent scratch[2] = {{0,},}; glusterd_volinfo_t *volinfo = NULL; + struct stat st = {0,}; + char entry_path[PATH_MAX] = {0,}; GF_ASSERT (this); priv = this->private; @@ -3143,6 +3145,22 @@ glusterd_store_retrieve_volumes (xlator_t *this, glusterd_snap_t *snap) (!strcmp (entry->d_name, "info")))) goto next; + snprintf (entry_path, PATH_MAX, "%s/%s", path, entry->d_name); + ret = sys_lstat (entry_path, &st); + if (ret == -1) { + gf_msg (this->name, GF_LOG_ERROR, 0, + GD_MSG_INVALID_ENTRY, + "Failed to stat entry %s : %s", path, + strerror (errno)); + goto next; + } + + if (!S_ISDIR (st.st_mode)) { + gf_msg_debug (this->name, 0, "%s is not a vaild volume" + , entry->d_name); + goto next; + } + volinfo = glusterd_store_retrieve_volume (entry->d_name, snap); if (!volinfo) { gf_msg (this->name, GF_LOG_ERROR, 0, -- cgit