* MAX_NESTED_LINKS
@ 2004-08-31 2:40 Steve French
2004-08-31 7:38 ` MAX_NESTED_LINKS Arjan van de Ven
0 siblings, 1 reply; 2+ messages in thread
From: Steve French @ 2004-08-31 2:40 UTC (permalink / raw)
To: linux-kernel
MAX_NESTED_LINKS seems to have been set to 5, but the comment in
fs/namei.c indicates a larger number. Is that intentional?
* This limits recursive symlink follows to 8, while
* limiting consecutive symlinks to 40.
@@ -405,19 +407,30 @@
static inline int do_follow_link(struct dentry *dentry, struct nameidata
*nd)
{
int err = -ELOOP;
if (current->link_count >= MAX_NESTED_LINKS)
goto loop;
if (current->total_link_count >= 40)
goto loop;
BUG_ON(nd->depth >= MAX_NESTED_LINKS);
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: MAX_NESTED_LINKS
2004-08-31 2:40 MAX_NESTED_LINKS Steve French
@ 2004-08-31 7:38 ` Arjan van de Ven
0 siblings, 0 replies; 2+ messages in thread
From: Arjan van de Ven @ 2004-08-31 7:38 UTC (permalink / raw)
To: Steve French; +Cc: linux-kernel
[-- Attachment #1: Type: text/plain, Size: 343 bytes --]
On Tue, 2004-08-31 at 04:40, Steve French wrote:
> MAX_NESTED_LINKS seems to have been set to 5, but the comment in
> fs/namei.c indicates a larger number. Is that intentional?
yes for now; 8 overflowed stacks on several architectures.
Al Viro has a patch series is finalizing getting that merged that makes
it safe to use 8 again.
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2004-08-31 7:38 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-08-31 2:40 MAX_NESTED_LINKS Steve French
2004-08-31 7:38 ` MAX_NESTED_LINKS Arjan van de Ven
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®