Title: [104475] trunk/Source/WebCore
Revision
104475
Author
[email protected]
Date
2012-01-09 12:17:09 -0800 (Mon, 09 Jan 2012)

Log Message

[GStreamer] webkitwebsrc: pad template is leaked
https://bugs.webkit.org/show_bug.cgi?id=74224

Patch by Judy Hao <[email protected]> on 2012-01-09
Reviewed by Philippe Normand.

Use a GstPadTemplate smart pointer in webkit_web_src_init to
avoid a memory leak.

Fixing memory leaks. So, no new test case is introduced.

* platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
(webkit_web_src_init):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (104474 => 104475)


--- trunk/Source/WebCore/ChangeLog	2012-01-09 20:15:33 UTC (rev 104474)
+++ trunk/Source/WebCore/ChangeLog	2012-01-09 20:17:09 UTC (rev 104475)
@@ -1,3 +1,18 @@
+2012-01-09  Judy Hao  <[email protected]>
+
+        [GStreamer] webkitwebsrc: pad template is leaked
+        https://bugs.webkit.org/show_bug.cgi?id=74224
+
+        Reviewed by Philippe Normand.
+
+        Use a GstPadTemplate smart pointer in webkit_web_src_init to
+        avoid a memory leak.
+
+        Fixing memory leaks. So, no new test case is introduced.
+
+        * platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
+        (webkit_web_src_init):
+
 2012-01-09  Xianzhu Wang  <[email protected]>
 
         Avoid unnecessary TextureManager::reduceMemoryToLimit().

Modified: trunk/Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp (104474 => 104475)


--- trunk/Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp	2012-01-09 20:15:33 UTC (rev 104474)
+++ trunk/Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp	2012-01-09 20:17:09 UTC (rev 104475)
@@ -212,7 +212,7 @@
 
 static void webkit_web_src_init(WebKitWebSrc* src)
 {
-    GstPadTemplate* padTemplate = gst_static_pad_template_get(&srcTemplate);
+    GRefPtr<GstPadTemplate> padTemplate = adoptGRef(gst_static_pad_template_get(&srcTemplate));
     WebKitWebSrcPrivate* priv = WEBKIT_WEB_SRC_GET_PRIVATE(src);
 
     src->priv = priv;
@@ -232,7 +232,7 @@
 
 
     GRefPtr<GstPad> targetPad = adoptGRef(gst_element_get_static_pad(GST_ELEMENT(priv->appsrc), "src"));
-    priv->srcpad = gst_ghost_pad_new_from_template("src", targetPad.get(), padTemplate);
+    priv->srcpad = gst_ghost_pad_new_from_template("src", targetPad.get(), padTemplate.get());
 
     gst_element_add_pad(GST_ELEMENT(src), priv->srcpad);
     gst_pad_set_query_function(priv->srcpad, webKitWebSrcQuery);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to