From: Fabian Keil Date: Fri, 4 Jan 2013 12:19:47 +0000 (+0000) Subject: Remove an incorrect assertion in server_content_type() X-Git-Tag: v_3_0_20~105 X-Git-Url: http://www.privoxy.org/gitweb/?p=privoxy.git;a=commitdiff_plain;h=4b70ff5ebf195bae5f3398957ddd6d948b435e44 Remove an incorrect assertion in server_content_type() While it was supposed "to see if we already parsed another Content-Type header" it actually only verified that at least one Content-Type headers exists. This is guaranteed to be true as otherwise the function wouldn't be executed by sed(). --- diff --git a/parsers.c b/parsers.c index 47be9199..86d7b793 100644 --- a/parsers.c +++ b/parsers.c @@ -1,4 +1,4 @@ -const char parsers_rcs[] = "$Id: parsers.c,v 1.271 2012/12/07 12:50:17 fabiankeil Exp $"; +const char parsers_rcs[] = "$Id: parsers.c,v 1.272 2012/12/09 12:27:01 fabiankeil Exp $"; /********************************************************************* * * File : $Source: /cvsroot/ijbswa/current/parsers.c,v $ @@ -2105,12 +2105,6 @@ static jb_err server_content_type(struct client_state *csp, char **header) /* Remove header if it isn't the first Content-Type header */ if ((csp->content_type & CT_DECLARED)) { - /* - * Another, slightly slower, way to see if - * we already parsed another Content-Type header. - */ - assert(NULL != get_header_value(csp->headers, "Content-Type:")); - log_error(LOG_LEVEL_ERROR, "Multiple Content-Type headers. Removing and ignoring: \'%s\'", *header);