[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [PATCH] httpd: don't send error messages with gzip encoding
From:       Denys Vlasenko <vda.linux () googlemail ! com>
Date:       2011-03-27 21:26:45
Message-ID: 201103272326.45798.vda.linux () googlemail ! com
[Download RAW message or body]

On Friday 25 March 2011 13:38, Peter Korsgaard wrote:
> If CONFIG_FEATURE_HTTPD_GZIP is enabled and request contained
> 'Accept-Encoding: gzip', then errors were sent with 'Content-Encoding: gzip'
> even though they aren't.
> 
> Fix it by clearing content_gzip before sending the headers.
> 
> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
> ---
>  networking/httpd.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/networking/httpd.c b/networking/httpd.c
> index b8113a8..9c1aa2a 100644
> --- a/networking/httpd.c
> +++ b/networking/httpd.c
> @@ -1065,6 +1065,7 @@ static void send_headers(int responseNum)
>  static void send_headers_and_exit(int responseNum) NORETURN;
>  static void send_headers_and_exit(int responseNum)
>  {
> +	IF_FEATURE_HTTPD_GZIP(content_gzip = 0;)
>  	send_headers(responseNum);
>  	log_and_exit();
>  }

Applied, thanks!
-- 
vda
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic