On Sun, Feb 12, 2006 at 06:06:01PM +0000, Al Viro wrote:
> On Fri, Feb 10, 2006 at 07:31:07PM -0800, Linda Walsh wrote:
> > The maximum number of followed symlinks seems to be set to 5.
> >
> > This seems small when compared to other filesystem limits.
> > Is there some objection to it being raised? Should it be
> > something like Glib's '20' or '255'?
Just a note (which Al probably considered too obvious to point out), but
MAX_NESTED_LINKS isn't the maximum number of followed symlinks. It's
the number of recursions we're limited to. The maximum number of
symlinks followed is 40 (see fs/namei.c:do_follow_link).
Al, would it be worth making 40 an enumerated constant in the same
enumeration as MAX_NESTED_LINKS? Something like this:
Index: include/linux/namei.h
===================================================================
RCS file: /var/cvs/linux-2.6/include/linux/namei.h,v
retrieving revision 1.4
diff -u -p -r1.4 namei.h
--- include/linux/namei.h 12 Nov 2005 04:09:17 -0000 1.4
+++ include/linux/namei.h 12 Feb 2006 19:35:31 -0000
@@ -11,7 +11,10 @@ struct open_intent {
struct file *file;
};
-enum { MAX_NESTED_LINKS = 5 };
+enum {
+ MAX_NESTED_LINKS = 5,
+ MAX_FOLLOW_SYMLINKS = 40,
+};
struct nameidata {
struct dentry *dentry;
Index: fs/namei.c
===================================================================
RCS file: /var/cvs/linux-2.6/fs/namei.c,v
retrieving revision 1.31
diff -u -p -r1.31 namei.c
--- fs/namei.c 12 Nov 2005 04:08:32 -0000 1.31
+++ fs/namei.c 12 Feb 2006 19:35:31 -0000
@@ -598,7 +598,7 @@ static inline int do_follow_link(struct
int err = -ELOOP;
if (current->link_count >= MAX_NESTED_LINKS)
goto loop;
- if (current->total_link_count >= 40)
+ if (current->total_link_count >= MAX_FOLLOW_SYMLINKS)
goto loop;
BUG_ON(nd->depth >= MAX_NESTED_LINKS);
cond_resched();
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
