Title: [124333] trunk/Source/WebCore
Revision
124333
Author
[email protected]
Date
2012-08-01 07:33:32 -0700 (Wed, 01 Aug 2012)

Log Message

[GTK] Memory leak found while running webaudio tests
https://bugs.webkit.org/show_bug.cgi?id=92861

Patch by Sudarsana Nagineni <[email protected]> on 2012-08-01
Reviewed by Philippe Normand.

Fix a memory leak in decodeAudioForBusCreation by adopting
an allocation of GInputStream.

No new tests. No change in behavior.

* platform/audio/gstreamer/AudioFileReaderGStreamer.cpp:
(WebCore::AudioFileReader::decodeAudioForBusCreation):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (124332 => 124333)


--- trunk/Source/WebCore/ChangeLog	2012-08-01 13:55:07 UTC (rev 124332)
+++ trunk/Source/WebCore/ChangeLog	2012-08-01 14:33:32 UTC (rev 124333)
@@ -1,3 +1,18 @@
+2012-08-01  Sudarsana Nagineni  <[email protected]>
+
+        [GTK] Memory leak found while running webaudio tests
+        https://bugs.webkit.org/show_bug.cgi?id=92861
+
+        Reviewed by Philippe Normand.
+
+        Fix a memory leak in decodeAudioForBusCreation by adopting
+        an allocation of GInputStream.
+
+        No new tests. No change in behavior.
+
+        * platform/audio/gstreamer/AudioFileReaderGStreamer.cpp:
+        (WebCore::AudioFileReader::decodeAudioForBusCreation):
+
 2012-08-01  Alexei Filippov  <[email protected]>
 
         Web Inspector: count DOM storage cache memory for native snapshot

Modified: trunk/Source/WebCore/platform/audio/gstreamer/AudioFileReaderGStreamer.cpp (124332 => 124333)


--- trunk/Source/WebCore/platform/audio/gstreamer/AudioFileReaderGStreamer.cpp	2012-08-01 13:55:07 UTC (rev 124332)
+++ trunk/Source/WebCore/platform/audio/gstreamer/AudioFileReaderGStreamer.cpp	2012-08-01 14:33:32 UTC (rev 124333)
@@ -305,7 +305,7 @@
     if (m_data) {
         ASSERT(m_dataSize);
         source = gst_element_factory_make("giostreamsrc", 0);
-        GRefPtr<GInputStream> memoryStream = g_memory_input_stream_new_from_data(m_data, m_dataSize, 0);
+        GRefPtr<GInputStream> memoryStream = adoptGRef(g_memory_input_stream_new_from_data(m_data, m_dataSize, 0));
         g_object_set(source, "stream", memoryStream.get(), NULL);
     } else {
         source = gst_element_factory_make("filesrc", 0);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to