This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: multiprocessing/win32: WindowsError: [Error 0] Success on Pipe()
Type: behavior Stage: resolved
Components: Extension Modules, Windows Versions: Python 2.6
process
Status: closed Resolution: rejected
Dependencies: Superseder:
Assigned To: Nosy List: BreamoreBoy, amyodov, asksol, davin, jnoller
Priority: normal Keywords:

Created on 2010-06-28 12:54 by amyodov, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Messages (6)
msg108824 - (view) Author: Alexander Myodov (amyodov) Date: 2010-06-28 12:54
I am using Python 2.6.5/win32, and working with multiprocessing module, doing that with Python interpreter embedded using Cython (if that may be related to the problem).

While creating a subprocess and a Pipe to communicate with it, I've got the following traceback (particulaly on the line "parent_conn, child_conn = Pipe()"):

Traceback (most recent call last):
  File "test_fork.py", line 24, in init test_fork (test_fork.c:810)
    vasia.main()
  File "Z:\multiprocessing_cython_w32\vasia.py", line 15, in main
    parent_conn, child_conn = Pipe()
  File "Z:\python-win32\2.6.5\Lib\multiprocessing\__init__.py", line 106, in Pipe
    return Pipe(duplex)
  File "Z:\python-win32\2.6.5\Lib\multiprocessing\connection.py", line 202, in Pipe
    h2, win32.PIPE_READMODE_MESSAGE, None, None
WindowsError: [Error 0] Success

Lines 202-204 in multiprocessing/connection.py contain the following: 

        win32.SetNamedPipeHandleState(                                                                                             
            h2, win32.PIPE_READMODE_MESSAGE, None, None                                                                            
            ) 

It seems to me that for some reason SetNamedPipeHandleState might be returning 0 even while it didn't fail actually; a quick check confirmed that it could be fixed by changing the above lines to the following:

        try:                                                                                                                       
            win32.SetNamedPipeHandleState(                                                                                         
                h2, win32.PIPE_READMODE_MESSAGE, None, None                                                                        
                )                                                                                                                  
        except WindowsError, e:                                                                                                    
(win32)                                                                                                       
            if e.args[0] != 0: # 0 is error code for SUCCESS                                                                       
                raise
msg108826 - (view) Author: Alexander Myodov (amyodov) Date: 2010-06-28 13:00
Sorry for formatting above, a copypaste issue.

The lines 202-204:
        win32.SetNamedPipeHandleState(
            h2, win32.PIPE_READMODE_MESSAGE, None, None
            )

The change that fixes the problem (at least for me):
        try:
            win32.SetNamedPipeHandleState(
                h2, win32.PIPE_READMODE_MESSAGE, None, None
                )
        except WindowsError, e:
            if e.args[0] != 0: # 0 is error code for SUCCESS
                raise
msg122005 - (view) Author: Brian Curtin (brian.curtin) * (Python committer) Date: 2010-11-21 22:32
Can you provide a test case for this?
msg137760 - (view) Author: Alexander Myodov (amyodov) Date: 2011-06-06 17:44
Sorry for being a little bit slow to respond...
No I was not able to come up with a testcase that could generate this problem in a reproducible way on any Windows box I had. This problem sometimes occured on various OS versions, being probably a Windows oof configuration problem rather than the problem of the Python code itself. Nevertheless, changing it as I described above didn't cause any adverse side effects, at least for me.
msg223151 - (view) Author: Mark Lawrence (BreamoreBoy) * Date: 2014-07-15 21:34
I'm not sure how we take this forward as the code was changed via #11750, can somebody please advise.
msg236154 - (view) Author: Davin Potts (davin) * (Python committer) Date: 2015-02-17 22:12
Though the code may have changed a bit in the meantime (Issue11750 in particular), the calls to _winapi.SetNamedPipeHandleState in Lib/multiprocessing/connection.py are still present and largely the same as when this issue was first opened.

The implementation of _winapi.SetNamedPipeHandleState still has the potential to raise a WindowsError with its message (i.e. e.args[0]) set to whatever the Windows GetLastError() function returns.  My reading of the MSDN docs and the code in Modules/_winapi.c is as follows:
1. A WindowsError exception is raised only if the Windows function SetNamedPipeHandleState returns 0.
2. The Windows function SetNamedPipeHandleState only returns 0 if it failed.
3. When that function fails, the Windows function GetLastError is expected to return a non-zero value to provide insight on the nature of the failure.
4. MSDN docs suggest that some functions may call SetLastError with 0 to indicate success (seen when subsequently calling GetLastError).  This creates a bit of a conundrum given what the OP observed in the provided traceback:  the Windows function SetNamedPipeHandleState failed yet the call to GetLastError returned a 0.

Possibly the MSDN docs are incomplete on this specific matter and/or there could be other environmental factors on the system(s) where this issue has been observed not to mention other wrinkles from the OP's reported use of Cython to embed the Python interpreter to trigger the issue.  A google search for other situations triggering this same behavior did turn up mentions of encountering it inside the Wine environment running on Ubuntu -- debugging Wine's re-implementation of Windows APIs is certainly out-of-scope here.  It is possible that this issue is caused by environmental issues and may not be possible to provoke in a supported standard Windows system environment.  Without a test case or some other way to provoke or reproduce the issue, there is little capability to pursue this further.

The OP kindly followed up several years ago to say that he could not find a way to reproduce the issue reliably.  Given that, the above review of the code, and the search results from looking for other instances of this sort of issue, I am going ahead with closing this issue.
History
Date User Action Args
2022-04-11 14:57:02adminsetgithub: 53345
2015-02-17 22:12:45davinsetstatus: open -> closed

nosy: + davin
messages: + msg236154

resolution: rejected
stage: test needed -> resolved
2014-07-15 21:56:36brian.curtinsetnosy: - brian.curtin
2014-07-15 21:34:56BreamoreBoysetnosy: + BreamoreBoy
messages: + msg223151
2011-06-06 17:44:11amyodovsetmessages: + msg137760
2010-11-21 22:32:32brian.curtinsetnosy: + asksol, brian.curtin

messages: + msg122005
stage: test needed
2010-06-28 13:55:15r.david.murraysetnosy: + jnoller
2010-06-28 13:00:32amyodovsetmessages: + msg108826
title: multiprocessing/win32: -> multiprocessing/win32: WindowsError: [Error 0] Success on Pipe()
2010-06-28 12:54:58amyodovcreate