httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Collinson, Graham" <Graham.Collin...@viatel.com>
Subject RE: [PATCH 15511/15512]
Date Wed, 06 Jul 2005 08:57:07 GMT
Hi everyone,
Did anyone take a look at my patch?
 
I don't think this one will require too much inspection to be commited
to 2.0 head:
 
--- mod_proxy.c 2005-02-04 19:21:18.000000000 +0000
+++ mod_proxy.new.c 2005-06-22 14:13:28.104300500 +0100
@@ -481,14 +481,23 @@

     ps->domain = (overrides->domain == NULL) ? base->domain :
overrides->domain;
     ps->viaopt = (overrides->viaopt_set == 0) ? base->viaopt :
overrides->viaopt;
+    ps->viaopt_set = overrides->viaopt_set || base->viaopt_set;
     ps->req = (overrides->req_set == 0) ? base->req : overrides->req;
+    ps->req_set = overrides->req_set || base->req_set;
     ps->recv_buffer_size = (overrides->recv_buffer_size_set == 0) ?
base->recv_buffer_size : overrides->recv_buffer_size;
+    ps->recv_buffer_size_set = overrides->recv_buffer_size_set ||
base->recv_buffer_size_set;
     ps->io_buffer_size = (overrides->io_buffer_size_set == 0) ?
base->io_buffer_size : overrides->io_buffer_size;
+    ps->io_buffer_size_set = overrides->io_buffer_size_set ||
base->io_buffer_size_set;
     ps->maxfwd = (overrides->maxfwd_set == 0) ? base->maxfwd :
overrides->maxfwd;
+    ps->maxfwd_set = overrides->maxfwd_set || base->maxfwd_set;
     ps->error_override = (overrides->error_override_set == 0) ?
base->error_override : overrides->error_override;
+    ps->error_override_set = overrides->error_override_set ||
base->error_override_set;
     ps->preserve_host = (overrides->preserve_host_set == 0) ?
base->preserve_host : overrides->preserve_host;
+    ps->preserve_host_set = overrides->preserve_host_set ||
base->preserve_host_set;
     ps->timeout= (overrides->timeout_set == 0) ? base->timeout :
overrides->timeout;
+    ps->timeout_set = overrides->timeout_set || base->timeout_set;
     ps->badopt = (overrides->badopt_set == 0) ? base->badopt :
overrides->badopt;
+    ps->badopt_set = overrides->badopt_set || base->badopt_set;

     return ps;
}

________________________________

From: Collinson, Graham [mailto:Graham.Collinson@viatel.com] 
Sent: 01 July 2005 00:01
To: dev@httpd.apache.org
Subject: [PATCH 15511/15512]


It seems to me that someone missed out a few lines for the <opt>_set
variables in mod_proxy merge_proxy_config.
Of course I could be wrong but the patch 15511 (for 2.0.53) I've
attached for bug 11540 works for me - I can now specify a proxytimeout
patch 15512 for 2.1.3-beta is untested but is a similar simple fix,
anyone wish to comment?



For more information about the Viatel Group, please visit www.viatel.com
<http://www.viatel.com/> .

THIS MESSAGE IS INTENDED ONLY FOR THE USE OF THE INTENDED RECIPIENT TO
WHICH IT IS ADDRESSED AND MAY CONTAIN INFORMATION THAT IS PRIVILEGED,
CONFIDENTIAL AND EXEMPT FROM DISCLOSURE. If the reader of this message
is not the intended recipient, or an employee or agent responsible for
delivering the message to the intended recipient, you are notified that
any dissemination, distribution or copying of this e-mail is prohibited,
and you should delete this e-mail from your system. 

This message has been scanned for viruses and spam by Viatel MailControl
<http://www.viatel.com/> .


Mime
View raw message