[MLK] Leaking a condition variable

RESOLVED FIXED

Status

()

Core
JavaScript Engine
P3
normal
RESOLVED FIXED
18 years ago
16 years ago

People

(Reporter: Patrick C. Beard, Assigned: brendan)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

18 years ago
JS_NewRuntime() creates rt->stateChange, but JS_DestroyRuntime() doesn't destroy
it. Here's a patch:

Index: mozilla/js/src/jsapi.c
===================================================================
RCS file: /cvsroot/mozilla/js/src/jsapi.c,v
retrieving revision 3.34
diff -r3.34 jsapi.c
654a655
> 	JS_DESTROY_CONDVAR(rt->stateChange);
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 1

18 years ago
D'oh!  Swore I added that destroy when I added the condvar, but can't find it on
XULDOMJS_19991106_BRANCH.  Must have gotten lost due to fat fingers.  Thanks,

/be
(Assignee)

Updated

18 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Assignee)

Comment 2

18 years ago
Fix checked in.  Thanks again,

/be
You need to log in before you can comment on or make changes to this bug.