httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Nick Kew <n...@webthing.com>
Subject Re: svn commit: r731965 - /httpd/httpd/trunk/modules/generators/mod_cgid.c
Date Tue, 06 Jan 2009 15:47:45 GMT
[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?


#if APR_HAVE_IOVEC

#if APR_HAVE_SYS_UIO_H
#include <sys/uio.h>
#endif

#else

struct iovec
{
     char *iov_base;
     size_t iov_len;
};

#endif

-- 
Nick Kew

Mime
View raw message