https://issues.apache.org/bugzilla/show_bug.cgi?id=45405


D. Stussy <[EMAIL PROTECTED]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #22293|0                           |1
        is obsolete|                            |
  Attachment #22300|0                           |1
        is obsolete|                            |




--- Comment #13 from D. Stussy <[EMAIL PROTECTED]>  2008-07-23 14:59:18 PST ---
Created an attachment (id=22309)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=22309)
Updated patch

Updated patch - Cosmetic issues only - functionality not broken:

Changes:
+++ modules/proxy/proxy_util.c
@@ -2349,6 +2374,24 @@
...
+                ap_log_error(APLOG_MARK, APLOG_ERR, 0, s, "proxy: %s: can not
bind to %s:%u+%u",
+                        proxy_function,bind->addr, bind->port, bind->range,
NULL);
+                return DECLINED;

=>   ..., bind->range - 1, NULL);

Internally, it's one more than the admin specified in the configuration, so we
have to subtract one to get the original value that was human-specified.


Comment in parser:
+    * we dont accept a range with out a port either.
without is one word in English.

Added two additional comment lines.


I believe we're done.  I have not actually tested the patch.


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to