August 2025 - Wine-GitLab - list.winehq.org

Re: [PATCH v13 0/1] MR8778: sane.ds: Adding a window for loading and saving the scanner configuration
by Esme Povirk (＠madewokherd) Aug. 19, 2025

Aug. 19, 2025

Esme Povirk (@madewokherd) commented about dlls/sane.ds/cfg.c: > + > + new_content = NULL; > + found = FALSE; > + > + sprintf(search, "%s=", option->name); > + if (content) > + { > + CHAR* lines[1000] = {0}; > + int count = 0; > + CHAR* context = NULL; > + size_t new_size = 0; > + CHAR* line; > + > + line = strtok_s(content, "\n", &context); > + > + while (line && count < 1000) Please don't duplicate the buffer length. You can use the `ARRAY_SIZE` macro. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8778#note_113164

1 0

Re: [PATCH v13 0/1] MR8778: sane.ds: Adding a window for loading and saving the scanner configuration
by Esme Povirk (＠madewokherd) Aug. 19, 2025

Aug. 19, 2025

Esme Povirk (@madewokherd) commented about dlls/sane.ds/cfg.c: > + sprintf(buffer, "%s=%s\n", (option->name), option->value.bool_val ? "true" : "false"); > + break; > + } > + > + size = GetFileSize(hFile, NULL); > + content = (CHAR*)malloc(size + sizeof(CHAR)); > + > + ReadFile(hFile, content, size, &read, NULL); > + content[size / sizeof(CHAR)] = '\0'; > + CloseHandle(hFile); > + > + new_content = NULL; > + found = FALSE; > + > + sprintf(search, "%s=", option->name); > + if (content) This is checking whether the malloc succeeded after the allocated value has already been used. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8778#note_113163

1 0

[PATCH 0/1] MR8785: rsaenh: Use TomCrypt for hash implementations.
by Hans Leidekker (＠hans) Aug. 19, 2025

Aug. 19, 2025

Some apps unload bcrypt which causes a crash if rsaenh is used afterwards. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8785

2 1

[PATCH 0/4] MR8781: jscript: Misc array and toString fixes.
by Gabriel Ivăncescu Aug. 19, 2025

Aug. 19, 2025

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

2 6

[PATCH 0/1] MR8783: dwrite: Add fallback for Dingbats.
by Ziqing Hui (＠zhui) Aug. 19, 2025

Aug. 19, 2025

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

3 2

[PATCH 0/1] MR8758: winegstreamer: Use a stride alignment of 2 for NV12 in align_video_info_planes().
by Conor McCarthy (＠cmccarthy) Aug. 19, 2025

Aug. 19, 2025

Fixes the todos in !8034. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8758

3 3

[PATCH v2 0/11] MR8782: iphlpapi: Implement ipv6 ping.
by Paul Gofman (＠gofman) Aug. 19, 2025

Aug. 19, 2025

-- v2: iphlpapi/tests: Add tests for Icmp6SendEcho2(). iphlpapi: Implement Icmp6SendEcho2(). nsiproxy.sys: Don't try to check for original packet for ping socket. iphlpapi: Implement Icmp6CreateFile(). iphlpapi: Implement Icmp6ParseReplies(). iphlpapi: Only supply APC routine if no event in icmp_send_echo(). https://gitlab.winehq.org/wine/wine/-/merge_requests/8782

2 11

[PATCH 0/11] MR8782: iphlpapi: Implement ipv6 ping.
by Paul Gofman (＠gofman) Aug. 19, 2025

Aug. 19, 2025

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

2 12

Re: [PATCH v11 0/1] MR8778: sane.ds: Adding a window for loading and saving the scanner configuration
by Esme Povirk (＠madewokherd) Aug. 18, 2025

Aug. 18, 2025

Esme Povirk (@madewokherd) commented about dlls/sane.ds/cfg.c: > + } > + *last_slash = L'\0'; > + attr = GetFileAttributesW(path); > + if (attr == INVALID_FILE_ATTRIBUTES) > + { > + if (!CreateDirectoryW(path, NULL)) > + { > + DWORD err = GetLastError(); > + ERR("CreateDirectoryW returned error: %lu\n", err); > + return FALSE; > + } > + } > + return TRUE; > +} > + > +BOOL is_exist_file(WCHAR* path) I would suggest simply trying to create the file you will use when it's time to use it and handling any errors that come up. Checking first can cause race conditions: https://en.wikipedia.org/wiki/Time-of-check_to_time-of-use -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8778#note_113132

1 0

Re: [PATCH v2 0/1] MR7923: shell32: Change type of KNOWNFOLDERID FOLDERID_UserProgramFiles from...
by Alexandre Julliard (＠julliard) Aug. 18, 2025

Aug. 18, 2025

Superseded by !7490. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7923#note_113126

1 0