Return-Path: Delivered-To: apmail-apr-dev-archive@www.apache.org Received: (qmail 5742 invoked from network); 6 Jan 2009 16:34:05 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 6 Jan 2009 16:34:05 -0000 Received: (qmail 91010 invoked by uid 500); 6 Jan 2009 16:34:03 -0000 Delivered-To: apmail-apr-dev-archive@apr.apache.org Received: (qmail 90963 invoked by uid 500); 6 Jan 2009 16:34:03 -0000 Mailing-List: contact dev-help@apr.apache.org; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Id: Delivered-To: mailing list dev@apr.apache.org Received: (qmail 90954 invoked by uid 99); 6 Jan 2009 16:34:03 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 06 Jan 2009 08:34:03 -0800 X-ASF-Spam-Status: No, hits=1.2 required=10.0 tests=SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (nike.apache.org: local policy) Received: from [64.202.165.221] (HELO smtpout05.prod.mesa1.secureserver.net) (64.202.165.221) by apache.org (qpsmtpd/0.29) with SMTP; Tue, 06 Jan 2009 16:33:54 +0000 Received: (qmail 15431 invoked from network); 6 Jan 2009 16:31:49 -0000 Received: from unknown (76.252.112.72) by smtpout05-04.prod.mesa1.secureserver.net (64.202.165.221) with ESMTP; 06 Jan 2009 16:31:48 -0000 Message-ID: <49638770.7030904@rowe-clan.net> Date: Tue, 06 Jan 2009 10:31:44 -0600 From: "William A. Rowe, Jr." User-Agent: Thunderbird 2.0.0.18 (X11/20081119) MIME-Version: 1.0 To: Nick Kew CC: dev@httpd.apache.org, dev@apr.apache.org Subject: Re: svn commit: r731965 - /httpd/httpd/trunk/modules/generators/mod_cgid.c References: <20090106150535.3691123888A5@eris.apache.org> <49637D21.7010304@webthing.com> In-Reply-To: <49637D21.7010304@webthing.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org Nick Kew wrote: > [crossposting because this httpd issue raises an APR issue] > > Jeff Trawick wrote: >> On Tue, Jan 6, 2009 at 10:05 AM, > > wrote: >> >> Author: trawick >> Date: Tue Jan 6 07:05:33 2009 >> New Revision: 731965 >> >> URL: http://svn.apache.org/viewvc?rev=731965&view=rev >> >> Log: >> the length arguments are apr_size_t, not int > > How is it apr_size_t? > > I see the following in apr_want.h. Neither definition uses > apr_size_t. "man iovec" shows int on Solaris, size_t on > FreeBSD and Linux. Never AFAICS an apr_size_t. > > apr_want.h contains the following, which appears to leave open > the possibility of not having it at all. Do we have a deeper bug > here that calls for an apr_iovec datatype matching the platform? Yes, I think we need an apr_iov_len_t type. Bill