Return-Path: Delivered-To: new-httpd-archive@hyperreal.org Received: (qmail 12768 invoked by uid 6000); 13 May 1999 19:29:08 -0000 Received: (qmail 12601 invoked from network); 13 May 1999 19:28:58 -0000 Received: from twinlark.arctic.org (204.107.140.52) by taz.hyperreal.org with SMTP; 13 May 1999 19:28:58 -0000 Received: (qmail 10711 invoked by uid 500); 13 May 1999 19:28:57 -0000 Date: Thu, 13 May 1999 12:28:57 -0700 (PDT) From: Dean Gaudet To: new-httpd@apache.org Subject: Re: cvs commit: apache-1.3/src/modules/standard mod_autoindex.c mod_include.c In-Reply-To: <373B24A3.40AA3BBE@rsbx.net> Message-ID: X-Comment: Visit http://www.arctic.org/~dgaudet/legal for information regarding copyright and disclaimer. MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: new-httpd-owner@apache.org Precedence: bulk Reply-To: new-httpd@apache.org On Thu, 13 May 1999, Raymond S Brand wrote: > As far as I can tell it's perfectly safe. The original pool that the sub_req > is in has been ap_pool_join()ed to the parent_req pool; it is also a sub pool > of the parent pool. The making of the (empty) sub pool here is, as the comment > says, to allow the sub_req to safely destroyed. The code that destroys reqs > does no other work than to call ap_destroy_pool() so this should be safe. ap_pool_join is an advisory debugging tool -- alloc.c does nothing to ensure that you preserve the lifetimes appropriately. Try two #includes within one .shtml -- set variable foo=bar in the first and set variable bleh=whatever in the second. Do #printenvs in between. Watch for garbage. Dean