Return-Path: Delivered-To: apmail-apache-cvs-archive@apache.org Received: (qmail 74671 invoked by uid 500); 15 Sep 2000 21:42:53 -0000 Mailing-List: contact apache-cvs-help@apache.org; run by ezmlm Precedence: bulk X-No-Archive: yes Reply-To: new-httpd@apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list apache-cvs@apache.org Received: (qmail 74643 invoked by uid 500); 15 Sep 2000 21:42:52 -0000 Delivered-To: apmail-apache-2.0-cvs@apache.org X-Authentication-Warning: koj: rbb owned process doing -bs Date: Fri, 15 Sep 2000 14:45:28 -0700 (PDT) From: rbb@covalent.net X-Sender: rbb@koj To: new-httpd@apache.org cc: apache-2.0-cvs@apache.org Subject: Re: cvs commit: apache-2.0/src/modules/standard mod_include.c In-Reply-To: <20000915212346.64361.qmail@locus.apache.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Spam-Rating: locus.apache.org 1.6.2 0/1000/N On 15 Sep 2000 trawick@locus.apache.org wrote: > trawick 00/09/15 14:23:46 > > Modified: src/modules/standard mod_include.c > Log: > Handle a tag that appears at the start of a bucket. This is incorrect. A split at the start of a bucket should create a zero length bucket and a bucket with the rest of the data. If splitting a bucket at the zero offset returns an error, then we end up with a lot of special case code throughout the server. > Get rid of an endless loop (waiting for dptr2 to become NULL instead of > AP_BRIGADE_SENTINEL()). This change is also incorrect. If the find_string does not find the starting sequence, then it will return NULL. With this change, we will search for an ending sequence even if we don't have a starting sequence. Ryan