qtmux: Don't unconditionally use strnlen()

It's not available on older OSX and we can as well use memchr() here.

https://bugzilla.gnome.org/show_bug.cgi?id=756154

mpegaudioparse: Fix buffer memory leak during failures

mapped buffer is not being unmapped during failures

https://bugzilla.gnome.org/show_bug.cgi?id=756231

auparse: Fix event memory leak

Free the event after being handled to prevent memory leak.

https://bugzilla.gnome.org/show_bug.cgi?id=756799

qtdemux: fix caps leak

If the QtDemuxStream are re-used they may already have caps which used
to be leaked.

Reproduced using the
validate.dash.playback.seek_forward.dash_exMPD_BIP_TC1 validate
scenario.

https://bugzilla.gnome.org/show_bug.cgi?id=756561

qtdemux: Fix taglist memory leak

Free the stream and its sub items instead of just the stream

https://bugzilla.gnome.org/show_bug.cgi?id=756544

qtmux: Fix date memory leak

When getting date from taglist, the memory should be freed after
using it.

https://bugzilla.gnome.org/show_bug.cgi?id=756171

qtmux: Fix sample memory leak

When getting sample from taglist, the memory should be freed after
using it.

https://bugzilla.gnome.org/show_bug.cgi?id=756068

cutter: Fix buffer leak

Buffer is added to the internal cache, and pushed only when accumulated
buffer duration crosses 200 ms. So when the chain ends, the buffer accumulated
is not freed. Freeing the cache when the state changes from PAUSED to READY.

https://bugzilla.gnome.org/show_bug.cgi?id=754212

splitmuxsink: post messages when fragments are being opened and closed

This can be useful for applications that need to track the created fragments
(to log them in a recording database, for example)

https://bugzilla.gnome.org/show_bug.cgi?id=750108

splitmuxsink: allow non-video streams to serve as reference

In the absence of a video stream, the first stream will be used as
reference.

https://bugzilla.gnome.org/show_bug.cgi?id=753617

splitmuxsink: initialize mux_start_time properly

mux_start_time refers to the running_time of the buffer
that goes first in the output file. Normally this time is
0, so this variable is initialized to 0 during the state
change to PAUSED.

However, when dealing with dynamic pipelines and starting
a recording while the pipeline has already run for a while,
the running_time of the first buffer is > 0 and this causes
a problem with detecting the end of the first file(s) when
splitting by duration, because the code will later compare
the threshold_time with (last buffer running_time - mux_start_time)
and will get it wrong until mux_start_time advances enough
to make this difference < threshold_time, creating empty files
in the meantime.

https://bugzilla.gnome.org/show_bug.cgi?id=753624

gdkpixbufsink: don't leak old pixel buffer when setting a new overlay

https://bugzilla.gnome.org/show_bug.cgi?id=755773

flacenc: avoid potential string overflow

We don't necessarily have full control over the input tags, so
it's possible that the ISRC tag contains a longer string than
expected, in which case we'd write over the end of the static-size
13 byte buffer that is FLAC__StreamMetadata_CueSheet_Track::isrc.
Make sure to only copy the ISRC if it's not too long, and make
sure the buffer we write to is always NUL-terminated by using
g_strlcpy().

CID 1324931.

matroskademux: Remove leftover assertion from 0.10

We now allocate memory via GstAllocator and as such can handle arbitrary
alignments, not only <= G_MEM_ALIGN.

https://bugzilla.gnome.org/show_bug.cgi?id=755708

aacparse: fix uninitialized variables in LOAS config reading

On reading LOAS config, flag v=1 and vA=1 combination can occur, leading to warning
"Spec says "TBD"...". Returning TRUE on this case while parameters 'sample_rate' and
'channels' are pointing to uninitialized values can end on setting random values as
rate and channels on src caps.

https://bugzilla.gnome.org/show_bug.cgi?id=755611

Release 1.6.0

Update .po files

smptealpha: Do not set width/height before comparing with old values

