July 2023 - Wine-gitlab - list.winehq.org

Re: [PATCH v3 0/4] MR3423: Iphlpapi: Implement NotifyAddrChange (part 1 of 2)
by Paul Gofman (＠gofman) 27 Jul '23

27 Jul '23

Beyond the test included in the patches, I did roughly the following: ptr = VirtualAlloc(NULL 4096 * 2, PAGE_NOACCESS); VirtualProtect(ptr, 4096, PAGE_READWRITE); params = ptr + 4096 - sizeof(*params). And used that pointer. So far it doesn't access it beyond the end of accessible memory (that succeeds). It might happen that with some other values for unknown parameter it is using more data from the structure, but IMO exploring more details about those parameters, in the absence of any known app which uses those functions, looks like a waste of time to me. It is relatively easy here with nsi and was already explored for other functions so I didn't want to break this tradition in nsi, but in general it seems impractical to me to spend huge amount of time reversing the exact parameters of the undocumented functions which nothing is using directly. Not feasible for implementing larger APIs and the benefit is not obvious. And once something is using directly in a way we don't support we can rather easily look how and test with those parameters. I could possibly add a fixme for nonzero unk parameter? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3423#note_40570

1 0

Re: [PATCH v3 0/4] MR3423: Iphlpapi: Implement NotifyAddrChange (part 1 of 2)
by Zebediah Figura (＠zfigura) 27 Jul '23

27 Jul '23

Do we know for sure that the Ex version includes only those five parameters? It seems surprising that they'd reroll the API without adding anything. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3423#note_40569

1 0

Re: [PATCH v3 0/4] MR3423: Iphlpapi: Implement NotifyAddrChange (part 1 of 2)
by Paul Gofman (＠gofman) 27 Jul '23

27 Jul '23

On Thu Jul 27 08:45:24 2023 +0000, Huw Davies wrote: > > So far I don't have anything for the other iphlpapi notification > functions (e. g., NotifyIpInterfaceChange). > I suspect the newer notifications like this one use the > `Nsi(De)RegisterChangeNotification()` api instead. Yes, it seems like it, I was just reasoning on how that could work (most notably, those notifications will need to deliver changed row). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3423#note_40564

1 0

Re: [PATCH v3 0/4] MR3423: Iphlpapi: Implement NotifyAddrChange (part 1 of 2)
by Paul Gofman (＠gofman) 27 Jul '23

27 Jul '23

v2: - split the MR (stop after handle caching commit); - test and add NsiRequestChangeNotificationEx (and move the implementation there); - Mind alphabetical sort when adding NsiCancelChangeNotification. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3423#note_40563

1 0

[PATCH v2 0/4] MR3423: Iphlpapi: Implement NotifyAddrChange (part 1 of 2)
by Paul Gofman (＠gofman) 27 Jul '23

27 Jul '23

-- v2: nsi: Cache nsi device handle. nsi/tests: Add test for change notifications. nsi: Add stub for NsiCancelChangeNotification. nsi: Add stubs for NsiRequestChangeNotification[Ex]. https://gitlab.winehq.org/wine/wine/-/merge_requests/3423

2 4

[PATCH 0/1] MR3429: po: Update Finnish translation.
by Lauri Kenttä 27 Jul '23

27 Jul '23

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/3429

1 1

[PATCH v22 0/2] MR3391: mshtml: implement window.MutationObserver with MutationObserver stub
by Yuxuan Shui (＠yshui) 27 Jul '23

27 Jul '23

This get us pass the "Update your browser" blocker in adobe's sign-in page. The page itself doesn't make use of `window.MutationObserver`. However the sign-in page is still broken. -- v22: mshtml: add stubs for MutationObserver methods mshtml: implement window.MutationObserver with MutationObserver stub https://gitlab.winehq.org/wine/wine/-/merge_requests/3391

3 3

[PATCH v21 0/2] MR3391: mshtml: implement window.MutationObserver with MutationObserver stub
by Yuxuan Shui (＠yshui) 27 Jul '23

27 Jul '23

This get us pass the "Update your browser" blocker in adobe's sign-in page. The page itself doesn't make use of `window.MutationObserver`. However the sign-in page is still broken. -- v21: mshtml: add stubs for MutationObserver methods mshtml: implement window.MutationObserver with MutationObserver stub https://gitlab.winehq.org/wine/wine/-/merge_requests/3391

3 3

[PATCH v23 0/1] MR3074: Move getpeername() implementation to wineserver
by Ally Sommers (＠dropbear) 27 Jul '23

27 Jul '23

`getpeername()` is currently handled in ntdll. This merge request changes ntdll to forward `getpeername()` to wineserver. The implementation utilises new `peer_addr` and `peer_addr_len` fields in wineserver's `struct sock`. This fixes multiple `todo_wine`s in `ws2_32/tests` and allows for more accurate peer names to be provided in case the address the socket is bound to on the Unix side does not match what `bind()` was called with on the Windows side. *This merge request was originally intended to be included in !2786 (Add support for AF_UNIX sockets) but was split into its own merge request because this is not an insignificant change.* -- v23: server: Move getpeername() implementation from ntdll/unix. https://gitlab.winehq.org/wine/wine/-/merge_requests/3074

3 2

[PATCH v20 0/2] MR3391: mshtml: implement window.MutationObserver with MutationObserver stub
by Yuxuan Shui (＠yshui) 27 Jul '23

27 Jul '23

This get us pass the "Update your browser" blocker in adobe's sign-in page. The page itself doesn't make use of `window.MutationObserver`. However the sign-in page is still broken. -- v20: mshtml: add stubs for MutationObserver methods mshtml: implement window.MutationObserver with MutationObserver stub https://gitlab.winehq.org/wine/wine/-/merge_requests/3391

4 4