]> granicus.if.org Git - php/commitdiff
Prevent crashing with some bogus POSTs.
authorfoobar <sniper@php.net>
Mon, 1 Apr 2002 23:02:16 +0000 (23:02 +0000)
committerfoobar <sniper@php.net>
Mon, 1 Apr 2002 23:02:16 +0000 (23:02 +0000)
main/rfc1867.c

index 780e25c37939188edd5417ead735f4fa84cf996c..5c997cda6a8dbb043f0d6ad66d97e5466a97e193 100644 (file)
@@ -704,10 +704,17 @@ SAPI_API SAPI_POST_HANDLER_FUNC(rfc1867_post_handler)
                        /* If file_uploads=off, skip the file part */
                        if (!PG(file_uploads)) {
                                efree(filename);
-                               efree(param);
+                               if (param) efree(param);
                                continue;
                        }
 
+                       /* Return with an error if the posted data is garbled */
+                       if (!param) {
+                               sapi_module.sapi_error(E_WARNING, "File Upload Mime headers garbled");
+                               efree(filename);
+                               SAFE_RETURN;
+                       }
+
                        /* Handle file */
                        fp = php_open_temporary_file(PG(upload_tmp_dir), "php", &temp_filename TSRMLS_CC);
                        if (!fp) {