Bug 104901 - [GTK] Don't leak GStaticRecMutex
: [GTK] Don't leak GStaticRecMutex
Status: RESOLVED FIXED
Product: WebKit
Classification: Unclassified
Component: WebKit Gtk
: 528+ (Nightly build)
: Unspecified Unspecified
: P2 Minor
Assigned To: Nobody
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-12-13 04:05 PST by Alberto Garcia
Modified: 2012-12-13 06:03 PST (History)
6 users (show)

See Also:


Attachments
Patch (2.56 KB, patch)
2012-12-13 04:29 PST, Alberto Garcia
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alberto Garcia 2012-12-13 04:05:05 PST
There's a GStaticRecMutex allocated in webkit_web_audio_src_init(), but it's never freed.
Comment 1 Alberto Garcia 2012-12-13 04:29:06 PST
Created attachment 179249 [details]
Patch
Comment 2 Martin Robinson 2012-12-13 04:31:31 PST
Comment on attachment 179249 [details]
Patch

Nice catch.
Comment 3 WebKit Review Bot 2012-12-13 05:12:47 PST
Comment on attachment 179249 [details]
Patch

Attachment 179249 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/15314239

New failing tests:
inspector-protocol/debugger-terminate-dedicated-worker-while-paused.html
Comment 4 WebKit Review Bot 2012-12-13 06:03:14 PST
Comment on attachment 179249 [details]
Patch

Clearing flags on attachment: 179249

Committed r137587: <http://trac.webkit.org/changeset/137587>
Comment 5 WebKit Review Bot 2012-12-13 06:03:18 PST
All reviewed patches have been landed.  Closing bug.