www-apache-bugdb mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Tony Finch <...@dotat.at>
Subject Re: protocol/6973: http_protocol.c isn't compatible with HTTP RFC
Date Wed, 13 Dec 2000 23:40:02 GMT
The following reply was made to PR protocol/6973; it has been noted by GNATS.

From: Tony Finch <dot@dotat.at>
To: Ory Segal <ory.segal@sanctuminc.com>
Cc: apbugs@apache.org
Subject: Re: protocol/6973: http_protocol.c isn't compatible with HTTP RFC
Date: Wed, 13 Dec 2000 23:37:23 +0000

 Actually, scrub that last email. This is a newly discovered problem.
 (I could have sworn I checked that line for sanity when dealing with
 6711.)
 
 What do you think of this patch?
 
 Index: http_protocol.c
 ===================================================================
 RCS file: /home/cvs/apache-1.3/src/main/http_protocol.c,v
 retrieving revision 1.295
 diff -u -r1.295 http_protocol.c
 --- http_protocol.c	2000/11/14 09:57:09	1.295
 +++ http_protocol.c	2000/12/13 23:35:04
 @@ -151,6 +151,8 @@
          /* In the form "-5" */
          *start = clength - atol(dash + 1);
          *end = clength - 1;
 +	if (*start == *end)
 +	    return 0;
      }
      else {
          *dash = '\0';
 @@ -170,8 +172,8 @@
  
      if (*start > *end)
  	return 0;
 -
 -    return (*start > 0 || *end < clength - 1);
 +    else
 +	return 1;
  }
  
  static int internal_byterange(int, long *, request_rec *, const char **, long *,
 
 
 Tony.
 -- 
 f.a.n.finch    fanf@covalent.net    dot@dotat.at
 "There are flying saucers. There's no doubt they are
 in our skies. They've been there for some time."

Mime
View raw message