Re: [LTP] [PATCH v2 2/2] lstat/lstat02.c: add ELOOP errno test

[chrubis@suse.cz]

Hi!
> > > +	SAFE_MKDIR(cleanup, "test_eloop", MODE_RWX);
> > > +	SAFE_SYMLINK(cleanup, "../test_eloop", "test_eloop/test_eloop");
> > > +	for (i = 0; i < 43; i++)
> > > +		strcat(elooppathname, "/test_eloop");
> > >  }
> > 
> > Hmm, where has the 43 came from?
> > 
> 
> I notice that the consecutive symlinks limits in kernel is hardwired to
> 40 in RHEL 5, 6, 7, but I'm not sure whether this value is the same in
> all linux kernels.

I've looked into the kernel sources and it's hardcoded to 40 in
fs/namei.c in follow_link() so I guess that we are fine with this value.

We should probably add a short comment about where the number came from.

> And the layer of the directory test_eloop/test_eloop is 2, if we want
> the test return ELOOP errno , the layer must more than 41. 43 = 2 + 41.
> 
> > Moreover the elloppathname size is PATH_MAX, you cannot be sure that the
> > whole string will fit (it likely will, but I would rather have value
> > based on the actual string length).
> > 
> 
> How about:
> static char elooppathname[(sizeof("/test_eloop") - 1) * 43 + 2] = ".";

I would simplify this to:

#define ELOPFILE "/test_eloop"

static char elooppathname[sizeof(ELOPFILE) * 43] = ".";

That way the buffer would be a few bytes longer than we need.

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
Flow-based real-time traffic analytics software. Cisco certified tool.
Monitor traffic, SLAs, QoS, Medianet, WAAS etc. with NetFlow Analyzer
Customize your own dashboards, set traffic alerts and generate reports.
Network behavioral analysis & security monitoring. All-in-one tool.
http://pubads.g.doubleclick.net/gampad/clk?id=126839071&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list