Re: [igt-dev] [PATCH i-g-t] runner_tests: Fix the file descriptor leakage subtest

[Antonio Argenziano <antonio.argenziano@intel.com>]

On 04/06/19 08:12, Arkadiusz Hiler wrote:
> Some libraries (looking at you, GnuTLS) may leave fds opened after the
> implicitly called library constructor. We don't have full control over
> them as they may be dependencies of our dependencies and may get pulled
> in if the user's and distribution's compile/configure/USE are just
> right.
> 
> Let's close all the non-standard fds ahead of executing anything in the
> igt_runner tests, so we can test for descriptor leakage caused by any of
> the igt_runner code-paths exercised there.
> 
> Cc: Antonio Argenziano <antonio.argenziano@intel.com>
> Cc: Petri Latvala <petri.latvala@intel.com>
> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

LGTM.

Reviewed-by: Antonio Argenziano <antonio.argenziano@intel.com>

> ---
>   runner/runner_tests.c | 21 ++++++++++++++++++++-
>   1 file changed, 20 insertions(+), 1 deletion(-)
> 
> diff --git a/runner/runner_tests.c b/runner/runner_tests.c
> index 1fc09688..c09cda70 100644
> --- a/runner/runner_tests.c
> +++ b/runner/runner_tests.c
> @@ -208,8 +208,27 @@ igt_main
>   {
>   	struct settings *settings = malloc(sizeof(*settings));
>   
> -	igt_fixture
> +	igt_fixture {
> +		int i;
> +
> +		/*
> +		 * Let's close all the non-standard fds ahead of executing
> +		 * anything, so we can test for descriptor leakage caused by
> +		 * any of the igt_runner code-paths exercised here.
> +		 *
> +		 * See file-descriptor-leakage subtest at the end.
> +		 *
> +		 * Some libraries (looking at you, GnuTLS) may leave fds opened
> +		 * after the implicitly called library constructor. We don't
> +		 * have full control over them as they may be dependencies of
> +		 * our dependencies and may get pulled in if the user's and
> +		 * distribution's compile/configure/USE are just right.
> +		 */
> +		for (i = 3; i < 400; i++)
> +			close(i);
> +
>   		init_settings(settings);
> +	}
>   
>   	igt_subtest("default-settings") {
>   		const char *argv[] = { "runner",
> 
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev