{"thread":{"id":"13222","subject":"[PATCH] archive.c: format_subst - fixed bogus argument to memchr","startedAt":"2008-04-23T01:06:27Z","lastAt":"2008-04-23T01:06:27Z","messageCount":1,"participants":["Ariel Badichi"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"75022","messageId":"873apdpegc.fsf@sneeze.site","threadId":"13222","inReplyTo":null,"subject":"[PATCH] archive.c: format_subst - fixed bogus argument to memchr","fromName":"Ariel Badichi","fromEmail":"abadichi@bezeqint.net","sentAt":"2008-04-23T01:06:27Z","receivedAt":"2008-04-23T01:06:27Z","isPatch":true,"sender":{"key":"abadichi@bezeqint.net","avatar":null},"body":"\nAlso removed a superfluous test.\n\nSigned-off-by: Ariel Badichi <abadichi@bezeqint.net>\n---\n archive.c |    4 ++--\n 1 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/archive.c b/archive.c\nindex fb159fe..7a32c19 100644\n--- a/archive.c\n+++ b/archive.c\n@@ -16,9 +16,9 @@ static void format_subst(const struct commit *commit,\n \t\tconst char *b, *c;\n \n \t\tb = memmem(src, len, \"$Format:\", 8);\n-\t\tif (!b || src + len < b + 9)\n+\t\tif (!b)\n \t\t\tbreak;\n-\t\tc = memchr(b + 8, '$', len - 8);\n+\t\tc = memchr(b + 8, '$', (src + len) - b - 8);\n \t\tif (!c)\n \t\t\tbreak;\n \n-- \n1.5.5.1.57.g5909c\n"}]}