[PATCH] Added support for ConvertLengthToIpv4Mask in iphlpapi

Huw Davies huw at codeweavers.com
Fri Mar 16 03:46:23 CDT 2018


On Fri, Mar 16, 2018 at 04:21:15PM +0800, Zhiyi Zhang wrote:
> Hi Uberdaff,
>      On success, ConvertLengthToIpv4Mask returns NO_ERROR. Any nonzero
>      return value indicates failure and the Mask parameter is set
>      to INADDR_NONE defined in the Ws2def.h header file.
> MSDN says Mask parameter also gets set on failure.
> 
>      if(MaskLength <= 32)
>              return ERROR_INVALID_PARAMETER;
> Maybe we should check MaskLength lower bound as well?
> 
> I haven't really look into it yet because even MSDN can be wrong. But I suggest
> you add a test to verify.
> If you're sending the patch on others behalf and you're not familiar with how
> to write a test, contact original author.

Yes, you should definitely add some tests.  They will show
that your implementation isn't correct.

Also, change the commit message to:
iphlpapi: Add support for ConvertLengthToIpv4Mask().

and add the prototype to netioapi.h

Huw.



More information about the wine-devel mailing list