apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@jaguNET.com>
Subject Re: svn commit: r731965 - /httpd/httpd/trunk/modules/generators/mod_cgid.c
Date Wed, 07 Jan 2009 13:43:49 GMT

On Jan 6, 2009, at 11:31 AM, William A. Rowe, Jr. wrote:

> Nick Kew wrote:
>> [crossposting because this httpd issue raises an APR issue]
>>
>> Jeff Trawick wrote:
>>> On Tue, Jan 6, 2009 at 10:05 AM, <trawick@apache.org
>>> <mailto:trawick@apache.org>> wrote:
>>>
>>>    Author: trawick
>>>    Date: Tue Jan  6 07:05:33 2009
>>>    New Revision: 731965
>>>
>>>    URL: http://svn.apache.org/viewvc?rev=731965&view=rev
>>>    <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.
>

+1


Mime
View raw message