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

[Bug 41799] winspool.drv: race can cause WINSPOOL_LoadSystemPrinters to fail

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.

[Bug 41799] winspool.drv: race can cause WINSPOOL_LoadSystemPrinters to fail

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.

[Bug 41799] winspool.drv: race can cause WINSPOOL_LoadSystemPrinters to fail

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.