summaryrefslogtreecommitdiffstats
path: root/api
diff options
context:
space:
mode:
authorMohammed Rafi KC <rkavunga@redhat.com>2016-04-26 16:59:30 +0530
committerRajesh Joseph <rjoseph@redhat.com>2016-05-23 22:48:33 -0700
commit1d52eae8532e027a97dd947b3889b38abe0c113c (patch)
tree6a4181b9f9980e80a3edd53938d62262f17c2c4f /api
parentb45e2896eacb27b8a76729e9c460df5b5aa818a3 (diff)
gfapi: fill iatt in readdirp_cbk if entry->inode is null
If any of dirent have inode as null in readdirp_cbk, which indicates that the stat information is not valid. So for such entries, we send explicit lookup to fill the stat information. Backport of> >Change-Id: I0604bce34583db0bb04b5aae8933766201c6ddad >BUG: 1330567 >Signed-off-by: Mohammed Rafi KC <rkavunga@redhat.com> >Reviewed-on: http://review.gluster.org/14079 >NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> >Smoke: Gluster Build System <jenkins@build.gluster.com> >CentOS-regression: Gluster Build System <jenkins@build.gluster.com> >Reviewed-by: Niels de Vos <ndevos@redhat.com> (cherry picked from commit 9423bdeed169076ebedd9af40b52aaac58c9839e) Change-Id: I90a218c78d5544a3b49b29079c64a8b76e7939df BUG: 1331263 Signed-off-by: Mohammed Rafi KC <rkavunga@redhat.com> Reviewed-on: http://review.gluster.org/14109 Smoke: Gluster Build System <jenkins@build.gluster.com> CentOS-regression: Gluster Build System <jenkins@build.gluster.com> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> Reviewed-by: Rajesh Joseph <rjoseph@redhat.com>
Diffstat (limited to 'api')
-rw-r--r--api/src/glfs-fops.c11
1 files changed, 11 insertions, 0 deletions
diff --git a/api/src/glfs-fops.c b/api/src/glfs-fops.c
index 6f67e6c..ff7f457 100644
--- a/api/src/glfs-fops.c
+++ b/api/src/glfs-fops.c
@@ -2601,6 +2601,17 @@ glfd_entry_refresh (struct glfs_fd *glfd, int plus)
list_for_each_entry (entry, &entries.list, list) {
if (entry->inode)
inode_set_need_lookup (entry->inode, THIS);
+ else if (!IA_ISDIR (entry->d_stat.ia_type)) {
+ /* entry->inode for directories will be
+ * always set to null to force a lookup
+ * on the dentry. Also we will have
+ * proper stat if directory present on
+ * hashed subvolume.
+ */
+ gf_fill_iatt_for_dirent (entry,
+ fd->inode,
+ subvol);
+ }
}
gf_link_inodes_from_dirent (THIS, fd->inode, &entries);