Edgewall Software
Modify

Opened 21 years ago

Closed 21 years ago

Last modified 21 years ago

#249 closed defect (fixed)

possible svn_pool leak in trac.real_main

Reported by: anonymous Owned by: Jonas Borgström
Priority: normal Milestone: 0.7
Component: general Version: 0.6.1
Severity: major Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

At the end of trac.real_main the svn_pool is destroyed and the repo connection is shut down. If module.run throws an exception, isn't this going to get skipped? I don't know Python that well, so maybe it gets cleaned up as the variables are popped of the call stack or something.

Attachments (0)

Change History (3)

comment:1 by Jonas Borgström, 21 years ago

Milestone: 0.6.2
Status: newassigned
Version: devel0.6.1

Yeah, this is something that's been fixed in trunk. using a try: finally: block. I guess this change should be backported into 0.6-stable.

comment:2 by Jonas Borgström, 21 years ago

Resolution: fixed
Status: assignedclosed

Fixed in [420].

comment:3 by daniel, 21 years ago

Milestone: 0.6.20.7

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.