summaryrefslogtreecommitdiffstats
path: root/fs/autofs4/symlink.c
diff options
context:
space:
mode:
authorIan Kent <ikent@redhat.com>2014-01-24 00:55:01 +0100
committerLinus Torvalds <torvalds@linux-foundation.org>2014-01-24 01:36:59 +0100
commit8dc51fe5ab9edcaebb5438d6462befdc6922b4a1 (patch)
treee51fe46746a94f0a87e607338b5784f064c49241 /fs/autofs4/symlink.c
parentautofs: use IS_ROOT to replace root dentry checks (diff)
downloadlinux-8dc51fe5ab9edcaebb5438d6462befdc6922b4a1.tar.xz
linux-8dc51fe5ab9edcaebb5438d6462befdc6922b4a1.zip
autofs: fix symlinks aren't checked for expiry
The autofs4 module doesn't consider symlinks for expire as it did in the older autofs v3 module (so it's actually a long standing regression). The user space daemon has focused on the use of bind mounts instead of symlinks for a long time now and that's why this has not been noticed. But with the future addition of amd map parsing to automount(8), not to mention amd itself (of am-utils), symlink expiry will be needed. The direct and offset mount types can't be symlinks and the tree mounts of version 4 were always real mounts so only indirect mounts need expire symlinks. Since the current users of the autofs4 module haven't reported this as a problem to date this patch probably isn't a candidate for backport to stable. Signed-off-by: Ian Kent <ikent@redhat.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to '')
-rw-r--r--fs/autofs4/symlink.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/autofs4/symlink.c b/fs/autofs4/symlink.c
index f27c094a1919..1e8ea192be2b 100644
--- a/fs/autofs4/symlink.c
+++ b/fs/autofs4/symlink.c
@@ -14,6 +14,10 @@
static void *autofs4_follow_link(struct dentry *dentry, struct nameidata *nd)
{
+ struct autofs_sb_info *sbi = autofs4_sbi(dentry->d_sb);
+ struct autofs_info *ino = autofs4_dentry_ino(dentry);
+ if (ino && !autofs4_oz_mode(sbi))
+ ino->last_used = jiffies;
nd_set_link(nd, dentry->d_inode->i_private);
return NULL;
}