Sebastian Dröge [Thu, 28 Nov 2013 14:10:16 +0000 (15:10 +0100)]
h264parse: Fix splitting of multiple NALU per buffer
Conversion to byte-stream/nal crashes without that because the
baseparse frame of all NALUs is finished for the first NALU, then
used again for parsing the second NALU. Just that now the buffer
of the frame is already gone. Instead we create temporary frames
for every NALU.
Antonio Ospite [Thu, 31 Oct 2013 16:16:14 +0000 (17:16 +0100)]
mssdemux: fix setting codec_data when using WaveFormatEx
When using WaveFormatEx the actual codec private data is at the end of
the wav header structure, after the cbSize field, see for example the
notes at http://wiki.multimedia.cx/index.php?title=WAVEFORMATEX
The previous code was erroneously passing the whole WAVEFORMATEX
structure as codec_data.
The smoothstreaming specifications[1,2,3] always mention WAVEFORMATEX
and never smaller structures like WAVEFORMAT or PCMWAVEFORMAT, so the
buffer can assumed to be at least 18 bytes and always consuming the wav
header gives the (possibly empty) codec private data.
[1] http://web.archive.org/web/
20120907004742/http://www.iis.net/community/files/media/smoothspecs/%5BMS-SMTH%5D.pdf
[2] http://download.microsoft.com/download/B/0/B/
B0B199DB-41E6-400F-90CD-
C350D0C14A53/%5BMS-SSTR%5D.pdf
[3] http://download.microsoft.com/download/9/5/E/
95EF66AF-9026-4BB0-A41D-
A4F81802D92C/%5BMS-SSTR%5D.pdf
https://bugzilla.gnome.org/show_bug.cgi?id=699924
Antonio Ospite [Thu, 31 Oct 2013 14:58:27 +0000 (15:58 +0100)]
mssdemux: use mssdemux_debug debug category for gstmssmanifest.c too
Antonio Ospite [Wed, 16 Oct 2013 14:22:36 +0000 (16:22 +0200)]
mssdemux: fix coding style
Jan Schmidt [Mon, 25 Nov 2013 04:17:33 +0000 (15:17 +1100)]
dvdspu: Don't unmap an unreffed buffer
gst_dvd_spu_finish_spu_buf drops the reference to the current
buffer, so make sure to call gst_buffer_unmap before it.
Sebastian Dröge [Sat, 23 Nov 2013 18:11:42 +0000 (19:11 +0100)]
openexrdec: Consider the pixel aspect ratio stored in the file
Sebastian Dröge [Sat, 23 Nov 2013 17:37:00 +0000 (18:37 +0100)]
openexr: Add OpenEXR decoder element
This currently converts from ARGB64_F16 (16 bit float per component)
to ARGB64 by clipping. We should add support for the F16 format and
implement a conversion filter element that can apply gamma curves,
change exposure, etc.
Sebastian Dröge [Sat, 23 Nov 2013 10:29:08 +0000 (11:29 +0100)]
openexr: Add OpenEXR plugin
See http://www.openexr.com
Sebastian Dröge [Sat, 23 Nov 2013 10:35:35 +0000 (11:35 +0100)]
openjpeg: Fix names of debug categories
Mark Nauwelaerts [Sat, 23 Nov 2013 13:35:06 +0000 (14:35 +0100)]
dvdspu: pop new packet when invalid next command block
... to prevent staying stuck on current command block and busy looping.
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=712219
Mark Nauwelaerts [Sat, 23 Nov 2013 10:23:35 +0000 (11:23 +0100)]
mpeg4videoparse: accept VideoObjectLayer as start of frame and config
... which it deserves as much as VideoObject.
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=667564
Tim-Philipp Müller [Thu, 21 Nov 2013 15:54:32 +0000 (15:54 +0000)]
qtwrapper: g_memmove() is deprecated
Just use plain memmove(), g_memmove() is deprecated in
recent GLib versions.
https://bugzilla.gnome.org/show_bug.cgi?id=712811
Jan Schmidt [Wed, 20 Nov 2013 01:46:23 +0000 (12:46 +1100)]
dvdspu: Handle vobsub packets smaller than 4 bytes
In particular, handle the case of an empty packet with size 0 bytes.
Partially fixes: #712643
Olivier Crête [Wed, 20 Nov 2013 01:18:48 +0000 (20:18 -0500)]
srtpenc: Set the first session flag to false once one is create
Otherwise it always thinks it's the first session.
https://bugzilla.gnome.org/show_bug.cgi?id=711495
Olivier Crête [Wed, 20 Nov 2013 01:12:54 +0000 (20:12 -0500)]
srtpdec: Make sure that stream-id/caps/segment are sent before buffers
It may be possible that only one of the two sink pads is linked in that case,
the events need to be created from the other pad.
Olivier Crête [Wed, 20 Nov 2013 00:10:06 +0000 (19:10 -0500)]
srtpdec: Pass is_rtcp to setcaps function
Olivier Crête [Thu, 4 Apr 2013 21:59:01 +0000 (17:59 -0400)]
srtp: Implement SRTCP demultiplexing
Separate multiplexed RTP and RTCP if they arrive
on the rtp_sink pad.
Sebastian Dröge [Mon, 18 Nov 2013 23:38:33 +0000 (00:38 +0100)]
daala: Fix compilation
Thiago Santos [Thu, 14 Nov 2013 01:25:59 +0000 (22:25 -0300)]
mssdemux: do not try to deinit the downloadrate while it is used
This can cause an assertion or deadlocks.
Thiago Santos [Mon, 11 Nov 2013 19:35:35 +0000 (16:35 -0300)]
mssdemux: also consider /manifest
The spec says /Manifest, but /manifest is easily found on some
services, so handle that, too.
https://bugzilla.gnome.org/show_bug.cgi?id=710223
Thibault Saunier [Thu, 14 Nov 2013 12:02:35 +0000 (09:02 -0300)]
mpegtsmux: Fix compilation when debug is disabled
Reynaldo H. Verdejo Pinochet [Wed, 13 Nov 2013 21:59:48 +0000 (18:59 -0300)]
codecparsers: Add -lm needed by H.265 parser
Thibault Saunier [Wed, 13 Nov 2013 20:16:04 +0000 (17:16 -0300)]
pngparse: Proxy upstream framerate on srcpad if avalaible
If the user sets it himself with capsfilter, the information should be
kept and transmitted downstream.
Thibault Saunier [Tue, 12 Nov 2013 21:36:04 +0000 (18:36 -0300)]
mpegtsmux: Properly add subtitle descriptor for DVb subpicture subtitles
We were adding an empty descriptor for them and it was not possible to
actually use them on muxed streams.
Matthieu Bouron [Wed, 13 Nov 2013 14:28:30 +0000 (14:28 +0000)]
eglglessink: fix non-egl display context query
Mark Nauwelaerts [Mon, 11 Nov 2013 15:15:00 +0000 (16:15 +0100)]
mpegvideoparse: look beyond start code before leaping
In case more data than a start code alone is needed to decide whether
it ends a frame, arrange for more input data and decide when available.
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=711627
Matthieu Bouron [Mon, 11 Nov 2013 13:24:08 +0000 (13:24 +0000)]
avfvideosrc: declare missing instance variables
Fixes build on 32-bit system.
https://bugzilla.gnome.org/show_bug.cgi?id=711844
Mark Nauwelaerts [Sun, 10 Nov 2013 11:17:18 +0000 (12:17 +0100)]
mpegvideoparse: tweak extension handling
Mark Nauwelaerts [Sun, 10 Nov 2013 11:15:33 +0000 (12:15 +0100)]
mpgegdemux: do not use pushed buffer in debug message
Matej Knopp [Sat, 9 Nov 2013 12:32:21 +0000 (13:32 +0100)]
mpegvideoparse: check for same TSN only for interlaced streams (fixes unit test)
https://bugzilla.gnome.org/show_bug.cgi?id=711721
Sebastian Dröge [Sat, 9 Nov 2013 11:18:02 +0000 (12:18 +0100)]
audiomixer: Add test for the ORC code
Sebastian Dröge [Sat, 9 Nov 2013 08:43:56 +0000 (09:43 +0100)]
Revert "mpegvparse: Fix last start code position when input buffer is empty"
This reverts commit
7c23a6f23c185330d74ae4ccce79c3641a2fabac.
The fix was not complete, see discussion in
https://bugzilla.gnome.org/show_bug.cgi?id=711627
Sebastian Dröge [Fri, 8 Nov 2013 19:23:27 +0000 (20:23 +0100)]
mpegts: Add some more element-type annotations for g-i
Andoni Morales Alastruey [Thu, 7 Nov 2013 20:37:29 +0000 (21:37 +0100)]
coremediabuffer: keep an extra ref to the image and data buffer
Greg Rutz [Thu, 7 Nov 2013 17:57:22 +0000 (10:57 -0700)]
mpegvparse: Fix last start code position when input buffer is empty
When the input buffer is empty and we need more data to determine
whether or not to terminate the previous frame, the last start code
location needs to be set to 4 bytes before the the current position
(size of start_code is 32-bits)
https://bugzilla.gnome.org/show_bug.cgi?id=711627
Aurélien Zanelli [Thu, 7 Nov 2013 13:39:26 +0000 (14:39 +0100)]
codecparsers: Fix bad condition in h264 parser
https://bugzilla.gnome.org/show_bug.cgi?id=711615
Matthieu Bouron [Thu, 7 Nov 2013 12:29:48 +0000 (12:29 +0000)]
applemedia: coremediabuffer: use correct retain/release functions on CMSampleBufferRef buffer
Fixes a crash while releasing the sample buffer after the image
buffer it contains.
Also fixes a minor coding style error.
https://bugzilla.gnome.org/show_bug.cgi?id=711606
Matthieu Bouron [Thu, 7 Nov 2013 14:26:22 +0000 (14:26 +0000)]
qtkitvideosrc: lower rank to GST_RANK_SECONDARY
Makes avfvideosrc be prefered over qtkitvideosrc since it is deprecated
with OS X 10.9.
https://bugzilla.gnome.org/show_bug.cgi?id=711614
Matthieu Bouron [Thu, 7 Nov 2013 13:33:36 +0000 (13:33 +0000)]
avfvideosrc: set rank to GST_RANK_PRIMARY
Makes avfvideosrc to be used with autodetection.
https://bugzilla.gnome.org/show_bug.cgi?id=711614
Thiago Santos [Thu, 7 Nov 2013 15:47:51 +0000 (12:47 -0300)]
hlssink: do not lose ref to the multifilesink
It is needed to update location properties and it was being lost on
state changes, causing issues if the pipeline was to be reused
Thiago Santos [Thu, 7 Nov 2013 15:35:55 +0000 (12:35 -0300)]
hlssink: make sure it is handled as a sink
It only gets the sink flag set when it adds the multifilesink, that
happens in null->ready and it might be too late. Set the flag
explicitly on the constructor.
https://bugzilla.gnome.org/show_bug.cgi?id=711086
Andoni Morales Alastruey [Thu, 7 Nov 2013 12:05:08 +0000 (13:05 +0100)]
avfvideosrc: fix compiler warning in iOS
Andoni Morales Alastruey [Thu, 7 Nov 2013 11:29:34 +0000 (12:29 +0100)]
avfvideosrc: fix build in iOS with isEqualTo doesn't exists
Andoni Morales Alastruey [Thu, 7 Nov 2013 11:20:31 +0000 (12:20 +0100)]
avfvideosrc: fix build with older SDK not supporting AVDeviceFormat
AVDeviceFormat and AVFrameRateRange are available in iOS since 7.0
so we need a more dynamic approach to support compilation with older
SDK's. We use a NSObject to avoid referencing those types and key-value
coding or preformSelector to access properties.
Andoni Morales Alastruey [Thu, 7 Nov 2013 00:38:20 +0000 (01:38 +0100)]
avfvideosrc: fix checks for HAVE_IOS
Andoni Morales Alastruey [Thu, 7 Nov 2013 00:37:38 +0000 (01:37 +0100)]
avfvideosrc: include config.h
Andoni Morales Alastruey [Wed, 6 Nov 2013 17:53:25 +0000 (18:53 +0100)]
avfvideosrc: add more debug
Andoni Morales Alastruey [Wed, 6 Nov 2013 17:50:46 +0000 (18:50 +0100)]
avfvideosrc: fix compilation with gcc
The synthesize block needs to be moved to fix a compiler error
Andoni Morales Alastruey [Wed, 6 Nov 2013 17:49:35 +0000 (18:49 +0100)]
avfvideosrc: sort caps with the highest resolution first
When started without any caps filter we prefer the highest
supported resolution.
Matthieu Bouron [Mon, 4 Nov 2013 11:14:35 +0000 (11:14 +0000)]
avfvideosrc: support screen capture on OSX
https://bugzilla.gnome.org/show_bug.cgi?id=711432
Matthieu Bouron [Thu, 31 Oct 2013 13:05:40 +0000 (13:05 +0000)]
avfvideosrc: minor cosmetic
Matthieu Bouron [Thu, 31 Oct 2013 13:03:58 +0000 (13:03 +0000)]
avfvideosrc: use input device formats to set/get caps if available
https://bugzilla.gnome.org/show_bug.cgi?id=711211
Matthieu Bouron [Tue, 29 Oct 2013 15:37:16 +0000 (15:37 +0000)]
avfvideosrc: update caps if frame size has changed
On OSX, setting the pixel format on the output reset the capture device
to its native resolution, so we need to update the caps if the output
frame size has changed before a proper solution is found.
Matthieu Bouron [Tue, 29 Oct 2013 15:36:06 +0000 (15:36 +0000)]
avfvideosrc: check if low preset is available
Matthieu Bouron [Mon, 28 Oct 2013 18:22:13 +0000 (18:22 +0000)]
avfvideosrc: use a dedicated queue for AVFoundation calls
Replace the main queue with a dedicated queue for AVFoundation calls
so the execution on this queue won't block if the main queue
is not running.
Matthieu Bouron [Tue, 29 Oct 2013 17:33:11 +0000 (17:33 +0000)]
avfvideosrc: remove trailing space
Matthieu Bouron [Wed, 30 Oct 2013 14:51:50 +0000 (14:51 +0000)]
avfvideosrc: dispatch AVFoundation calls synchronously in the main queue
Matthieu Bouron [Wed, 30 Oct 2013 12:40:01 +0000 (12:40 +0000)]
applemedia: enable avfvideosrc if AVFoundation is present
Matthieu Bouron [Wed, 30 Oct 2013 12:39:24 +0000 (12:39 +0000)]
avfvideosrc: only enable 1920x1080 preset on iOS
Matthieu Bouron [Mon, 28 Oct 2013 11:20:27 +0000 (11:20 +0000)]
configure.ac: detect presence of AVFoundation
Matthieu Bouron [Mon, 28 Oct 2013 11:53:26 +0000 (11:53 +0000)]
applemedia: port avfvideosrc to 1.0
Sebastian Dröge [Thu, 7 Nov 2013 13:12:17 +0000 (14:12 +0100)]
audiomixer: Add unit test for handling unaligned buffers
That is, buffers that start or end in the middle of an output block.
Sebastian Dröge [Thu, 7 Nov 2013 13:12:00 +0000 (14:12 +0100)]
audiomixer: Fix EOS handling if we have some pending data
Sebastian Dröge [Thu, 7 Nov 2013 12:57:35 +0000 (13:57 +0100)]
audiomixer: Add test for the discont handling
Sebastian Dröge [Thu, 7 Nov 2013 12:55:32 +0000 (13:55 +0100)]
audiomixer: Refactor sync test to be more reusable
Sebastian Dröge [Thu, 7 Nov 2013 12:48:06 +0000 (13:48 +0100)]
audiomixer: Remove some racy-ness from the unit test
We might already be EOS when we start the mainloop, so only
set the pipeline to PLAYING from the main loop.
Sebastian Dröge [Thu, 7 Nov 2013 08:54:40 +0000 (09:54 +0100)]
codecparsers: Remove some unused functions
Thiago Santos [Thu, 7 Nov 2013 01:13:13 +0000 (22:13 -0300)]
camerabin: Make sure video-done is post only when file is closed
Force filesink to null before posting video-done to make sure the
file was closed.
Had to do it from a separate thread to avoid calling state_change from
a sync message handler.
https://bugzilla.gnome.org/show_bug.cgi?id=709373
Tim-Philipp Müller [Wed, 6 Nov 2013 23:11:14 +0000 (23:11 +0000)]
win32: add new codecparser API to .def file
Sreerenj Balachandran [Wed, 9 Oct 2013 07:54:14 +0000 (10:54 +0300)]
videoparsers: add H.265 HEVC video parser element
https://bugzilla.gnome.org/show_bug.cgi?id=708438
Sreerenj Balachandran [Tue, 1 Oct 2013 10:39:41 +0000 (13:39 +0300)]
codecparsers: add H.265 HEVC bitstream parser
https://bugzilla.gnome.org/show_bug.cgi?id=708438
Wim Taymans [Tue, 5 Nov 2013 11:43:44 +0000 (12:43 +0100)]
aiffparse: fix seeking in push mode
Sebastian Dröge [Wed, 6 Nov 2013 14:50:08 +0000 (15:50 +0100)]
audiomixer: Don't take channel mask in consideration in mono or stereo
This could cause negotiation to fail.
https://bugzilla.gnome.org/show_bug.cgi?id=708633
Sebastian Dröge [Wed, 6 Nov 2013 14:31:31 +0000 (15:31 +0100)]
audiomixer: Add test to the .gitignore file
Sebastian Dröge [Wed, 6 Nov 2013 14:18:58 +0000 (15:18 +0100)]
audiomixer: Add simply synchronization test
Sebastian Dröge [Wed, 6 Nov 2013 14:18:50 +0000 (15:18 +0100)]
audiomixer: Add new element based on adder that does synchronized audio mixing
Tim-Philipp Müller [Tue, 5 Nov 2013 11:19:37 +0000 (11:19 +0000)]
Automatic update of common submodule
From 865aa20 to dbedaa0
Sebastian Dröge [Mon, 4 Nov 2013 18:58:35 +0000 (19:58 +0100)]
eglglessink: Query upstream first to get a EGLDisplay
Sebastian Dröge [Mon, 4 Nov 2013 09:04:59 +0000 (10:04 +0100)]
applemedia-nonpublic: Link with gmodule for g_module_open() and others
Sebastian Dröge [Mon, 4 Nov 2013 09:04:42 +0000 (10:04 +0100)]
applemedia: Link with gmodule for g_module_open() and others
Sebastian Dröge [Fri, 1 Nov 2013 21:32:32 +0000 (22:32 +0100)]
configure: Always define AM_CONDITIONAL USE_EGLGLES
L. Sorin [Fri, 4 Oct 2013 10:48:10 +0000 (12:48 +0200)]
curl: curlsftpsink - new libcurl-based sink element for SFTP
Note: SFTP = SSH File Transfer Protocol
The sink acts as a client and uploads data to the SFTP server.
https://bugzilla.gnome.org/show_bug.cgi?id=709795
Reynaldo H. Verdejo Pinochet [Thu, 31 Oct 2013 20:36:25 +0000 (13:36 -0700)]
cdaudio: Remove cdaudio plugin
This element is being removed due to it been
considered obsolete and the functionaily it
provides no longer needed.
Details on the decision can be found here:
https://bugzilla.gnome.org/show_bug.cgi?id=711223
Fixes:
https://bugzilla.gnome.org/show_bug.cgi?id=711232
Sebastian Dröge [Fri, 1 Nov 2013 09:41:16 +0000 (10:41 +0100)]
yadif: Fix compiler warning caused by specifying const twice
yadif.c:49:24: warning: duplicate 'const' declaration specifier [-Wduplicate-decl-specifier]
DECLARE_ASM_CONST (16, const xmm_reg, pb_1) = {
^
yadif.c:33:50: note: expanded from macro 'DECLARE_ASM_CONST'
#define DECLARE_ASM_CONST(n,t,v) static const t __attribute__((used)) __attribute__ ((aligned (n))) v
^
yadif.c:52:24: warning: duplicate 'const' declaration specifier [-Wduplicate-decl-specifier]
DECLARE_ASM_CONST (16, const xmm_reg, pw_1) = {
^
yadif.c:33:50: note: expanded from macro 'DECLARE_ASM_CONST'
#define DECLARE_ASM_CONST(n,t,v) static const t __attribute__((used)) __attribute__ ((aligned (n)))
Sebastian Dröge [Fri, 1 Nov 2013 09:40:06 +0000 (10:40 +0100)]
yadif: Fix build with XCode
https://bugzilla.gnome.org/show_bug.cgi?id=710451
Sreerenj Balachandran [Mon, 28 Oct 2013 13:04:16 +0000 (15:04 +0200)]
h264parse: Remove the explicit PAR calculation based on VUI headers
The PAR calculation based on VUI headers is already doing by the
h264 codecparser and we does't need to duplicate those code.
https://bugzilla.gnome.org//show_bug.cgi?id=711009
Antonio Ospite [Thu, 17 Oct 2013 10:05:44 +0000 (12:05 +0200)]
rotate: Fix angle property help text
The help text of the angle property was probably copied from the circle
plugin, fix it to mention the rotation determined by the angle.
https://bugzilla.gnome.org/show_bug.cgi?id=710391
Reynaldo H. Verdejo Pinochet [Thu, 31 Oct 2013 01:48:08 +0000 (18:48 -0700)]
cdaudio: port to 1.X
https://bugzilla.gnome.org/show_bug.cgi?id=711223
Andoni Morales Alastruey [Wed, 30 Oct 2013 17:25:50 +0000 (18:25 +0100)]
androidmedia: fix rank for more software decoders
In Galaxy S4 the codecs list contains some OMX.SEC.foo.sw.dec
as software decoders
https://bugzilla.gnome.org/show_bug.cgi?id=711214
Antonio Ospite [Thu, 17 Oct 2013 10:53:31 +0000 (12:53 +0200)]
geometrictransform: Fix setting black background for AYUV buffers
When the frame buffer is AYUV writing all zeros does not set it to
black, in YUV colorspace 0x10 is the black level for luminance and 0x80
is the black level for chrominance.
Fix setting the background to black when the out_frame format is AYUV;
in all the other supported formats zeroing the data with memset is still
the right thing to do.
https://bugzilla.gnome.org/show_bug.cgi?id=710392
Matthieu Bouron [Wed, 23 Oct 2013 22:34:27 +0000 (23:34 +0100)]
applemedia-nonpublic: fix plugin name
https://bugzilla.gnome.org/show_bug.cgi?id=711135
Matthieu Bouron [Wed, 23 Oct 2013 16:22:16 +0000 (17:22 +0100)]
vtenc: use proper release callback type in CVPixelBufferCreateWithPlanarBytes
https://bugzilla.gnome.org/show_bug.cgi?id=711140
Matthieu Bouron [Fri, 20 Sep 2013 23:20:36 +0000 (00:20 +0100)]
avfvideosrc: fix truncated line
https://bugzilla.gnome.org/show_bug.cgi?id=710657
Andoni Morales Alastruey [Wed, 30 Oct 2013 15:20:56 +0000 (16:20 +0100)]
eglglessink: remove unused functions in the eagl backend
https://bugzilla.gnome.org/show_bug.cgi?id=711159
Andoni Morales Alastruey [Fri, 18 Oct 2013 09:54:12 +0000 (11:54 +0200)]
androidmedia: fix access to invalid buffers in the decoding loop
Flushing the decoder invalidates all buffers, so it should be done
after quiting the decoding loop. Otherwise we can jump into
"failed_release" and stop everything
https://bugzilla.gnome.org/show_bug.cgi?id=711156
Edward Hervey [Mon, 28 Oct 2013 08:33:47 +0000 (09:33 +0100)]
h264parse: Unify value setting
the initial par_n = par_d = 0; was always overwritten since the switch/case
handles all values
And remove the 0 case (it's the same handling as default)
Edward Hervey [Mon, 28 Oct 2013 08:33:16 +0000 (09:33 +0100)]
h264parse: Avoid ending up with undefined values
If caps aren't fixed, we'd end up with undefined values for format
and align.
Edward Hervey [Fri, 11 Oct 2013 09:23:16 +0000 (11:23 +0200)]
tsdemux: Demote GST_ERROR to GST_WARNING
discontinuities are not errors per-se and we handle them correctly.
Alex Ashley [Fri, 25 Oct 2013 10:11:30 +0000 (11:11 +0100)]
hlsdemux: fix memory leak in gst_hls_demux_get_next_fragment
This patch fixes three memory leaks in hlsdemux, one that occurs
during normal operation and two that occur during error conditions.
The gst_hls_demux_get_next_fragment function calls
gst_fragment_get_buffer which increments the reference count
on the buffer but gst_hls_demux_get_next_fragment never calls unref on
the buffer. This means that the reference count for each downloaded
fragment never gets to zero and so its memory is never released.
This patch adds a call to gst_buffer_unref after the flags have been
updated on the buffer.
There is a leak-on-error in gst_hls_demux_decrypt_fragment if it fails
to download the key file. If the key fails to download, null is
returned without doing an unref on the encrypted fragment. The
semantics of gst_hls_demux_decrypt_fragment is that it takes ownership
of the encrypted fragment and releases it before returning.
There is a leak-on-error in gst_hls_src_buf_to_utf8_playlist in the
unlikely event that the gst_buffer_map fails. In the "happy path"
operation of gst_hls_src_buf_to_utf8_playlist the buffer gets an unref
before the function returns, therefore the error condition must do the
same.
https://bugzilla.gnome.org/show_bug.cgi?id=710881
Matthieu Bouron [Wed, 23 Oct 2013 16:11:20 +0000 (17:11 +0100)]
mpegtsmux: fix uninitialized variable warning
https://bugzilla.gnome.org/show_bug.cgi?id=710759