Attachment 32644 Details for Bug 57807 – Patch introducing AP_FCGI_MAX_RECORD_LEN for mod_proxy_fcgi

[patch] Patch introducing AP_FCGI_MAX_RECORD_LEN for mod_proxy_fcgi

patch-modules_proxy_mod_proxy_fcgi.c (text/plain), 1.21 KB, created by manu on 2015-04-13 04:22:43 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: manu

Created: 2015-04-13 04:22:43 UTC

Size: 1.21 KB

patch

obsolete

>$NetBSD$
>
>Allow maximum environment variable vlue lentgth to be overriden
>
>--- modules/proxy/mod_proxy_fcgi.c.orig	2015-04-12 19:15:14.000000000 +0200
>+++ modules/proxy/mod_proxy_fcgi.c	2015-04-12 19:19:16.000000000 +0200
>@@ -17,8 +17,19 @@
> #include "mod_proxy.h"
> #include "util_fcgi.h"
> #include "util_script.h"
> 
>+/* 
>+ * Limit for environment vaiable value size,
>+ * must be lower than AP_FCGI_MAX_CONTENT_LEN 
>+ */
>+#ifndef AP_FCGI_MAX_RECORD_LEN
>+#define AP_FCGI_MAX_RECORD_LEN (16 * 1024)
>+#endif
>+#if AP_FCGI_MAX_RECORD_LEN > AP_FCGI_MAX_CONTENT_LEN
>+#error "AP_FCGI_MAX_RECORD_LEN must be less than AP_FCGI_MAX_CONTENT_LEN"
>+#endif
>+
> module AP_MODULE_DECLARE_DATA proxy_fcgi_module;
> 
> typedef struct {
>     int need_dirwalk;
>@@ -297,11 +308,9 @@
> 
>     /* Send envvars over in as many FastCGI records as it takes, */
>     next_elem = 0; /* starting with the first one */
> 
>-    avail_len = 16 * 1024; /* our limit per record, which could have been up
>-                            * to AP_FCGI_MAX_CONTENT_LEN
>-                            */
>+    avail_len = AP_FCGI_MAX_RECORD_LEN; 
> 
>     while (next_elem < envarr->nelts) {
>         starting_elem = next_elem;
>         required_len = ap_fcgi_encoded_env_len(r->subprocess_env,

Actions: View | Diff

Attachments on bug 57807: 32644