- wine-gitlab - winehq.org

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

19 Aug '25

Esme Povirk (@madewokherd) commented about dlls/sane.ds/cfg.c: > + BOOL found; > + HANDLE hFile = CreateFileW( > + path, > + GENERIC_READ, > + 0, NULL, OPEN_EXISTING, > + FILE_ATTRIBUTE_NORMAL, > + NULL > + ); > + > + if (hFile == INVALID_HANDLE_VALUE) > + { > + ERR("CreateFileW returned error: %lu\n", GetLastError()); > + return FALSE; > + } > + > + sprintf(search, "%s=", name); This `sprintf` is done twice. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8778#note_113167

1 0

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

19 Aug '25

Esme Povirk (@madewokherd) commented about dlls/sane.ds/cfg.c: > + while (line && count < 1000) > + { > + if (!found && strstr(line, search) == line) > + { > + lines[count++] = buffer; > + found = TRUE; > + } > + else > + { > + lines[count++] = strdup(line); > + } > + line = strtok_s(NULL, "\n", &context); > + } > + if (!found) > + { > + lines[count++] = buffer; This could write past the end of `lines`. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8778#note_113166

1 0

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

19 Aug '25

Esme Povirk (@madewokherd) commented about dlls/sane.ds/cfg.c: > + CHAR* context = NULL; > + size_t new_size = 0; > + CHAR* line; > + > + line = strtok_s(content, "\n", &context); > + > + while (line && count < 1000) > + { > + if (!found && strstr(line, search) == line) > + { > + lines[count++] = buffer; > + found = TRUE; > + } > + else > + { > + lines[count++] = strdup(line); I don't see a reason to make an additional allocation for each line. The original pointer should remain valid until `content` is freed. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8778#note_113165

1 0

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

19 Aug '25

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) 19 Aug '25

19 Aug '25

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) 19 Aug '25

19 Aug '25

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 (＠insn) 19 Aug '25

19 Aug '25

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

3 6

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

19 Aug '25

-- 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) 19 Aug '25

19 Aug '25

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) 19 Aug '25

19 Aug '25

-- 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

2025

2024

2023

2022

wine-gitlab