[PATCH 3/3] iphlpapi: Implement ParseNetworkString for IPv6 addresses and services

Alex Henrie alexhenrie24 at gmail.com
Thu Mar 19 20:50:04 CDT 2020


On Wed, Mar 18, 2020 at 11:06 PM Nikolay Sivov <nsivov at codeweavers.com> wrote:
>
> On 3/19/20 6:06 AM, Alex Henrie wrote:
> > +    if (type & NET_STRING_IPV6_ADDRESS)
> > +    {
> > +        status = RtlIpv6StringToAddressExW(str, &temp_addr6, &temp_scope, &temp_port);
> > +        if (SUCCEEDED(status) && !temp_port)
> > +        {
> NTSTATUS is usually checked for 0 aka STATUS_SUCCESS.

Hmm, ParseNetworkString already has code that uses the SUCCEEDED macro
instead of checking specifically for STATUS_SUCCESS. If Alexandre
rejects the patch over it, I'll resend and correct all four uses of
SUCCEEDED at the same time.

-Alex



More information about the wine-devel mailing list