about summary refs log tree commit diff
path: root/src/passwd
diff options
context:
space:
mode:
authorSzabolcs Nagy <nsz@port70.net>2015-04-18 15:51:16 +0000
committerRich Felker <dalias@aerifal.cx>2016-01-30 20:53:04 -0500
commit7eaa76fc2e7993582989d3838b1ac32dd8abac09 (patch)
tree8d3c0772c1c85b0ba9017fbf919a654a821d8b2d /src/passwd
parenta8cc2253843e30dfbdf0bb2954439d9f2b2e8704 (diff)
downloadmusl-7eaa76fc2e7993582989d3838b1ac32dd8abac09.tar.gz
musl-7eaa76fc2e7993582989d3838b1ac32dd8abac09.tar.xz
musl-7eaa76fc2e7993582989d3838b1ac32dd8abac09.zip
regex: reject repetitions in some cases with REG_BADRPT
Previously repetitions were accepted after empty expressions like
in (*|?)|{2}, but in BRE the handling of * and \{\} were not
consistent: they were accepted as literals in some cases and
repetitions in others.

It is better to treat repetitions after an empty expression as an
error (this is allowed by the standard, and glibc mostly does the
same). This is hard to do consistently with the current logic so
the new rule is:

Reject repetitions after empty expressions, except after assertions
^*, $? and empty groups ()+ and never treat them as literals.

Empty alternation (|a) is undefined by the standard, but it can be
useful so that should be accepted.
Diffstat (limited to 'src/passwd')
0 files changed, 0 insertions, 0 deletions