Otherwise we end up considering the values did not change and we wrongly
work with the old video format (which will lead to wrong
behaviour/segfaults).

https://bugzilla.gnome.org/show_bug.cgi?id=755621

qtdemux: Accumulate segments for edit lists before activating the next segment

eceb2ccc739092d964d78945e19c2ecedbd214e2 broke segment seeks by always
accumulating segments manually when activating a segment. This is only
needed when handling edit lists, not when activating a segment because of a
seek. Do the accumulation when switching edit list segments instead.

This fixes segment seeks again, while keeping edit lists playback working.

https://bugzilla.gnome.org/show_bug.cgi?id=755471

spectrum: send phase values in the GstMessage for Phase info

https://bugzilla.gnome.org/show_bug.cgi?id=755463

matroska-mux: Don't output a warning on MONO multiview mode.

rtptheoradepay: Fix memory leaks

The same memory leaks were fixed in identical fashion for
vorbisdepay in 06efeff5d979576a252e5dae57f46d6445b1df12 in 2009.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=755277

rtp{vorbis,theora}{pay,depay}: Cosmetic cleanup

* use g_list_free_full(), don't iterate elements maually when freeing
* call gst_rtp_*_pay_clear_packet(), don't duplicate its code
* use gst_buffer_unref() to clarify that it is buffers being released,
  instead of refering directly to gst_mini_object_unref()

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=755277

rtp{vorbis,theora}pay: Store headers in the packet buffers lists, not a NULL buffer

https://bugzilla.gnome.org/show_bug.cgi?id=755265

Release 1.5.91

Update .po files

po: Update translations

avidemux: Fix taglist leak

gst_tag_list_insert() does not take ownership of the inserted taglist.

https://bugzilla.gnome.org/show_bug.cgi?id=755138

aacparse: Skip LOAS AAC until a valid config is seen.

It's normal when dropping into the middle of a stream to
not always have the config available immediately, so skip LOAS
until a valid config is seen without either setting invalid
caps or erroring out.

https://bugzilla.gnome.org/show_bug.cgi?id=751386

rtpjitterbuffer: reset just a bit more upon flush_stop

rtpjitterbuffer: remove dead struct member

multiudpsink: fix GError memory leak when hostname resolution fails

https://bugzilla.gnome.org/show_bug.cgi?id=754869

matroskamux: drop HEADER flag from output buffers

Drop HEADER flag from output buffers if they are not indeed
headers.

Fixes resending of headers in tcp connection handling

https://bugzilla.gnome.org/show_bug.cgi?id=754768

matroskamux: fix matroskamux ! matroskademux

Don't carry over DISCONT flags from the input buffers to the
output buffer, or the demuxer might reset its state when it
receives the first data buffer just after parsing the simple
block header, and then expect sane data to follow.
Fixes matroskamux ! demux erroring out.

https://bugzilla.gnome.org/show_bug.cgi?id=754768
https://bugzilla.gnome.org/show_bug.cgi?id=657805

rtsp: fix small README typo

https://bugzilla.gnome.org/show_bug.cgi?id=754807

wavpackparse: set both pts and dts so baseparse doesn't make up wrong dts after seeks

https://bugzilla.gnome.org/show_bug.cgi?id=752106

flacparse: set both pts and dts so baseparse doesn't make up wrong dts after a seek

flac contains the sample offset in the frame header, so after a seek
without index flacparse will know the exact position we landed on and
timestamp buffers accordingly. It only set the pts though, which means
the baseparse-set dts which was set to the seek position prevails, and
since the seek was based on an estimate, there's likely a discrepancy
between where we wanted to land and where we did land, so from here on
that dts/pts difference will be maintained, with dts possibly multiple
seconds ahead of pts, which is just wrong. The easiest way to fix this
is to just set both pts and dts based on the sample offset, but perhaps
parsed audio should just not have dts set at all.

https://bugzilla.gnome.org/show_bug.cgi?id=752106

