[nginx] Gunzip: proper error handling on gunzipping an empty res...
Maxim Dounin
mdounin at mdounin.ru
Thu Oct 31 00:19:00 UTC 2013
details: http://hg.nginx.org/nginx/rev/6aa75f4982ce
branches:
changeset: 5436:6aa75f4982ce
user: Maxim Dounin <mdounin at mdounin.ru>
date: Thu Oct 31 04:16:20 2013 +0400
description:
Gunzip: proper error handling on gunzipping an empty response.
With previous code, an empty (malformed) response resulted in a request
finalized without sending anything to a client.
diffstat:
src/http/modules/ngx_http_gunzip_filter_module.c | 10 +++++++---
1 files changed, 7 insertions(+), 3 deletions(-)
diffs (20 lines):
diff --git a/src/http/modules/ngx_http_gunzip_filter_module.c b/src/http/modules/ngx_http_gunzip_filter_module.c
--- a/src/http/modules/ngx_http_gunzip_filter_module.c
+++ b/src/http/modules/ngx_http_gunzip_filter_module.c
@@ -500,9 +500,13 @@ ngx_http_gunzip_filter_inflate(ngx_http_
return NGX_OK;
}
- if (rc == Z_STREAM_END && ctx->flush == Z_FINISH
- && ctx->zstream.avail_in == 0)
- {
+ if (ctx->flush == Z_FINISH && ctx->zstream.avail_in == 0) {
+
+ if (rc != Z_STREAM_END) {
+ ngx_log_error(NGX_LOG_ERR, r->connection->log, 0,
+ "inflate() returned %d on response end", rc);
+ return NGX_ERROR;
+ }
if (ngx_http_gunzip_filter_inflate_end(r, ctx) != NGX_OK) {
return NGX_ERROR;
More information about the nginx-devel
mailing list