Commit 3a6900df authored by Sebastian Dröge's avatar Sebastian Dröge 🍵

Release 1.11.1

parent 1c60b00a
=== release 1.11.1 ===
2017-01-12 Sebastian Dröge <slomo@coaxion.net>
* configure.ac:
releasing 1.11.1
2017-01-12 14:36:22 +0200 Sebastian Dröge <sebastian@centricular.com>
* po/el.po:
* po/hr.po:
* po/id.po:
* po/zh_CN.po:
po: Update translations
2017-01-11 17:53:32 -0800 Andre McCurdy <armccurdy@gmail.com>
* gst/isomp4/qtdemux.c:
qtdemux: free seqh after calling qtdemux_parse_svq3_stsd_data()
The seqh buffer allocated in qtdemux_parse_svq3_stsd_data() needs to
be freed by the caller after use.
https://bugzilla.gnome.org/show_bug.cgi?id=777157
Signed-off-by: Andre McCurdy <armccurdy@gmail.com>
2017-01-10 16:01:35 +0100 Edward Hervey <edward@centricular.com>
* gst/isomp4/fourcc.h:
* gst/isomp4/qtdemux.c:
isomp4: Don't spam debug log with knonw/padding atoms
Only output WARNING messages for atoms we don't know how to handle
instead of for padding/known atoms we don't need to do any processing
on
https://bugzilla.gnome.org/show_bug.cgi?id=777095
2017-01-09 19:05:10 +0000 Tim-Philipp Müller <tim@centricular.com>
* gst/rtp/gstrtph263depay.c:
* gst/rtp/gstrtpsbcdepay.c:
* gst/rtpmanager/rtpjitterbuffer.c:
* gst/rtsp/gstrtspsrc.c:
* sys/v4l2/gstv4l2bufferpool.c:
Fix indentation
2017-01-09 19:04:04 +0000 Tim-Philipp Müller <tim@centricular.com>
* tests/check/elements/rtpjitterbuffer.c:
tests: rtpjitterbuffer: fix compiler warning due to c99-ism
rtpjitterbuffer.c:592:3: error: ‘for’ loop initial declarations are only allowed in C99 mode
2016-11-11 14:31:03 +1100 Matthew Waters <matthew@centricular.com>
* gst/autodetect/gstautodetect.c:
autodetect: bring the element state down after success
Otherwise some messages that are emitted by the element on NULL->READY
will not reach the application.
https://bugzilla.gnome.org/show_bug.cgi?id=764947
2017-01-08 01:13:32 +1100 Jan Schmidt <jan@centricular.com>
* gst/isomp4/atoms.c:
* gst/isomp4/atoms.h:
* gst/isomp4/gstqtmux.c:
qtmux: Write tfdt atom into fragmented files.
The DASH spec requires that tfdt atoms be present, so
write one out. ISO/IEC 23009-1:2014 6.3.4.2
https://bugzilla.gnome.org/show_bug.cgi?id=708221
2017-01-07 23:55:42 +1100 Jan Schmidt <jan@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Don't reset output timestamps when no tfdt
If a fragmented stream doesn't have a tfdt, don't
reset the output timestamps at each fragment boundary
by erroneously using the default value of 0. Introduced
by commit 69fc48
https://bugzilla.gnome.org/show_bug.cgi?id=754230
2016-12-16 16:51:48 -0300 Thibault Saunier <thibault.saunier@osg.samsung.com>
* ext/vpx/meson.build:
* gst/equalizer/meson.build:
* gst/isomp4/meson.build:
* meson.build:
meson: Install presets files
2017-01-03 10:12:30 +0530 Garima Gaur <garima.g@samsung.com>
* gst/avi/gstavidemux.c:
avidemux: fix some caps leaks
https://bugzilla.gnome.org//show_bug.cgi?id=776789
2016-12-22 17:34:08 +0200 Vivia Nikolaidou <vivia@ahiru.eu>
* gst/multifile/gstsplitmuxsink.c:
splitmuxsink: Return a bin with a "location" property as a sink
Splitmuxsink might be called with a custom bin as a sink. If it has a
"location" property, it can be used.
2016-11-18 22:42:18 +1100 Jan Schmidt <jan@centricular.com>
* gst/multifile/gstsplitmuxsink.c:
* gst/multifile/gstsplitmuxsink.h:
splitmux: Rewrite buffer collection and scheduling
Majorly change the way that splitmuxsink collects
incoming data and sends it to the output, so that it
makes all decisions about when / where to split files
on the input side.
Use separate queues for each stream, so they can be
grown individually and kept as small as possible.
This removes raciness I observed where sometimes
some data would end up put in a different output file
over multiple runs with the same input.
Also fixes hangs with input queues getting full
and causing muxing to stall out.
2016-11-17 23:40:27 +1100 Jan Schmidt <jan@centricular.com>
* gst/multifile/gstsplitmuxsink.c:
* gst/multifile/gstsplitmuxsink.h:
* tests/check/elements/splitmux.c:
splitmuxsink: Add format-location-full signal
Add a new signal for formatting the filename, which receives
a GstSample containing the first buffer from the reference
stream that will be muxed into that file.
Useful for creating filenames that are based on the
running time or other attributes of the buffer.
To make it work, opening of files and setting filenames is
now deferred until there is some data to write to it,
which also requires some changes to how async state changes
and gap events are handled.
2016-12-31 01:54:01 +1100 Jan Schmidt <jan@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Always snap to the start of the keyframe
When performing a key-unit seek, always snap to the start ts
of the keyframe buffer we landed on so that the keyframe is
entirely within the resulting outgoing segment. That seems
the most sensible result, since the user requested snapping
to the keyframe position.
2016-12-31 01:48:04 +1100 Jan Schmidt <jan@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Omit cslg_shift when snapping seeks
Segments times and seek requests are stored and handled
in raw 'PTS' time, without the cslg_shift - which only applies
to outgoing samples. Omit the cslg_shift portion when
extracting PTS to compare for internal seek snaps.
If the cslg_shift is included, then keyframe+snap-before seeks
generate a segment start/stop time that already includes the
cslg_shift, and it's then added a 2nd time, causing the
first buffer(s) to have timestamps that are out of segment.
2016-12-30 22:31:38 +1100 Jan Schmidt <jan@centricular.com>
* gst/isomp4/atoms.c:
qtmux: Remove bogus check in atom_stsc_add_new_entry()
Remove an old check from atom_stsc_add_new_entry() that
extends the last entry in the STSC if the samples per chunk
matches, as the new interleave merging logic requires that
the final entry by updateable. There's already code
below which simply merges the final entry into the previous
one when needed, so rely on that instead.
Fixes asserts like:
ERROR:atoms.c:2940:atom_stsc_update_entry: assertion failed:
(atom_array_index (&stsc->entries, len - 1).first_chunk == first_chunk)
2016-04-24 21:38:51 +0900 Seungha Yang <sh.yang@lge.com>
* gst/isomp4/qtdemux.c:
qtdemux: Fix key_time in gst_qtdemux_adjust_seek()
time in segment should be PTS based (not DTS).
https://bugzilla.gnome.org/show_bug.cgi?id=765498
2016-12-28 22:49:27 +1100 Jan Schmidt <jan@centricular.com>
* gst/multifile/gstsplitmuxpartreader.c:
* gst/multifile/gstsplitmuxpartreader.h:
* gst/multifile/gstsplitmuxsrc.c:
splitmuxsrc: Pass seek flags when activating.
Pass all seek flags when activating a part
based on a seek, so that SNAP flags are preserved.
2016-11-26 01:13:19 +1100 Jan Schmidt <jan@centricular.com>
* gst/multifile/gstsplitmuxpartreader.c:
splitmux: Fix a small race in the splitmuxsrc
Make sure the state of the parser is set to
collecting streams before chaining up to the
parent change_state() method, to close a
small window that can cause playback to
never commence.
2017-01-02 15:06:33 +0100 Edward Hervey <edward@centricular.com>
* tests/check/elements/amrparse.c:
check: Remove dead code
2016-12-31 09:52:25 +0000 Tim-Philipp Müller <tim@centricular.com>
* gst/multifile/gstmultifilesink.c:
* gst/multifile/gstmultifilesink.h:
multifilesink: refactor max_files handling a bit
Use GQueue instead of a GSList so we don't have to traverse
the whole list to append something every time. And it also
keeps track of the number of items in it for us.
Add a function to add filenames to the list of old files and
use it in more places, so that memory doesn't build up in
other modes either if no max_files limit is specified.
https://bugzilla.gnome.org/show_bug.cgi?id=766991
2016-05-29 17:21:47 +0100 Ursula Maplehurst <ursula@kangatronix.co.uk>
* gst/multifile/gstmultifilesink.c:
multifilesink: don't leak memory when no max-files limit is set
Technically we weren't leaking the memory, just storing it internally
and never using it until the element is freed. But we'd still use more
and more memory over time, so this is not good over longer periods
of time. Only keep track of files if there's actually a limit set,
so that we will prune the list from time to time.
https://bugzilla.gnome.org/show_bug.cgi?id=766991
2016-12-29 12:39:20 +0100 Mark Nauwelaerts <mnauw@users.sourceforge.net>
* gst/matroska/matroska-demux.c:
matroskademux: adjust segment stop for KEY_UNIT negative rate seeking
2016-12-29 12:25:35 +0100 Mark Nauwelaerts <mnauw@users.sourceforge.net>
* gst/isomp4/qtdemux.c:
qtdemux: implement pull mode SNAP flag seeking
2016-12-29 11:26:33 +0100 Mark Nauwelaerts <mnauw@users.sourceforge.net>
* gst/avi/gstavidemux.c:
avidemux: tweak KEY_UNIT SNAP seek handling
Previously, seeking to position y where y is (strictly) within a keyframe
would seek to that keyframe both with SNAP_BEFORE and SNAP_AFTER,
where the latter is now adjusted to really snap to the next keyframe.
2016-12-28 13:23:11 +0100 Mark Nauwelaerts <mnauw@users.sourceforge.net>
* gst/avi/gstavidemux.c:
avidemux: correctly perform pull mode KEY_UNIT seeking
Rather amazingly (and equally unnoticed), keyunit seeking resulted in segments
where start != time (which is bogus for simple avi timeline). So, properly
adjust the segment (start) rather than fiddling with segment time (only).
2016-12-28 13:04:54 +0100 Mark Nauwelaerts <mnauw@users.sourceforge.net>
* gst/avi/gstavidemux.c:
avidemux: restore considering of pull mode KEY_UNIT seeking
... by using the original seek event's flags rather than the corresponding
segment flags, which do not have such counterpart flags (and
do no longer have them covertly sneaking in nowadays).
2015-05-08 12:44:01 +0200 Nicola Murino <nicola.murino@gmail.com>
* gst/matroska/matroska-mux.c:
matroskamux: only drop actual streamheader buffers with xiph codecs
With Xiph codecs the stream header buffers are both in the caps and are
usually also at the beginning of each input stream, but it's perfectly
possible that the input stream does not have the stream header buffers
inline in the data. Matroskamux would drop the first N buffers assuming
they're stream headers, but this meant it would drop actual payload data
when the stream didn't contain the stream headers inline. Fix this by
only dropping leading buffers if they're flagged as stream headers. This
fixes issues with streams that are being tapped into after streaming
has started.
https://bugzilla.gnome.org/show_bug.cgi?id=749098
2016-12-21 17:43:58 +0100 Nicola Murino <nicola.murino@gmail.com>
* tests/check/elements/matroskamux.c:
matroskamux: adjust unit test to modified behaviour
Now matroskamux mark all packets of audio-only streams as keyframes so
in test_block_group after pushing the test audio data 4 buffers are produced
and not more 2. The last buffer is the original data and must match with what
pushed. The remaining ones are matroskamux headers
https://bugzilla.gnome.org/show_bug.cgi?id=754696
2016-05-30 01:15:31 +0200 Nicola Murino <nicola.murino@gmail.com>
* gst/matroska/matroska-mux.c:
matroskamux: mark all packets of audio-only streams as keyframes
This helps with streaming audio-only streams via multifdsink,
tcpserversink and such.
https://bugzilla.gnome.org/show_bug.cgi?id=754696
2015-03-28 18:15:36 +0100 Nicola Murino <nicola.murino@gmail.com>
* gst/matroska/matroska-mux.c:
matroskamux: add G722 audio support
https://bugzilla.gnome.org/show_bug.cgi?id=746574
2016-12-13 11:11:07 +0900 Wonchul Lee <wonchul.lee@collabora.com>
* gst/udp/gstudpsrc.c:
updsrc: Add to join multiple multicast interfaces
https://bugzilla.gnome.org/show_bug.cgi?id=776030
2015-03-25 13:51:30 +0000 Tim-Philipp Müller <tim@centricular.com>
* gst/rtp/gstrtpklvdepay.c:
rtpklvdepay: add the SPARSE flag to the outgoing stream-start event
2016-12-14 14:37:45 -0800 Reynaldo H. Verdejo Pinochet <reynaldo@osg.samsung.com>
* gst/rtpmanager/gstrtpbin.c:
* gst/rtpmanager/gstrtpsession.c:
rtpmanager: place content before Since-version API marker
Avoids confusing the parser
2016-12-14 14:16:53 -0800 Reynaldo H. Verdejo Pinochet <reynaldo@osg.samsung.com>
* ext/shout2/gstshout2.c:
shout2: fix 404 in package origin
2016-12-14 21:45:15 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Check if we have enough data available when parsing edit lists
Also consume the data entry by entry to get complicated indexing out of
the code.
https://bugzilla.gnome.org/show_bug.cgi?id=776107
2016-12-14 19:15:03 +0100 Víctor Manuel Jáquez Leal <vjaquez@igalia.com>
* sys/v4l2/gstv4l2object.c:
v4l2object: Don't check size in a non-list value
After commit 1ea9735a I see these error while using the webcam
integrated in my laptop:
GStreamer-CRITICAL **: gst_value_list_get_size: assertion 'GST_VALUE_HOLDS_LIST (value)' failed
The issue is gst_v4l2src_value_simplify() was doing its job of
generating a single value, rather than the original list. That why,
when getting the list size, a critical warning was raised.
This patch takes advantage of the compiler optimizations to verify
first if the list was simplified, thus use it directly, otherwise,
if it is a list, verify its size.
https://bugzilla.gnome.org/show_bug.cgi?id=776106
2016-12-14 10:39:12 +0100 Havard Graff <havard.graff@gmail.com>
* tests/check/elements/rtpjitterbuffer.c:
tests/jitterbuffer: Major refactoring and cleanups
* Changed PCMU->TEST for common macros
* Changed verify-functions (lost & rtx) into macros.
* Remove option to add marker-bit for test-buffers (not used anywhere)
* Add new push_test_buffer function that makes sure there are correlation
between dts and the time on the clock. (classic test-mistake)
* Established a generic starting-point for tests with the
construct_deterministic_initial_state function and use it where
applicable, which removes lots of "boilerplate" everywhere.
* Add basic lost-event test
* Remove as much "magic constants" as possible.
* Remove 3 tests that no longer are testing anything that others don't,
and was completely unmaintainable.
* Remove unnecessary use of the testclock
* Verify each test is testing what it actually says it does (and modify
where it doesn't)
In general, make the tests much smaller, better, more maintainable and
readable.
https://bugzilla.gnome.org/show_bug.cgi?id=774409
2016-12-14 09:54:11 +0000 Tim-Philipp Müller <tim@centricular.com>
* .gitignore:
* Makefile.am:
* configure.ac:
* gst-plugins-good.spec.in:
Remove generated .spec file
Likely extremely bitrotten, and we should not ship this anyway.
2016-12-14 10:15:10 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Check that the XiTh size is big enough
https://bugzilla.gnome.org/show_bug.cgi?id=775794
2016-12-09 20:27:53 +0900 Heekyoung Seo <heekyoung.seo@lge.com>
* gst/isomp4/qtdemux.c:
qtdemux: Check node length of video sample description
Add check for node length of video sample description and its fields and
for the XiTh atom.
Also unify the code a bit.
https://bugzilla.gnome.org/show_bug.cgi?id=775794
2016-12-08 18:50:52 +0900 Heekyoung Seo <heekyoung.seo@lge.com>
* gst/isomp4/fourcc.h:
* gst/isomp4/qtdemux.c:
qtdemux: Enable xvid/mp2 codec support
Add support for xvid video and mp2 audio, add m2v1 fourcc.
https://bugzilla.gnome.org/show_bug.cgi?id=775794
2016-12-13 22:32:46 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/rtp/gstrtpvp9depay.c:
* tests/check/elements/rtpjitterbuffer.c:
* tests/check/elements/rtprtx.c:
* tests/check/elements/vp9enc.c:
gst: Don't declare variables inside the for loop header
This is a C99 feature.
2016-12-11 13:27:27 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/audiofx/gstscaletempo.c:
scaletempo: Ensure to reinit buffers whenever they were not allocated yet
That is, whenever we go through start/stop we have to ensure that on the
next opportunity the buffers are reallocated again. Otherwise the
buffers might be NULL because the element was reused with the same
configuration as before (i.e. set_caps() wouldn't have reinited the
buffers).
https://bugzilla.gnome.org/show_bug.cgi?id=775898
2016-12-10 12:52:18 +0000 Tim-Philipp Müller <tim@centricular.com>
* docs/design/Makefile.am:
* docs/design/design-rtpauxiliary.txt:
* docs/design/design-rtpcollision.txt:
* docs/design/design-rtpretransmission.txt:
docs: design: remove, moved to gst-docs
2016-12-09 17:17:35 -0300 Thibault Saunier <tsaunier@gnome.org>
* meson.build:
meson: Support building without Gst debug
2016-12-09 17:55:39 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/flx/gstflxdec.c:
* gst/flx/gstflxdec.h:
flxdec: Only send SEGMENT events after CAPS
I.e., don't just forward the event but delay it if we don't have caps on
the srcpad yet.
2016-12-09 17:49:40 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/flx/gstflxdec.c:
flxdec: Unref and unmap buffers in all code paths as needed
https://bugzilla.gnome.org/show_bug.cgi?id=775888
2016-12-06 17:42:31 +0530 Arun Raghavan <arun@osg.samsung.com>
* sys/v4l2/gstv4l2object.c:
v4l2object: Don't set empty interlace-mode list
If for some reason we fail to probe formats (all try_fmt calls fail, for
example), this is not a critical error, but we end up with an empty list
of interlace modes. This causes all subsequent negotiation to fail.
This patch fixes interlace-mode setting to be skipped if we failed to
detect any.
https://bugzilla.gnome.org/show_bug.cgi?id=775702
2016-12-07 17:22:22 +0530 Garima Gaur <garima.g@samsung.com>
* gst/monoscope/gstmonoscope.c:
monoscope: Unref allocation query after finished with it
https://bugzilla.gnome.org/show_bug.cgi?id=775752
2016-12-06 07:48:47 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/flx/gstflxdec.c:
flxdec: Allocate 0-initialized memory for the decoded frame
Otherwise we might leak arbitrary information from the uninitialized
memory if not every pixel is written.
https://scarybeastsecurity.blogspot.gr/2016/12/1days-0days-pocs-more-gstreamer-flic.html
2016-12-05 07:57:19 -0700 Matt Staples <staples255@gmail.com>
* gst/rtsp/gstrtspsrc.c:
rtspsrc: Fix session cleanup when handling redirect on PLAY
Redirect on PLAY wasn't doing the necessary session cleanup. Fixed by
removing code from gst_rtspsrc_send that changed the state varable upon
encountering a redirect. Better to let the redirect handlers in
gst_rtspsrc_retrieve_sdp and gst_rtspsrc_play do their own
state-dependent cleanup.
https://bugzilla.gnome.org/show_bug.cgi?id=775543
2016-09-07 16:10:27 +0300 Aleix Conchillo Flaque <aleix@oblong.com>
* gst/rtsp/gstrtspsrc.c:
rtspsrc: always send teardown request
Allow CMD_CLOSE to cancel all commands not only CMD_PAUSE
and ignore CMD_WAIT while closing.
https://bugzilla.gnome.org/show_bug.cgi?id=748360
2016-12-03 08:19:27 +0100 Edward Hervey <bilboed@bilboed.com>
* README:
* common:
Automatic update of common submodule
From f980fd9 to 39ac2f5
2016-12-01 17:08:09 +0100 Edward Hervey <bilboed@bilboed.com>
* gst/rtpmanager/gstrtpjitterbuffer.c:
* gst/rtpmanager/rtpjitterbuffer.c:
jitterbuffer: Don't leak duplicate items
When providing items with a seqnum, there is a (very small) probability
that an element with the same seqnum already exists. Don't forget
to free that item if it wasn't inserted.
And avoid returning undefined values when dealing with duplicate items
2016-12-01 11:23:02 +0100 Edward Hervey <edward@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Sanitize unknown codec caps
We might have non-printable characters in the unknown fourcc, replace
them with '_', in the same way we do it for unknown tags.
2016-12-01 20:04:28 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/avi/gstavidemux.c:
avidemux: Free vprp chunk also if it existed but we made no use of it
https://bugzilla.gnome.org/show_bug.cgi?id=775479
2016-12-01 17:38:33 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/matroska/matroska-read-common.c:
matroskademux: Fix memory leak when parsing attachments
gst_tag_image_data_to_image_sample() does not take ownership of the
passed memory, so don't set it to NULL to allow us to free it later.
https://bugzilla.gnome.org/show_bug.cgi?id=775472
2016-12-01 14:56:18 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/matroska/matroska-read-common.c:
matroskademux: Unify zlib/bzip2 decompress loops with the ones from qtdemux
Especially, simplify the code a bit.
2016-12-01 14:41:48 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Increase inflate buffer in bigger steps
1024 bytes is quite small, let's do 4096 bytes (or one page).
Also remove redundant if, we're always in that case when getting here.
2016-12-01 14:30:49 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Ensure that size of the pasp atom is as much as we need
https://bugzilla.gnome.org/show_bug.cgi?id=775455
2016-12-01 14:30:10 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Free compressed moov node and it's corresponding decompressed data
https://bugzilla.gnome.org/show_bug.cgi?id=775455
2016-12-01 14:29:21 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Check size of compressed MOOV header against available data
And actually read the size of the cmvd atom from the right position.
https://bugzilla.gnome.org/show_bug.cgi?id=775455
2016-12-01 14:27:55 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Fix zlib inflate loop
Handle errors cleanly, deallocate all memory and return the actual size
of the inflated data.
https://bugzilla.gnome.org/show_bug.cgi?id=775455
2016-12-01 13:38:16 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/audioparsers/gstaacparse.c:
aacparse: Make sure we have enough data in the codec_data to be able to parse it
Also error out cleanly if mapping the buffer failed.
https://bugzilla.gnome.org/show_bug.cgi?id=775450
2016-12-01 13:32:22 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Fix out of bounds read in tag parsing code
We can't simply assume that the length of the tag value as given
inside the stream is correct but should also check against the amount of
data we have actually available.
https://bugzilla.gnome.org/show_bug.cgi?id=775451
2016-12-01 15:06:06 +0530 Garima Gaur <garima.g@samsung.com>
* gst/rtp/gstrtph264depay.c:
* gst/rtp/gstrtpsbcdepay.c:
rtp: Fix some memory leaks in usage of gst_pad_get_current_caps()
https://bugzilla.gnome.org/show_bug.cgi?id=775071
2016-11-30 17:56:02 +0200 Vivia Nikolaidou <vivia@ahiru.eu>
* gst/isomp4/qtdemux.c:
qtdemux: Read interlacing information from 'fiel' atom
Read interlacing and TFF/BFF information from the 'fiel' atom and pass it
into the caps
https://bugzilla.gnome.org/show_bug.cgi?id=775414
2016-11-29 13:55:40 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/qtdemux.c:
qtdemux: Fix compiler warning
qtdemux.c: In function ‘qtdemux_parse_trak’:
qtdemux.c:10184:38: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 9 has type ‘gint {aka const int}’ [-Werror=format=]
GST_DEBUG_OBJECT (qtdemux, "Found jpeg: len %u, need %lu", len,
^
2016-11-28 13:45:24 -0800 Scott D Phillips <scott.d.phillips@intel.com>
* gst/isomp4/qtdemux.c:
qtdemux: Change off_t type to gint
off_t is a signed integer type provided by sys/types.h on posix systems.
Replace with gint for building on non-posix systems (like windows).
https://bugzilla.gnome.org/show_bug.cgi?id=775287
2016-11-22 21:00:25 -0800 Scott D Phillips <scott.d.phillips@intel.com>
* meson.build:
meson: add libm to has_function checks
The functions from math.h may be implemented in libm.
https://bugzilla.gnome.org/show_bug.cgi?id=774876
2016-10-27 23:02:37 +0530 Nirbheek Chauhan <nirbheek@centricular.com>
* ext/meson.build:
Revert "meson: dv plugin now works on MSVC"
This reverts commit 05a89613feff70cff416367f5aa807a1d5c68b63.
Let's not put in stuff that needs unreleased Meson. This can go in
for the next cycle.
2016-11-28 13:51:41 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/avi/gstavidemux.c:
avidemux: Ensure that tags are valid UTF-8 before adding them to the taglist
https://bugzilla.gnome.org/show_bug.cgi?id=775219
2016-11-28 12:22:49 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/multipart/multipartdemux.c:
multipartdemux: Post an error message on the bus if we got EOS without having added any pads
2016-11-28 12:00:09 +0200 Sebastian Dröge <sebastian@centricular.com>
* ext/soup/gstsouphttpsrc.c:
souphttpsrc: Handle non-UTF8 headers and error reasons more gracefully
Especially don't put them into GstStructures in one way or another, just
ignore them or error out cleanly depending on the importance of their
content.
2016-11-28 09:30:25 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/rtp/gstrtpvrawpay.c:
vrawpay: Error out cleanly if mapping the video frame fails
Instead of later dereferencing NULL and crashing.
2016-11-27 11:14:13 +0100 Edward Hervey <edward@centricular.com>
* gst/rtpmanager/gstrtprtxsend.c:
rtprtxsend: Update statistics before pushing
If an element queries the number of retransmission buffers pushed
*while* the push is still taking place (and before the object lock
is taken just after) it would end up with the wrong statistic
being reported.
Increment it just before the push, avoids races when getting statistics
https://bugzilla.gnome.org/show_bug.cgi?id=768723
2016-11-26 11:20:51 +0000 Tim-Philipp Müller <tim@centricular.com>
* .gitmodules:
common: use https protocol for common submodule
https://bugzilla.gnome.org/show_bug.cgi?id=775110
2016-07-28 18:51:24 +0200 Philipp Zabel <p.zabel@pengutronix.de>
* sys/v4l2/gstv4l2bufferpool.c:
gstv4l2bufferpool: lock flush_stop against regular qbuf
These can be called from different threads and both manipulate the
pool->buffers array. Lock them properly and let flush_stop move the
array contents into a temporary array on the stack to avoid having
to call release_buffer under the object lock.
https://bugzilla.gnome.org/show_bug.cgi?id=775015
2016-11-24 14:25:22 +0100 Philipp Zabel <p.zabel@pengutronix.de>
* sys/v4l2/gstv4l2bufferpool.c:
gstv4l2bufferpool: remove critical error message when process is called on an inactive pool
If the pool is inactive, it is guaranteed to also be flushing, so the
following check will return GST_FLOW_FLUSHING anyway.
This can happen if a v4l2src is blocking on DQBUF in create and is sent
an EOS event on another thread. In that case the pool is set to
flushing/inactive without locking, the v4l2src is unblocked, and may
call pool_process with a valid buffer on the already inactive pool.
https://bugzilla.gnome.org/show_bug.cgi?id=775014
2016-11-24 14:41:52 +0100 Philipp Zabel <p.zabel@pengutronix.de>
* sys/v4l2/gstv4l2src.c:
v4l2src: release buffer if create fails
gst_base_src_get_range does not expect a buffer to be returned in
the error case, so we are leaking a reference here if create fails.
https://bugzilla.gnome.org/show_bug.cgi?id=775014
2016-11-23 18:34:04 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/rtpmanager/gstrtpbin.c:
rtpbin: Handle create_session() returning NULL in bundle code
CID 1394492.
2016-11-22 16:42:55 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/gstqtmux.c:
qtmux: Make sure to only change DTS of writable buffers
And trivial cleanup
https://bugzilla.gnome.org/show_bug.cgi?id=774840
2016-11-22 16:42:26 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/gstqtmux.c:
qtmux: Error out much earlier if we don't have a valid PTS
https://bugzilla.gnome.org/show_bug.cgi?id=774840
2016-11-22 16:18:41 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/gstqtmux.c:
qtmux: Only use buffer durations if they are actually valid
https://bugzilla.gnome.org/show_bug.cgi?id=774840
2016-11-22 15:59:19 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/gstqtmux.c:
qtmux: Revert commits that set DTS and duration on buffers unconditionally
39f7e52266fde3b3c035e22cbcbb2bb1fa207b17 was setting the buffer duration
to 0 if is not valid, under the assumption that this is "the last"
buffer and no others are coming next. This is wrong, last_buf is the
previous buffer and not the very last one.
4e3c13c87c258c9c95e2217d32ab314d12b5fffc was setting DTS to 0 if there
was none. This will set DTS to 0 for all e.g. audio streams, completely
messing up calculations if streams don't start at 0.
https://bugzilla.gnome.org/show_bug.cgi?id=774840
2016-11-22 15:58:37 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/isomp4/gstqtmux.c:
qtmux: Only write "gap" edit list if there is a non-zero gap
https://bugzilla.gnome.org/show_bug.cgi?id=774840
2016-11-23 07:09:06 +1100 Matthew Waters <matthew@centricular.com>
* gst/flx/flx_color.c:
* gst/flx/flx_fmt.h:
* gst/flx/gstflxdec.c:
* gst/flx/gstflxdec.h:
flxdec: rewrite logic based on GstByteReader/Writer
Solves overreading/writing the given arrays and will error out if the
streams asks to do that.
Also does more error checking that the stream is valid and won't
overrun any allocated arrays. Also mitigate integer overflow errors
calculating allocation sizes.
https://bugzilla.gnome.org/show_bug.cgi?id=774859
2016-11-23 11:20:49 +0200 Sebastian Dröge <sebastian@centricular.com>
* gst/flx/gstflxdec.c:
flxdec: Don't unref() parent in the chain function
We don't own the reference here, it is owned by the caller and given to
us for the scope of this function. Leftover mistake from 0.10 porting.
https://bugzilla.gnome.org/show_bug.cgi?id=774897
2016-11-22 20:33:29 +0200 Sebastian Dröge <sebastian@centricular.com>