[PSI] #33: Fix darwin_process.c:set_exe() error handling

5 views
Skip to first unread message

PSI - Python System Information

unread,
May 14, 2009, 9:29:13 AM5/14/09
to
#33: Fix darwin_process.c:set_exe() error handling
-------------------------+--------------------------------------------------
Reporter: flub | Owner:
Type: defect | Status: new
Priority: critical | Milestone: 0.3b1
Component: psi.process | Version:
Keywords: darwin |
-------------------------+--------------------------------------------------
The error handling in the set_exe() function of darwin_process.c can
result in a Python exception being set but the function returning normal
0. This results in internal consistency problems for Python (which might
not be caught in some optimised Python builds - not sure).

--
Ticket URL: <http://www.psychofx.com/psi/trac/ticket/33>
PSI - Python System Information <http://www.psychofx.com/psi/>
PSI - Python System Information

PSI - Python System Information

unread,
May 17, 2009, 7:09:09 PM5/17/09
to
#33: Fix darwin_process.c:set_exe() error handling
--------------------------+-------------------------------------------------
Reporter: flub | Owner:
Type: defect | Status: new
Priority: critical | Milestone: 0.3b1
Component: psi.process | Version:
Resolution: | Keywords: darwin
--------------------------+-------------------------------------------------

Comment(by flub):

The attached patch solves the issues I think, but it's completely untested
(not even been compiled) since I don't have a darwin box. So if someone
could test this and commit it (including unevitable fixes) that would be
great.

--
Ticket URL: <http://www.psychofx.com/psi/trac/ticket/33#comment:1>

PSI - Python System Information

unread,
May 19, 2009, 7:05:57 PM5/19/09
to
#33: Fix darwin_process.c:set_exe() error handling
--------------------------+-------------------------------------------------
Reporter: flub | Owner:
Type: defect | Status: closed
Priority: critical | Milestone: 0.3b1
Component: psi.process | Version:
Resolution: fixed | Keywords: darwin
--------------------------+-------------------------------------------------
Changes (by flub):

* status: new => closed
* resolution: => fixed


Comment:

Fixed by erickt in r330

--
Ticket URL: <http://www.psychofx.com/psi/trac/ticket/33#comment:2>

Reply all
Reply to author
Forward
0 new messages