tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Konstantin Kolinko <knst.koli...@gmail.com>
Subject Re: svn commit: r1187916 - in /tomcat/jk/trunk: native/iis/jk_isapi_plugin.c xdocs/miscellaneous/changelog.xml
Date Tue, 25 Oct 2011 19:57:00 GMT
2011/10/25 Christopher Schultz <chris@christopherschultz.net>:
> All,
>
> On 10/25/2011 2:03 PM, Christopher Schultz wrote:
>> On 10/23/2011 12:19 PM, rjung@apache.org wrote:
>>>
>>> +        if (!strncmp(uri, path, len) &&
>>
>> strncmp doesn't use case-insensitive compare: will this ever match if
>> you use "web-inf" (as below)?
>
> Duh just saw Konstantin's response. Apologies for the noise.
>
> I still think the // might be a problem, though.

> +    while (uri = strchr(uri, '/')) {

If uri starts with '/' then strchr will return uri without advancing
the pointer. I see no problem with //.

Best regards,
Konstantin Kolinko

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message