]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
MFC r317707:
authorbrooks <brooks@FreeBSD.org>
Tue, 9 May 2017 16:27:20 +0000 (16:27 +0000)
committerbrooks <brooks@FreeBSD.org>
Tue, 9 May 2017 16:27:20 +0000 (16:27 +0000)
commitd1c5218b098101b458b1362327aa5dc754eefcf2
tree0d3ddfdf773828f0d0bff94a0e0de3b5672c7aae
parentb5b8ec81d73f98f3cce4fde5c68798bf1cae65aa
MFC r317707:

Correct an out-of-bounds read in regcomp when the RE is bad.

When passed the invalid regular expression "a**", the error is
eventually detected and seterr() is called. It sets p->error
appropriatly and p->next and p->end to nuls which is a never used char
nuls[10] which is zeros due to .bss initialization. Unfortunatly,
p_ere_exp() and p_simp_re() both have fall through cases where they set
the error, decrement p->next and access it which means a read from
whatever .bss variable comes before nuls.

Found with regex_test:repet_multi and CHERI bounds checking.

Reviewed by: ngie, pfg, emaste
Obtained from: CheriBSD
Sponsored by: DARPA, AFRL
Differential Revision: https://reviews.freebsd.org/D10541
lib/libc/regex/regcomp.c