Re: [PATCH 1/3] ws2_32: Implement SIO_ADDRESS_LIST_CHANGE with NotifyAddrChange (try 2).

[Bruno Jesus]

On Mon, Aug 5, 2013 at 1:24 AM, Erich E. Hoover
<erich.e.hoover(a)gmail.com> wrote:
> The attached patch is the next step in the series of fixes for
> Silverlight/PlayReady under Wine. This particular patch does not
> impact Netflix, but it does impact a number of other PlayReady
> streaming services (and a variety of non-streaming sites as well).
> Sorry it's taken so long to get back to this, life has been busy
> lately.
>
> With this patch the SIO_ADDRESS_LIST_CHANGE call gets passed on to
> NotifyAddrChange, which effectively fixes a problem where Silverlight
> apps can loop forever calling SIO_ADDRESS_LIST_CHANGE (Bug #32328).
> Part 2 actually implements the NotifyAddrChange call on Linux, though
> this patch is sufficient on its own to resolve the bug since the
> overlapped IO event no longer gets triggered.
>
> This version of the patch has bee updated to include tests that show
> that the notification is not tied to the socket.  For example,
> changing an interface that the socket is not bound to still results in
> an overlapped notification.  It's worth noting that these tests
> require interactive mode and a PC with two network cards, so for my
> testing I used an actual (non-VM) Windows 7 box with a wired and a
> wireless card.

Was this patch already reviewed? Is there anything else take can be
done to get it commited? I tested it in 2 different environments (win7
eth+wifi and winxp eth+eht) and the tests work as expected.

Regards,
Bruno