September 2023 - Wine-gitlab

Re: [PATCH v4 0/1] MR3784: nsiproxy.sys: Implement ipv6_forward_enumerate_all.
by Huw Davies (＠huw) 15 Sep '23

15 Sep '23

Huw Davies (@huw) commented about dlls/nsiproxy.sys/ip.c: > + struct in6_addr prefix; > + UINT prefix_len; > + struct in6_addr next_hop; > + UINT metric; > + UINT protocol; > + BYTE loopback; > +}; > + > +static void ipv6_forward_fill_entry( struct ipv6_route_data *entry, struct nsi_ipv6_forward_key *key, > + struct nsi_ip_forward_rw *rw, struct nsi_ipv6_forward_dynamic *dyn, > + struct nsi_ip_forward_static *stat ) > +{ > + if (key) > + { > + key->unk = 0; > + memcpy( key->prefix.u.Byte, entry->prefix.s6_addr, 16 ); Please use something like `sizeof(entry->prefix)` instead of `16`. Also for `next_hop` below. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3784#note_45497

1 0

Re: [PATCH v4 0/1] MR3784: nsiproxy.sys: Implement ipv6_forward_enumerate_all.
by Huw Davies (＠huw) 15 Sep '23

15 Sep '23

Huw Davies (@huw) commented about dlls/nsiproxy.sys/ip.c: > + { > + memset( dyn, 0, sizeof(*dyn) ); > + } > + > + if (stat) > + { > + stat->origin = NlroManual; > + stat->if_index = entry->if_index; > + } > +} > + > +struct in6_addr str_to_in6_addr(char *nptr, char **endptr) > +{ > + struct in6_addr ret; > + > + for (int i = 0; i < 16; i++) Again something like `sizeof(ret)` instead of `16`. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3784#note_45498

1 0

Re: [PATCH v4 0/1] MR3784: nsiproxy.sys: Implement ipv6_forward_enumerate_all.
by Huw Davies (＠huw) 15 Sep '23

15 Sep '23

Huw Davies (@huw) commented about dlls/nsiproxy.sys/ip.c: > + } > + > + if (stat) > + { > + stat->origin = NlroManual; > + stat->if_index = entry->if_index; > + } > +} > + > +struct in6_addr str_to_in6_addr(char *nptr, char **endptr) > +{ > + struct in6_addr ret; > + > + for (int i = 0; i < 16; i++) > + { > + if (!isalnum( *nptr ) || !isalnum( *nptr+1 )) `isxdigit()`? Also, please add a space either side of the `+`. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3784#note_45499

1 0

[PATCH 0/1] MR3841: wineps: Fix EMRPOLYDRAW16 printing.
by Piotr Caban (＠piotr) 15 Sep '23

15 Sep '23

It fixes EMRPOLYDRAW16 printing when -O2 option is used (in GCC 12.3.1). The compiler assumes that apts table has 0 or 1 elements and optimizes the loop out. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55486 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3841

3 5

[PATCH 0/1] MR3840: wineps: Remove dynamic glyph list.
by Piotr Caban (＠piotr) 15 Sep '23

15 Sep '23

We can't do much without unicode mapping. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3840

3 2

[PATCH 0/7] MR3852: dmusic: Parse instrument waves and download them to the port.
by Rémi Bernon 15 Sep '23

15 Sep '23

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

1 7

[PATCH 0/7] MR3851: dmime: Better implement DMUS_PMSG message processing.
by Rémi Bernon 15 Sep '23

15 Sep '23

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

2 8

[PATCH 0/1] MR3842: dsound: Get rid of the global device GUID arrays.
by Arek Hiler (＠ivyl) 15 Sep '23

15 Sep '23

They are not used for anything anymore and just impose limit of 10 devices of a given type (capturer / renderer) without doing bound checking. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3842

4 3

[PATCH 0/1] MR3847: winmm: Handle memory allocation failure in WINMM_OpenDevice (cppcheck).
by Alex Henrie (＠alexhenrie) 15 Sep '23

15 Sep '23

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

3 2

[PATCH v2 0/4] MR3848: wine{alsa,coreaudio,oss,pulse}: Ensure unixlib function tables and enum stay in sync.
by Brendan Shanks (＠bshanks) 15 Sep '23

15 Sep '23

-- v2: winepulse: Ensure unixlib function tables and enum stay in sync. wineoss: Ensure unixlib function tables and enum stay in sync. winecoreaudio: Ensure unixlib function tables and enum stay in sync. winealsa: Ensure unixlib function tables and enum stay in sync. https://gitlab.winehq.org/wine/wine/-/merge_requests/3848

3 5