[Patches] [ python-Patches-522961 ] Leak in Python/thread_nt.h

noreply@sourceforge.net noreply@sourceforge.net
Thu, 28 Feb 2002 13:35:54 -0800


Patches item #522961, was opened at 2002-02-26 08:58
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=522961&group_id=5470

Category: Windows
Group: Python 2.2.x
>Status: Closed
>Resolution: Accepted
Priority: 5
Submitted By: Gerald S. Williams (gsw_agere)
Assigned to: Tim Peters (tim_one)
Summary: Leak in Python/thread_nt.h

Initial Comment:
I submitted this into the bug tracker as [ #522621 ] 
Leak in thread_nt.h, although was advised to convert 
it into a proper patch request...

The NT threading module creates a dictionary to keep 
track of thread handles but never uses it. It doesn't 
maintain it properly, creating leaks. This patch 
removes the dictionary entirely.

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

>Comment By: Tim Peters (tim_one)
Date: 2002-02-28 16:35

Message:
Logged In: YES 
user_id=31435

Thanks!  This has been checked in,

Misc/ACKS; new revision: 1.160
Python/thread_nt.h; new revision: 2.22

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

Comment By: Tim Peters (tim_one)
Date: 2002-02-28 16:30

Message:
Logged In: YES 
user_id=31435

Deleted the older patch.

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

Comment By: Tim Peters (tim_one)
Date: 2002-02-28 16:13

Message:
Logged In: YES 
user_id=31435

Changed Category to Windows, assigned to me.

Gerald, there are two patches attached, uploaded within two 
minutes of each other.  Which one do you like better?

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

Comment By: Gerald S. Williams (gsw_agere)
Date: 2002-02-28 10:19

Message:
Logged In: YES 
user_id=329402

Note that this change removes <Python.h> dependencies from 
the PyThread code, which is a stated goal of that module.

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

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