Hi Michael, this isn't actually a problem with your patch, just something I spotted:

On Wed, Feb 6, 2013 at 3:15 PM, Michael Stefaniuc <mstefani@redhat.com> wrote:
On 02/06/2013 11:16 PM, Austin English wrote:
> On Feb 6, 2013 11:13 PM, "Michael Stefaniuc" <mstefani@redhat.de
> <mailto:mstefani@redhat.de>> wrote:
>>
>> ---
>>  dlls/iphlpapi/ipstats.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/dlls/iphlpapi/ipstats.c b/dlls/iphlpapi/ipstats.c
>> index f8191ba..966d24e 100644
>> --- a/dlls/iphlpapi/ipstats.c
>> +++ b/dlls/iphlpapi/ipstats.c
>> @@ -693,7 +693,7 @@ DWORD WINAPI GetIcmpStatisticsEx(PMIB_ICMP_EX
> stats, DWORD family)
>>      }
>>
>>      ret = GetIcmpStatistics(&ipv4stats);
>> -    if SUCCEEDED(ret)
>> +    if (SUCCEEDED(ret))

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).
--Juan