Re: cvs commit: apache-1.3/src/main http_protocol.c

[Dean Gaudet <dg...@arctic.org>]

This is only half the fix though... this can still call ap_escape_html
with location == NULL.  Granted, it's really a bug in the module which
raised the error requiring a location... so it should cause a 500 to be
raised instead or something like that. 

Dean

On 26 May 1999 coar@hyperreal.org wrote:

> coar        99/05/26 11:28:16
> 
>   Modified:    src      CHANGES
>                src/main http_protocol.c
>   Log:
>   	Make sure that redirect processing looks for the "Location"
>   	field in both headers_out and err_headers_out.
>   
>   Revision  Changes    Path
>   1.1363    +5 -0      apache-1.3/src/CHANGES
>   
>   Index: CHANGES
>   ===================================================================
>   RCS file: /home/cvs/apache-1.3/src/CHANGES,v
>   retrieving revision 1.1362
>   retrieving revision 1.1363
>   diff -u -r1.1362 -r1.1363
>   --- CHANGES	1999/05/25 10:23:15	1.1362
>   +++ CHANGES	1999/05/26 18:27:43	1.1363
>   @@ -1,5 +1,10 @@
>    Changes with Apache 1.3.7
>    
>   +  *) If a "Location" field was stored in r->err_headers_out rather
>   +     than r->headers_out, redirect processing wouldn't find it and
>   +     the server would core dump on ap_escape_html(NULL).
>   +     [Doug MacEachern, Ken Coar]
>   +
>      *) Add RULE_EXPAT, the src/lib/ directory structure, and a modified copy
>         of the Expat 1.0.2 distribution. [Greg Stein]