Skip to content

Commit a83c91d

Browse files
committed
Fix an off-by-one error of own memrchr implementation
and make it support `search_len == 0`, just for the case Ref [Bug #20796]
1 parent 257f78f commit a83c91d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

string.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4548,9 +4548,9 @@ static void*
45484548
memrchr(const char *search_str, int chr, long search_len)
45494549
{
45504550
const char *ptr = search_str + search_len;
4551-
do {
4551+
while (ptr > search_str) {
45524552
if ((unsigned char)*(--ptr) == chr) return (void *)ptr;
4553-
} while (ptr >= search_str);
4553+
}
45544554

45554555
return ((void *)0);
45564556
}

0 commit comments

Comments
 (0)