[Bug 41799] New: winspool.drv: race can cause WINSPOOL_LoadSystemPrinters to fail
https://bugs.winehq.org/show_bug.cgi?id=41799 Bug ID: 41799 Summary: winspool.drv: race can cause WINSPOOL_LoadSystemPrinters to fail Product: Wine Version: 1.9.23 Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: awilfox(a)adelielinux.org Distribution: --- Created attachment 56232 --> https://bugs.winehq.org/attachment.cgi?id=56232 winspool.drv: Clear last error when creating mutex WINSPOOL_LoadSystemPrinters creates a unique mutex and then checks if GetLastError is ERROR_ALREADY_EXISTS. Running 1.9.23 with staging or unpatched, and current -HEAD, I'm seeing a race where sometimes a stale error is left and it returns ERROR_ALREADY_EXISTS due to another component. This causes printing to always fail on my machine, since CUPS_LoadPrinters is never called and therefore no printers exist. This one-line patch has allowed printing to work on my system and still guards against multiple initialisations. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=41799 Bruno Jesus <00cpxxx(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |wine.dev(a)web.de -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=41799 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch --- Comment #1 from Austin English <austinenglish(a)gmail.com> --- FYI, patches are not picked up from bugzilla. Patches should be wine-patches(a)winehq.org. See https://wiki.winehq.org/SubmittingPatches -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=41799 winetest(a)luukku.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |winetest(a)luukku.com --- Comment #2 from winetest(a)luukku.com --- Also you need to use real name when submiting patches. Could you resend? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
wine-bugs@winehq.org