docs: remove properties and signals that no longer exist

https://bugzilla.gnome.org/show_bug.cgi?id=726443

flvmux: Make the element count in arrays not include end

One-line removal of tags_written++

This should fix rtmp output to crtmpserver, and hopefully
noone is expecting that the element count includes the end
element, as different bits of documentation say different
things about whether it should or not.

https://bugzilla.gnome.org/show_bug.cgi?id=661624

flvmux: Store incoming bitrate tags and send in the metadata

Apparently the Microsoft Azure RTMP server requires that the
videodatarate and audiodatarate metadata be provided, so
set those, even if it's to 0. Use the actual input bitrate
tags if available.

rtspsrc: Don't parse key data more than needed.

When an auxilliary streams are present in the SDP media,
there's no need to re-parse the SDP attributes multiple
times.

rtspsrc: Fix SRTP + RTX, auth access, a leak, and an invalid memory access.

In parse_keymgmt(), don't mutate the input string that's been passed
as const, especially since we might need the original value again if
the same key info applies to multiple streams (RTX, for example).

When a resource is 404, and we have auth info - retry with the auth
info the same as if we had receive unauthorised, in case the resource
isn't even visible until credentials are supplied.

Fix a memory leak handling Mikey data.

When generating a random keystring, don't overrun the 30 byte
buffer by generating 32 bytes into it.

udpsrc: Fix build with GLib < 2.44

G_IO_ERROR_CONNECTION_CLOSED was added in 2.44.

udpsrc: Ignore G_IO_ERROR_CONNECTION_CLOSED when receiving data

This happens on Windows if we use the same socket for sending packets,
and the remote sends ICMP port/host unreachable messages.

https://bugzilla.gnome.org/show_bug.cgi?id=754534

rtpvorbis/theoradepay: Fix handling of fragmented packets

This was broken in b1089fb520 by not considering the full packet length of a
fragmented packet but only the length of the first one.

https://bugzilla.gnome.org/show_bug.cgi?id=754417

dtmfsrc: Reply to latency query

tests: Fix rtpsession test failure

The time of the first RTCP packet is semi-random, so
sometimes it was produced before enough packets from
the second SSRC were received. First drop queued RTCP
packets, then advance the clock enough to ensure
that at least one new RTCP packet is produced.

https://bugzilla.gnome.org/show_bug.cgi?id=750731

level: improve the test for multi-channel mode

Change the test to verify the read-index for multiple messages per buffer.
See https://bugzilla.gnome.org/show_bug.cgi?id=754144

matroskademux: Align raw video frames to 32 bytes

Outputting unaligned video frames causes videoscale et al to
crash when attempting SIMD-accelerated conversion.

https://bugzilla.gnome.org/show_bug.cgi?id=736965

level: fix level calculations for mutliple channels

This was broken with 7b90bf32150897a141a29a12ecab555d8c5b7fab.

smpte: Fix memory leak

In gst_smpte_collected(), check upfront if input formats are same
or not. This avoids allocation of in1 and in2 buffers and
subsequent memory leak when input formats do not match.

https://bugzilla.gnome.org/show_bug.cgi?id=754153

tests: souphttpsrc: don't try to connect to dead radio server

rtspsrc: Trivial fix to check correct condition

When checking for describe method, because of missing parentheses, wrong
condition is being checked, which will result in wrong behavior.

https://bugzilla.gnome.org/show_bug.cgi?id=753912

matroska: read: fix tag list memory leak

gst_toc_entry_merge_tags makes a new ref of the taglist, so it should
be unref'ed as soon as the tags are merged to the tocentry

https://bugzilla.gnome.org/show_bug.cgi?id=753904

wavpackdec: fix taglist memory leak

When passing the taglist to gst_audio_decoder_merge_tags, the reference is increased
by audiodecoder and the caller should free the taglist being passed.

https://bugzilla.gnome.org/show_bug.cgi?id=753903

v4l2transform: fix pad closing

