You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Martin Kraemer <Ma...@mch.sni.de> on 1998/01/09 09:47:01 UTC

Re: [PATCH] protocol/1399: MISE 4.0 POST, then 401 Unauth, then garbage

On Wed, Nov 12, 1997 at 07:21:08AM -0800, Roy T. Fielding wrote:
> This is what I mean by adding discard_request_body(r) to die().
> Can somebody test it on the user's problem?
>      /*
> +     * If we want to keep the connection, be sure that the request body
> +     * (if any) has been read.
> +     */
> +    if ((r->status != HTTP_NOT_MODIFIED) && (r->status != HTTP_NO_CONTENT)
> +        && !status_drops_connection(r->status)
> +        && r->connection && (r->connection->keepalive != -1)) {
> +
> +        (void) discard_request_body(r);
> +    }

An (untested) +1: it looks like the right thing to do.
In which situation could r->connection be NULL?

   Martin
-- 
| S I E M E N S |  <Ma...@mch.sni.de>  |      Siemens Nixdorf
| ------------- |   Voice: +49-89-636-46021     |  Informationssysteme AG
| N I X D O R F |   FAX:   +49-89-636-44994     |   81730 Munich, Germany
~~~~~~~~~~~~~~~~My opinions only, of course; pgp key available on request