|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
PatchesPull RequestsHistoryAllCommentsChangesGit/SVN commits
[2002-03-03 18:14 UTC] sniper@php.net
|
|||||||||||||||||||||||||||
Copyright © 2001-2025 The PHP GroupAll rights reserved. |
Last updated: Sat Nov 29 06:00:01 2025 UTC |
The following was found by source code inspection: You can't add 1 to the result of a memchr() and then still expect to be able to check for a NULL result. Regards, Wolfram. --- php4-4.1.2/main/rfc1867.c.orig Thu Feb 21 19:46:45 2002 +++ php4-4.1.2/main/rfc1867.c Sun Mar 3 22:11:42 2002 @@ -162,15 +162,15 @@ SAFE_RETURN; } /* some other headerfield found, skip it */ - loc = (char *) memchr(ptr, '\n', rem)+1; + loc = (char *) memchr(ptr, '\n', rem); if (!loc) { /* broken */ php_error(E_WARNING, "File Uploa d Mime headers garbled ptr: [%c%c%c%c%c]", *ptr, *(ptr + 1), *(ptr + 2), *(ptr + 3), *(ptr + 4)); SAFE_RETURN; } - while (*loc == ' ' || *loc == '\t') { + while (*++loc == ' ' || *loc == '\t') { /* other field is folded, skip i t */ - loc = (char *) memchr(loc, '\n', rem-(loc-ptr))+1; + loc = (char *) memchr(loc, '\n', rem-(loc-ptr)); if (!loc) { /* broken */ php_error(E_WARNING, "Fi le Upload Mime headers garbled ptr: [%c%c%c%c%c]", *ptr, *(ptr + 1), *(ptr + 2), *(ptr + 3), *(ptr + 4));