memory/resource leak of arenaLock in plarena.c

RESOLVED FIXED in 4.2

Status

NSPR
NSPR
P2
normal
RESOLVED FIXED
17 years ago
17 years ago

People

(Reporter: Jeff Hostetler, Assigned: Wan-Teh Chang)

Tracking

({memory-leak})

4.1.2
memory-leak

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: Possible patch included in text)

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
in nsprpub/lib/ds/plarena.c there is a static lock
variable "arenaLock" that is never destroyed.

consider adding the following to the bottom of PL_ArenaFinish():

    if (arenaLock)
    {
	PR_DestroyLock(arenaLock);
	arenaLock = NULL;
    }

Comment 1

17 years ago
Marking NEW. Thanks for the patch.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: mlk, patch, review
Whiteboard: Possible patch included in text
(Assignee)

Comment 2

17 years ago
Created attachment 47365 [details] [diff] [review]
Proposed patch, based on the suggestion of jeff@NerdOne.com.
(Assignee)

Comment 3

17 years ago
Fix checked in on the tip of NSPR.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
OS: Windows 2000 → All
Priority: -- → P2
Hardware: PC → All
Resolution: --- → FIXED
Target Milestone: --- → 4.2
(Assignee)

Updated

16 years ago
Blocks: 129902
You need to log in before you can comment on or make changes to this bug.