httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe, Jr." <wr...@rowe-clan.net>
Subject Re: svn commit: r731965 - /httpd/httpd/trunk/modules/generators/mod_cgid.c
Date Tue, 06 Jan 2009 15:55:03 GMT
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
> Log:
> the length arguments are apr_size_t, not int
> 
> Modified:
>     httpd/httpd/trunk/modules/generators/mod_cgid.c
> 
> Modified: httpd/httpd/trunk/modules/generators/mod_cgid.c
> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/generators/mod_cgid.c?rev=731965&r1=731964&r2=731965&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/modules/generators/mod_cgid.c (original)
> +++ httpd/httpd/trunk/modules/generators/mod_cgid.c Tue Jan  6 07:05:33 2009
> @@ -354,7 +354,7 @@
>      va_start(ap, count);
>      for (i = 0; i < count; i++) {
>          vec[i].iov_base = va_arg(ap, caddr_t);
> -        vec[i].iov_len  = va_arg(ap, int);
> +        vec[i].iov_len  = va_arg(ap, apr_size_t);

More correctly, aren't these size_t native as opposed to apr_'s mapping?
This is a system structure, not an apr structure.


Mime
View raw message