From dev-return-6593-apmail-apr-dev-archive=apr.apache.org@apr.apache.org Thu May 23 21:06:51 2002 Return-Path: Delivered-To: apmail-apr-dev-archive@apr.apache.org Received: (qmail 66369 invoked by uid 500); 23 May 2002 21:06:51 -0000 Mailing-List: contact dev-help@apr.apache.org; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Delivered-To: mailing list dev@apr.apache.org Received: (qmail 66356 invoked from network); 23 May 2002 21:06:51 -0000 Date: Thu, 23 May 2002 17:05:21 -0400 (EDT) From: Cliff Woolley X-X-Sender: root@deepthought.cs.virginia.edu To: dev@apr.apache.org, Doug MacEachern Subject: Re: cvs commit: apr-util/misc apr_date.c In-Reply-To: <20020523210426.7321.qmail@icarus.apache.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N On 23 May 2002 jwoolley@apache.org wrote: > jwoolley 02/05/23 14:04:26 > > Modified: . CHANGES > include apr_date.h > misc apr_date.c > Log: > Fixed a segfault in apr_date_parse_rfc() for some date formats > because it was overlaying the input string in a totally bogus way. > > Reported by: Doug MacEachern > That ought to fix it. The only reason we were overwriting those bytes was to cause the post-if-elseif-calculation to result in 0 values for certain fields. This seems a clean enough way to do it. If you'd rather I expanded the macros out, I can... it's only three lines per if-block. --Cliff -------------------------------------------------------------- Cliff Woolley cliffwoolley@yahoo.com Charlottesville, VA