[LTP] [PATCH] cve-2017-17052: tolerate ENOMEM during test

[Cyril Hrubis]

Hi!
>         for (;;) {
> -               SAFE_MMAP(NULL, 0x1000000, PROT_READ,
> +               ptr = mmap(NULL, 0x1000000, PROT_READ,
>                                 MAP_POPULATE|MAP_ANONYMOUS|MAP_PRIVATE, -1, 0);
> +
> +               if (ptr == MAP_FAILED) {
> +                       if (errno == ENOMEM) {
> +                               usleep(1000);
> +                               continue;
> +                       }
> +                       tst_brk(TBROK | TTERRNO, "Unexpected mmap() error");

Maybe this part would be more readable with an else branch:

			if (errno == ENOMEM)
				usleep(1000);
			else
				tst_brk(TBROK | TTERRNO, "mmap() failed");


Otherwise:

Reviewed-by: Cyril Hrubis <chrubis@suse.cz>


> +               }
>         }
> 
>         return arg;
> 
> 
> 
> -- 
> Regards,
> Li Wang
> 
> 
> -- 
> Mailing list info: https://lists.linux.it/listinfo/ltp

-- 
Cyril Hrubis
chrubis@suse.cz