<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 15, 2022 at 10:19 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi!<br>
> > diff --git a/testcases/kernel/syscalls/open/open04.c b/testcases/kernel/syscalls/open/open04.c<br>
> > index d452405d4..01a8b12d6 100644<br>
> > --- a/testcases/kernel/syscalls/open/open04.c<br>
> > +++ b/testcases/kernel/syscalls/open/open04.c<br>
> > @@ -33,7 +33,12 @@ static void setup(void)<br>
> <br>
> >     for (i = first + 1; i < fds_limit; i++) {<br>
> >             sprintf(fname, FNAME ".%d", i);<br>
> > -           fd = SAFE_OPEN(fname, O_RDWR | O_CREAT, 0777);<br>
> > +           fd = open(fname, O_RDWR | O_CREAT, 0777);<br>
> > +           if (fd == -1) {<br>
> > +                   if (errno != EMFILE)<br>
> > +                           tst_brk(TBROK, "Expected EMFILE but got %d", errno);<br>
> > +                   break;<br>
> > +           }<br>
> >             fds[i - first] = fd;<br>
> >     }<br>
> >  }<br>
> <br>
> LGTM.<br>
> Reviewed-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
<br>
I faintly remmeber a similar patch where we decided not to work around<br>
for a test harness leaking filedescriptors into testcases.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">This also should be a solution, I searched the mailing list and got a patch[1].</div><div class="gmail_default" style="font-size:small">Do you mean adding that close-on-exec flag when opening fd in harness?</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">[1] <a href="https://lists.linux.it/pipermail/ltp/2020-November/019650.html">https://lists.linux.it/pipermail/ltp/2020-November/019650.html<br></a></div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>