Return-Path: Delivered-To: apmail-httpd-dev-archive@www.apache.org Received: (qmail 96639 invoked from network); 5 Nov 2008 10:18:41 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 5 Nov 2008 10:18:41 -0000 Received: (qmail 49918 invoked by uid 500); 5 Nov 2008 10:18:41 -0000 Delivered-To: apmail-httpd-dev-archive@httpd.apache.org Received: (qmail 49863 invoked by uid 500); 5 Nov 2008 10:18:41 -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: List-Id: Delivered-To: mailing list dev@httpd.apache.org Received: (qmail 49853 invoked by uid 99); 5 Nov 2008 10:18:41 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 05 Nov 2008 02:18:41 -0800 X-ASF-Spam-Status: No, hits=1.2 required=10.0 tests=SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (athena.apache.org: local policy) Received: from [80.229.52.226] (HELO munin.local) (80.229.52.226) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 05 Nov 2008 10:17:25 +0000 Received: from [127.0.0.1] (localhost [127.0.0.1]) by munin.local (Postfix) with ESMTP id 1E9AC2790A3 for ; Wed, 5 Nov 2008 10:17:23 +0000 (GMT) Mime-Version: 1.0 (Apple Message framework v753.1) In-Reply-To: <1733D7DC-F7AC-4CC6-9EA4-14A78DADDF2B@jaguNET.com> References: <20081104004457.06BBF238889E@eris.apache.org> <1733D7DC-F7AC-4CC6-9EA4-14A78DADDF2B@jaguNET.com> Content-Type: text/plain; charset=US-ASCII; delsp=yes; format=flowed Message-Id: <6BB33ACB-32F5-4C24-A757-302F10297067@webthing.com> Content-Transfer-Encoding: 7bit From: Nick Kew Subject: Re: svn commit: r711146 - in /httpd/httpd/trunk: modules/arch/unix/mod_unixd.c os/unix/unixd.c os/unix/unixd.h server/mpm/experimental/event/event.c server/mpm/prefork/prefork.c server/mpm/worker/worker.c Date: Wed, 5 Nov 2008 10:17:22 +0000 To: dev@httpd.apache.org X-Mailer: Apple Mail (2.753.1) X-Virus-Checked: Checked by ClamAV on apache.org On 4 Nov 2008, at 21:14, Jim Jagielski wrote: > > On Nov 3, 2008, at 7:44 PM, niq@apache.org wrote: > >> Author: niq >> Date: Mon Nov 3 16:44:56 2008 >> New Revision: 711146 >> >> URL: http://svn.apache.org/viewvc?rev=711146&view=rev >> Log: >> Further unixd hacks to remove duplication between old-unixd and >> mod_unixd, >> and get it working with old MPMS[1] + mod_unixd. It's still an >> uneasy >> split, as some modules (mod_cgid, suexec)[2] also use unixd. >> More thinking+hacking due. >> >> [1] Should be prefork/worker/event, but only worker is tested. >> [2] cgid is OK, suexec is untested. >> > > I was playing around with avoiding the overlap we have a unix.h > which somewhat-matches stuff in mod_unixd.c (I was doing this > to create a mod_unixd.h) but this started getting screwy fast. > > What, exactly, do we want the functional and architectural > differences between unixd.c and mod_unixd.c to be?? The old unixd felt like a pot-pourri. The new mod_unixd attempts to focus on a single task, now called drop_privileges. We don't have a clean break, 'cos I've been lazy and left the unixd config as-is, including the is-suexec-enabled flag which doesn't really belong in mod_unixd. Apart from that, what remains in os/unix is just whatever doesn't belong in mod_unixd. -- Nick Kew