[LTP] [PATCH] cve-2017-17052: tolerate ENOMEM during test
Li Wang
liwang@redhat.com
Fri Jan 30 01:01:01 CET 2026
On Thu, Jan 29, 2026 at 10:23 PM Cyril Hrubis <chrubis@suse.cz> wrote:
> 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");
>
Good suggestion, patch merged like this. Thanks!
--
Regards,
Li Wang
More information about the ltp
mailing list