[hate to reply to myself, but I don't want to waste other
    people's time looking into this thing.]

    I think I've found the bug.

    The comment in rfc1867.c around line 195 says:

        start_arr is set to point to 1st [

    But the code use strrchr to locate the bracket, so start_arr
    is set to the last bracket.  The attached one-liner fixes it.

    Comments?

    - Sascha
Index: rfc1867.c
===================================================================
RCS file: /repository/php4/main/rfc1867.c,v
retrieving revision 1.58
diff -u -r1.58 rfc1867.c
--- rfc1867.c   2001/01/15 10:50:39     1.58
+++ rfc1867.c   2001/01/19 13:22:27
@@ -195,7 +195,7 @@
                                         * start_arr is set to point to 1st [
                                         * end_arr points to last ]
                                         */
-                                       is_arr_upload = (start_arr = 
strrchr(namebuf,'[')) && 
+                                       is_arr_upload = (start_arr = 
+strchr(namebuf,'[')) && 
                                                                        (end_arr = 
strrchr(namebuf,']')) && 
                                                                        (end_arr = 
namebuf+strlen(namebuf)-1);
                                        if(is_arr_upload) {
-- 
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]

Reply via email to