Fixed bug: crash on missing Content-Type header

vkholodkov · vkholodkov · commit 8bfea20c4460 · 2008-12-20T16:45:40.000Z
diff --git a/Changelog b/Changelog
@@ -1,4 +1,8 @@
 
+Version 2.0.8
+ * Fixed bug: crash on missing Content-Type request header
+ * Fixed bug: compilation problem on amd 64
+
 Version 2.0.7
  * Change: file size and output body size restrictions
  * Added feature: directive upload_pass_args enables forwarding
diff --git a/ngx_http_upload_module.c b/ngx_http_upload_module.c
@@ -560,7 +560,7 @@ ngx_http_upload_handler(ngx_http_request_t *r)
 
     // Check whether Content-Type header is missing
     if(r->headers_in.content_type == NULL) {
-        ngx_log_error(NGX_LOG_ERR, u->log, ngx_errno,
+        ngx_log_error(NGX_LOG_ERR, r->connection->log, ngx_errno,
                       "missing Content-Type header");
         return NGX_HTTP_BAD_REQUEST;
     }

Original file line number	Diff line number	Diff line change
`@@ -560,7 +560,7 @@ ngx_http_upload_handler(ngx_http_request_t *r)`
`560`	`560`
`561`	`561`	`// Check whether Content-Type header is missing`
`562`	`562`	`if(r->headers_in.content_type == NULL) {`
`563`		`- ngx_log_error(NGX_LOG_ERR, u->log, ngx_errno,`
	`563`	`+ ngx_log_error(NGX_LOG_ERR, r->connection->log, ngx_errno,`
`564`	`564`	`"missing Content-Type header");`
`565`	`565`	`return NGX_HTTP_BAD_REQUEST;`
`566`	`566`	`}`