Signed-off-by: Jean-Michel Hautbois <jean-michel.hautbois@veo-labs.com>
https://bugzilla.gnome.org/show_bug.cgi?id=753875

Release 1.5.90

Update .po files

po: Update translations

multifilesrc: fix regression with starting from index set via index property

When we haven't started yet, set the start_index when we set the index property,
so that we start at the right index position after the initial seek. The index
property was never really meant to be for writing, but it used to work, so let's
support it for backwards compatibility.

https://bugzilla.gnome.org/show_bug.cgi?id=739472

qtdemux: fix offset calculation when parsing CENC aux info

Commit 7d7e54ce6863ff53e188d0276d2651b65082ffdb added support for
DASH common encryption, however commit
bb336840c0b0b02fa18dc4437ce0ded3d9142801 that went onto master
shortly before the CENC commit caused the calculation of the CENC
aux info offset to be incorrect.

The base_offset was being added if present, but if the base_offset
is relative to the start of the moof, the offset was being added twice.
The correct approach is to calculate the offset from the start of the
moof and use that offset when parsing the CENC aux info.

flacenc: actually return true for accept-caps query handling

rtp: copy metadata in the (de)payloaders which is missed before

https://bugzilla.gnome.org/show_bug.cgi?id=753706

directsoundsink: allow specifying audio playback device

https://bugzilla.gnome.org/show_bug.cgi?id=753670

flacenc: remove single entry if from loop

Iterate from the 2nd channel on and create the 1 channel struct
outside to make loop structure simpler and only slightly faster.

flacenc: implement proper accept-caps

Should just compare with what can be immediatelly accepted by
the element. flacenc can't renegotiate so if it has a caps already
it should only accept if it is that caps otherwise just use the
template caps

flacenc: improve sink pad template caps

Removes the need for custom caps query handling and makes it more
correct from the beginning on the template. It is a bit uglier
to read because there is 1 entry per channel but makes code easier
to maintain.

y4mencode: fix gst-launch version in documentation

audioencoders: use template subset check for accept-caps

It is faster than doing a query that propagates downstream and
should be enough

Elements: speexenc, wavpackenc, mulawenc, alawenc

videoencoders: use template subset check for accept-caps

It is faster than doing a query that propagates downstream and
should be enough

Elements: jpegenc, pngenc, vp8enc, vp9enc, y4menc

mpegaudioparse: use new baseparse API to fix tag handling

https://bugzilla.gnome.org/show_bug.cgi?id=679768

audioparsers: use new base parse API to fix tag handling

https://bugzilla.gnome.org/show_bug.cgi?id=679768

flacparse: use new baseparse API and fix tag handling

https://bugzilla.gnome.org/show_bug.cgi?id=679768

qtdemux: Use signed integer type to be able to check for negative subtraction results

CID 1315829

rtpvorbisdepay: remove dead code

payload_buffer must be NULL in ignore_reserved. Check will always be false.

Introduced by b1089fb5207697ba26edb4ff66ed0f465c6df3cf

CID #1316476

alawenc: port to AudioEncoder base class

audiodecoders: use default pad accept-caps handling

Avoids useless check of downstream caps when handling an
accept-caps query

Elements: flacdec, speexdec, wavpackdec, mulawdec, alawdec

videodecoders: use default pad accept-caps handling

Avoids useless check of downstream caps when handling an
accept-caps query

Elements: jpegdec, pngdec, vp8dec, vp9dec

alawdec: make error handling a bit nicer

Print the element along with the debug to make it easier to trace
the failures

alawdec: port to audiodecoder base class

mulawdec was already ported, alawdec was left behind.

qtdemux: only look for more samples in moofs in pull-mode

For playback of some fragmented formats with qtdemux it will
try to look for the next moof after finishing one but it is only
possible for pull-mode. For playback of streaming fragmented formats
such as DASH it should just not try to look for another moof but
instead wait for more data.

https://bugzilla.gnome.org/show_bug.cgi?id=752602

