wininet: Make the passive mode response check case insensitive.

Hans Leidekker hans at it.vu.nl
Thu Jun 7 06:04:37 CDT 2007


On Thursday 07 June 2007, Francois Gouget wrote:

> -           if (strncmp(p, "Entering Passive Mode", 21))
> +           if (strncasecmp(p, "Entering Passive Mode", 21))
> 
> You should not be using str*casecmp() in Wine (because it's a Unix 
> function, and because it uses the wrong locale). Try CompareString() 
> instead.

I'd say it's okay here because we're comparing a fixed string returned
from the FTP server (it's mentioned in the RFC for the ASCII based FTP
protocol).

 -Hans



More information about the wine-devel mailing list