On Tue, Sep 17, 2013 at 7:00 AM, Alexandre Julliard <julli...@winehq.org> wrote:
> Bruno Jesus <00cp...@gmail.com> writes:
>
>> @@ -5884,20 +5884,20 @@ SOCKET WINAPI WSASocketW(int af, int type, int 
>> protocol,
>>        return ret;
>>      }
>>
>> -    /* convert the socket family and type */
>> -    af = convert_af_w2u(af);
>> -    type = convert_socktype_w2u(type);
>> -
>>      if (lpProtocolInfo)
>>      {
>> -        if (af == FROM_PROTOCOL_INFO)
>> +        if (af <= 0)
>>              af = lpProtocolInfo->iAddressFamily;
>> -        if (type == FROM_PROTOCOL_INFO)
>> +        if (type <= 0)
>>              type = lpProtocolInfo->iSocketType;
>> -        if (protocol == FROM_PROTOCOL_INFO)
>> +        if (protocol <= 0)
>>              protocol = lpProtocolInfo->iProtocol;
>
> I don't see anything in the tests that would justify checking for
> negative values here.

You're right, sorry. New version sent.

> --
> Alexandre Julliard
> julli...@winehq.org

Reagrds,
Bruno


Reply via email to