[Python-bugs-list] [ python-Bugs-506611 ] sys.setprofile() coredumps

[noreply@sourceforge.net]

Bugs item #506611, was opened at 2002-01-21 15:17
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=105470&aid=506611&group_id=5470

Category: Python Interpreter Core
Group: Python 2.2.1 candidate
>Status: Closed
Resolution: Accepted
Priority: 7
Submitted By: Tim Peters (tim_one)
Assigned to: Neal Norwitz (nnorwitz)
Summary: sys.setprofile()  coredumps

Initial Comment:
Verified in current CVS (also on Windows, but doubt 
the platform matters).  It's not the call proper that 
coredumps, but an eval loop's later attempt to use a 
resulting NULL pointer.  From c.l.py:

"""
From: rihad
Sent: Monday, January 21, 2002 1:19 PM
Subject: minor Python bugs

Calling sys.setprofile() (with no args) makes the 
interpreter crash, and calling sys.settrace() with no 
args succeeds, even though it is documented to take 1 
arg (tracefunc).

And yes, my version is '2.2 (#28, Dec 21 2001, 
12:21:22) [MSC 32 bit (Intel)]'
"""

----------------------------------------------------------------------

>Comment By: Neal Norwitz (nnorwitz)
Date: 2002-03-03 10:17

Message:
Logged In: YES 
user_id=33168

Updated both setprofile() and settrace().
Checked in as sysmodule.c 2.101 & 2.98.6.3,
test_profilehooks.py 1.7 & 1.6.10.1, test_scope.py 1.25 and
1.24.4.1.



----------------------------------------------------------------------

Comment By: Fred L. Drake, Jr. (fdrake)
Date: 2002-03-02 11:19

Message:
Logged In: YES 
user_id=3066

This is the right approach, please check this in.  An
equivalent change to sys.settrace() would be appropriate as
well. 

The changes should be committed on both the trucnk and
release22-branch.

Thanks!

----------------------------------------------------------------------

Comment By: Michael Hudson (mwh)
Date: 2002-03-01 09:04

Message:
Logged In: YES 
user_id=6656

Certainly.  Setting group so this doesn't get forgotten.

----------------------------------------------------------------------

Comment By: Neal Norwitz (nnorwitz)
Date: 2002-03-01 08:18

Message:
Logged In: YES 
user_id=33168

We should try to fix the core dump for 2.2.1.


----------------------------------------------------------------------

Comment By: Neal Norwitz (nnorwitz)
Date: 2002-03-01 08:17

Message:
Logged In: YES 
user_id=33168

We should try to fix the core dump for 2.2.1.


----------------------------------------------------------------------

Comment By: Neal Norwitz (nnorwitz)
Date: 2002-01-24 16:46

Message:
Logged In: YES 
user_id=33168

What is the appropriate behaviour for settrace()?
Currently, it is the same as settrace(None) which
disables the trace function.

So is the doc correct or is the code correct?

I can supply another patch to doc/code for settrace()
depending on which needs to be changed.

----------------------------------------------------------------------

Comment By: Neal Norwitz (nnorwitz)
Date: 2002-01-24 15:24

Message:
Logged In: YES 
user_id=33168

I didn't read the part that settrace was a problem.
I can take a look at the settrace problem later.

----------------------------------------------------------------------

Comment By: Martin v. Löwis (loewis)
Date: 2002-01-24 15:02

Message:
Logged In: YES 
user_id=21627

It appears that this patches deals only with the setprofile
problem, not with the settrace problem. Is that intentional?

----------------------------------------------------------------------

Comment By: Neal Norwitz (nnorwitz)
Date: 2002-01-21 17:39

Message:
Logged In: YES 
user_id=33168

Attached is a patch which fixes the problem.
The patch includes a test.

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=105470&aid=506611&group_id=5470