On 12/12/2008 09:20 PM, [email protected] wrote:
> Author: jorton
> Date: Fri Dec 12 12:20:40 2008
> New Revision: 726109
> 
> URL: http://svn.apache.org/viewvc?rev=726109&view=rev
> Log:
> mod_ssl: Make the size of the per-dir-reneg request-body buffer
> configurable, by popular demand:
> 
> * modules/ssl/ssl_private.h: Define DEFAULT_RENEG_BUFFER_SIZE.
>   (SSLDirConfigRec): Add nRenegBufferSize field.
> 
> * modules/ssl/ssl_engine_config.c (ssl_cmd_SSLRenegBufferSize): New
>   function.
>   (ssl_config_perdir_create, ssl_config_perdir_merge): Handle
>   nRenegBufferSize.
> 
> * modules/ssl/ssl_engine_io.c (ssl_io_buffer_fill): Take max buffer
>   size as an argument rather than compile-time constant.
> 
> * modules/ssl/ssl_engine_kernel.c (ssl_hook_Access): Pass
>   nRenegBufferSize to ssl_io_buffer_fill.
> 
> * modules/ssl/mod_ssl.c (ssl_config_cmds): Add SSLRenegBufferSize.
> 
> PR: 39243
> 
> Modified:
>     httpd/httpd/trunk/CHANGES
>     httpd/httpd/trunk/modules/ssl/mod_ssl.c
>     httpd/httpd/trunk/modules/ssl/ssl_engine_config.c
>     httpd/httpd/trunk/modules/ssl/ssl_engine_io.c
>     httpd/httpd/trunk/modules/ssl/ssl_engine_kernel.c
>     httpd/httpd/trunk/modules/ssl/ssl_private.h

> Modified: httpd/httpd/trunk/modules/ssl/ssl_engine_config.c
> URL: 
> http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/ssl/ssl_engine_config.c?rev=726109&r1=726108&r2=726109&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/modules/ssl/ssl_engine_config.c (original)
> +++ httpd/httpd/trunk/modules/ssl/ssl_engine_config.c Fri Dec 12 12:20:40 2008
> @@ -295,6 +295,8 @@
>      dc->szCACertificateFile    = NULL;
>      dc->szUserName             = NULL;
>  
> +    dc->nRenegBufferSize = DEFAULT_RENEG_BUFFER_SIZE;
> +
>      return dc;
>  }
>  
> @@ -332,6 +334,8 @@
>      cfgMergeString(szCACertificateFile);
>      cfgMergeString(szUserName);
>  
> +    cfgMergeInt(nRenegBufferSize);
> +

Doesn't this require dc->RenegBufferSize to be initialized with UNSET instead
of DEFAULT_RENEG_BUFFER_SIZE to work correctly?


Regards

RĂ¼diger

Reply via email to