fix incorrect comparison loop condition in memmem
authorRich Felker <dalias@aerifal.cx>
Thu, 19 Jun 2014 04:42:28 +0000 (00:42 -0400)
committerRich Felker <dalias@aerifal.cx>
Thu, 19 Jun 2014 04:42:28 +0000 (00:42 -0400)
commitcef0f289f666b6c963bfd11537a6d80916ff889e
tree9360c286947f79ef1f3dc8ac98def74e125e7ba1
parent7c73cacd09a51a87484db5689864743e4984a84d
fix incorrect comparison loop condition in memmem

the logic for this loop was copied from null-terminated-string logic
in strstr without properly adapting it to work with explicit lengths.

presumably this error could result in false negatives (wrongly
comparing past the end of the needle/haystack), false positives
(stopping comparison early when the needle contains null bytes), and
crashes (from runaway reads past the end of mapped memory).
src/string/memmem.c