Return-Path: Delivered-To: apmail-httpd-dev-archive@httpd.apache.org Received: (qmail 62914 invoked by uid 500); 3 Apr 2002 15:45:28 -0000 Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm Precedence: bulk Reply-To: dev@httpd.apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list dev@httpd.apache.org Received: (qmail 62897 invoked from network); 3 Apr 2002 15:45:28 -0000 X-Authentication-Warning: mako.covalent.net: dougm owned process doing -bs Date: Wed, 3 Apr 2002 07:43:00 -0800 (PST) From: Doug MacEachern To: Subject: Re: fix t/ssl/http.t In-Reply-To: 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 Wed, 3 Apr 2002, Cliff Woolley wrote: > On Tue, 2 Apr 2002, Doug MacEachern wrote: > > > apr_bucket_immortal_create(HTTP_ON_HTTPS_PORT, \ > > - sizeof(HTTP_ON_HTTPS_PORT) - 1, \ > > + sizeof(HTTP_ON_HTTPS_PORT), \ > > Mmmm... no. I don't know how that makes it work its way through, but it's > not right. That null character should never be allowed in. like i said, "this is strange", never suggested the patch was correct. > The correct solution is this: cool, +1