Skip to content

Commit a45802e

Browse files
author
renyongquan
committed
Bugfix: Resulted in duplicated Content-length in nginx1.6.0 because of r->headers_out.content_length is NULL
1 parent f0c5a13 commit a45802e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

ngx_http_subrange_module.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -765,7 +765,7 @@ static ngx_int_t ngx_http_subrange_header_filter(ngx_http_request_t *r){
765765
content_length.data = ngx_palloc(r->pool, NGX_SIZE_T_LEN);
766766
content_length.len = ngx_sprintf(content_length.data, "%ui", ctx->content_range.total)
767767
- content_length.data;
768-
ngx_http_subrange_set_header(r, &r->headers_out.headers, content_length_key, content_length,NULL);
768+
ngx_http_subrange_set_header(r, &r->headers_out.headers, content_length_key, content_length, &r->headers_out.content_length);
769769

770770
r->headers_out.status_line = ngx_http_status_lines[0];
771771
r->headers_in.range = NULL; // clear the request range header to surpress ngx_http_range_filter_module
@@ -777,7 +777,7 @@ static ngx_int_t ngx_http_subrange_header_filter(ngx_http_request_t *r){
777777
content_length.data = ngx_palloc(r->pool, NGX_SIZE_T_LEN);
778778
content_length.len = ngx_sprintf(content_length.data, "%ui", r->headers_out.content_length_n)
779779
- content_length.data;
780-
ngx_http_subrange_set_header(r, &r->headers_out.headers, content_length_key, content_length, NULL);
780+
ngx_http_subrange_set_header(r, &r->headers_out.headers, content_length_key, content_length, &r->headers_out.content_length);
781781

782782
size = 0;
783783
size += sizeof("bytes -/") - 1 + 3 * NGX_SIZE_T_LEN;

0 commit comments

Comments
 (0)