https://bugzilla.gnome.org/show_bug.cgi?id=752603

dcaparse: Don't look for a second syncword

There are streams out there that consistently contain garbage between
every frame so we never ever find a second consecutive syncword.

See https://bugzilla.gnome.org/show_bug.cgi?id=738237

vp8enc, vp9enc: reset multipass file index when stopping encoder

Fixes multipass encoding when re-using the same element/pipeline
for subsequent encoding runs.

https://bugzilla.gnome.org/show_bug.cgi?id=747728

vp9enc: provide support for multiple pass cache files

Some files may provide different caps insight of one stream. Since
vp9enc support caps reinit, we should support cache reinit too.
If more then file cache file will be created, the naming will be:
cache cache.1 cache.2 ...

Based on patch by: Oleksij Rempel <linux@rempel-privat.de>

https://bugzilla.gnome.org/show_bug.cgi?id=747728

tests: aacparse: use caps query instead of accept-caps

The accept-caps query just does a shallow check at the current
element while at this test we want it to also look at downstream.
So use caps query there.

https://bugzilla.gnome.org/show_bug.cgi?id=753623

audioparsers: enable accept-template flag

Do a quick check with the pad template caps as it is enough. Users
should have figured the appropriate full caps on a previous caps query

https://bugzilla.gnome.org/show_bug.cgi?id=753623

rtspsrc: send the User-Agent header

Sometimes it is useful to know this information on the
server side. Other popular implementations (vlc, ffmpeg, ...)
also send this header on every message.

This includes a new "user-agent" property that the user
can set to use a custom User-Agent string. The default
is "GStreamer/<version>"

https://bugzilla.gnome.org/show_bug.cgi?id=750101

rtspsrc: wrap gst_rtsp_message_init_request in a local function

This will allow adding common request initialization, like the
user agent string, in just one place.

audioecho: make sure buffer gets reallocated if max_delay changes

https://bugzilla.gnome.org/show_bug.cgi?id=753490

vp8enc: provide support for multiple pass cache files

Some files may provide different caps insight of one stream. Since vp8enc
support caps reinit, we should support cache reinit too.
If more then file cache file will be created, the naming will be:
cache
cache.1
cache.2
...

https://bugzilla.gnome.org/show_bug.cgi?id=747728

rtpmp4gdepay: fix timestamps for RTP packets with multiple AUs

Use constantDuration to calculate the timestamp of non-first AU in the
RTP packet.

If constantDuration is not present in the MIME parameters, its value
must be calculated based on the timing information from two consecutive
RTP packets with AU-Index equal to 0.

https://bugzilla.gnome.org/show_bug.cgi?id=747881

souphttpsrc: remove unnecessary if, g_free is null safe

souphttpsrc: add property to set HTTP method

To allow souphttpsrc to be use HTTP methods other than GET
(e.g. HEAD), add a "method" property that is a string. If this
property is not set, GET is used.

https://bugzilla.gnome.org/show_bug.cgi?id=752413

check: Rename states unit test

Makes it easier to differentiate from other modules states unit test

goom: Rename get_type() function of base class to prevent symbol conflicts

This is a problem when statically linking.

rtpjitterbuffer: Keep the DTS estimate if we got no DTS after a jitterbuffer reset

Otherwise we will just output buffers without timestamps after a reset if no
timestamps are provided by upstream, e.g. when using RTSP over TCP.

https://bugzilla.gnome.org/show_bug.cgi?id=749536

matroska: Remove unused variable

https://bugzilla.gnome.org/show_bug.cgi?id=753556

rtp: Copy metadata in the (de)payloader, but only the relevant ones

The payloader didn't copy anything so far, the depayloader copied every
possible meta. Let's make it consistent and just copy all metas without
tags or with only the video tag.

https://bugzilla.gnome.org/show_bug.cgi?id=751774

qtdemux: fix small typo in comment

goom2k1/doc: Fixup previous commit