<div dir="ltr">Hi Michael, this isn't actually a problem with your patch, just something I spotted:<div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 6, 2013 at 3:15 PM, Michael Stefaniuc <span dir="ltr"><<a href="mailto:mstefani@redhat.com" target="_blank">mstefani@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 02/06/2013 11:16 PM, Austin English wrote:<br>
> On Feb 6, 2013 11:13 PM, "Michael Stefaniuc" <<a href="mailto:mstefani@redhat.de">mstefani@redhat.de</a><br>
</div><div><div class="h5">> <mailto:<a href="mailto:mstefani@redhat.de">mstefani@redhat.de</a>>> wrote:<br>
>><br>
>> ---<br>
>> �dlls/iphlpapi/ipstats.c | � �2 +-<br>
>> �1 files changed, 1 insertions(+), 1 deletions(-)<br>
>><br>
>> diff --git a/dlls/iphlpapi/ipstats.c b/dlls/iphlpapi/ipstats.c<br>
>> index f8191ba..966d24e 100644<br>
>> --- a/dlls/iphlpapi/ipstats.c<br>
>> +++ b/dlls/iphlpapi/ipstats.c<br>
>> @@ -693,7 +693,7 @@ DWORD WINAPI GetIcmpStatisticsEx(PMIB_ICMP_EX<br>
> stats, DWORD family)<br>
>> � � �}<br>
>><br>
>> � � �ret = GetIcmpStatistics(&ipv4stats);<br>
>> - � �if SUCCEEDED(ret)<br>
>> + � �if (SUCCEEDED(ret))<br></div></div></blockquote><div><br></div><div style>This code is incorrect. All paths that set ret yield a non-negative value for ret, so SUCCEEDED(ret) is always 1. Using SUCCEEDED/FAILED on something other than an HRESULT is asking for trouble. The correct thing to do is if (!ret).</div>
<div style>--Juan</div></div></div></div>