[OE-core] [PATCH 2/3] ptest-runner: close all fds in child process.

[Richard Purdie]

On Fri, 2019-05-31 at 16:25 -0400, Sakib Sajal wrote:
> When running 'ptest-runner bash', a test named vredir fails
> due to too many open file descriptors.
> The test sets the maximum number of open file descriptors
> to 6 (ulimit -n 6). The test passes by interactively calling
> run-ptest, but not when using ptest-runner.
> Security-focused applications will close all unused file descriptors
> in
> the child after forking but before execing.
> 
> From the failed ptest log:
>    run-vredir
>    87,88c87,88
>    < ./vredir6.sub: line 10: /dev/null: Too many open files
>    < ./vredir6.sub: line 13: /dev/null: Too many open files
>    FAIL: run-vredir
> 
> Upstream-Status: Submitted [yocto at yoctoproject.org]
> 
> Signed-off-by: Sakib Sajal <sakib.sajal at windriver.com>
> Signed-off-by: Randy Macleod <randy.macleod at windriver.com>

Thanks, this looks like a good fix.

Since I believe Anibal has merged this into the upstream source, could
you send a patch which updates the revision ptest-runner2 is using
instead please? That just then saves adding and then dropping the
patch.

Cheers,

Richard