Closed Bug 20168 Opened 25 years ago Closed 25 years ago

[MLK] Leaking a condition variable

Categories

(Core :: JavaScript Engine, defect, P3)

Product:
Core
Component:
JavaScript Engine
Type:
defect

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: beard, Assigned: brendan)

References

(
URL
)

Details

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);
Status: NEW → ASSIGNED
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
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Fix checked in.  Thanks again,

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