j...@apache.org wrote:
> Author: jim
> Date: Thu Oct 17 17:21:04 2013
> New Revision: 1533169
>
> URL: http://svn.apache.org/r1533169
> Log:
> from an idea by Yann Ylavic <ylavic....@gmail.com>
>
> Modified:
> httpd/httpd/trunk/modules/proxy/proxy_util.c
>
> Modified: httpd/httpd/trunk/modules/proxy/proxy_util.c
> URL:
> http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/proxy_util.c?rev=1533169&r1=1533168&r2=1533169&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/modules/proxy/proxy_util.c (original)
> +++ httpd/httpd/trunk/modules/proxy/proxy_util.c Thu Oct 17 17:21:04 2013
> @@ -102,7 +102,7 @@ PROXY_DECLARE(apr_status_t) ap_proxy_str
> thelen = thenil - dst;
> /* Assume the typical case is smaller copying into bigger
> so we have a fast return */
> - if ((thelen < dlen-1) || ((strlen(src)) == thelen)) {
> + if ((thelen < dlen-1) || (src[thelen] == '\0') {
Can't this be reduced to just
if (src[thelen] == '\0') {
?
Regards
Rüdiger
