rtpjitterbuffer: fix crash when do-retransmission=true and a lot of buffers are lost

The problem here was that the jitterbuffer lock was unlocked to push
the event, but that caused another thread to remove the timer currently
being processed, probably because the amount of rtx events
(and therefore timers) was getting too high. The solution is to
unlock and push the event only after timer processing has finished.

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

matroskademux: Avoid division by zero assert in gst_matroska_demux_search_pos

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

wavenc: generate a non-empty data header

Restore the behavior of the element to the state before commit
db29522a430e44450415ca3676abd1b77ee923d9. A non-empty header is
generated and when the EOS event is received the header is generated
again, this time with the correct size.

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

rtpsource: update receiver stats for sender

An internal sender in a session is also a receiver of its own packets so update
the receiver stats. Other senders in the session will use this info to generate
correct RB blocks in their SR reports.

rtpsource: refactor receiver stats update

qtdemux: handle fragmented files with mdat before moofs

Assume a file with atoms in the following order: moov, mdat, moof,
mdat, moof ...

The first moov usually doesn't contain any sample entries atoms (or
they are all set to 0 length), because the real samples are signaled
at the moofs. In push mode, qtdemux parses the moov and then finds the mdat,
but then it has 0 entries and assumes it is EOS.

This patch makes it continue parsing in case it is a fragmented file so that
it might find the moofs and play the media.

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

qtdemux: When using a buffered mdat, store all received data for later use

In push mode, when qtdemux can't use a seek to skip the mdat buffer it has
to buffer it for later use.

The issue is that after parsing the next moov/moof, there might be some
trailing bytes from the next atom in the file. This data was being discarded
along with the already parsed moov/moof and playback would fail to continue
after the contents of this moov/moof are played.

This is particularly bad on fragmented files that have the mdat before the
corresponding moof. So you'd get:

mdat|moof|mdat|moof ...

When a moof was received, it usually came with some extra bytes that would
belong to the next mdat (because upstream doesn't care about atoms alignment).
So those bytes were being discarded and playback would fail.

This patch makes qtdemux store those extra bytes to reuse them later after the
mdat is emptied.

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

multiudpsink: Also use the bind-port property if no bind-address was given

osxaudiosink: fix segfault when we can't get the channels layout

rtpvp8pay: Make Picture ID mode configurable and default to no picture ID

Some implementations (linphone) only support no picture at all in the
stream and will fail if one is provided.

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

Automatic update of common submodule

From 865aa20 to dbedaa0

Add call to gst_rtp_h264_pay_clear_sps_pps() when receiving a STREAM_START event

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

rtsp: Add missing gio-2.0 deps and includes

audioiirfilter: Fix initialization coefficient handling

Broke unit test.

rtspsrc: allow setting tls certificate validation flags

Added a new property "tls-validation-flags". If the url transport is
TLS, the validation flags will be set to the rtsp connection.

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

audioiirfilter: Don't crash if no filter coefficients are provided

...and by default use a identity filter.

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

wavpackenc: Fix writing of MD5 sums and other metadata blocks

These don't have the FINAL_BLOCK flag set.

hdv1394src: Make it possible to select a camera by its GUID

The source hdv1394src has the guid property that permits select a camera
connected from its GUID number.
However when this property is setted the selected camera is not changed.
The source continues using the default camera.

This problem was solved using the function iec61883_cmp_connect.
The reference for the function could be found here:
http://www.dennedy.org/libiec61883/API-iec61883-cmp-connect.html

The solution came from dvgrab source code.

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

tests: souphttpsrc: add explicit cast to silence warning

Silencing this warning:

elements/souphttpsrc.c:533:14: error: comparison between ‘SoupKnownStatusCode’ and ‘enum <anonymous>’ [-Werror=enum-compare]
   if (status != SOUP_STATUS_OK && !send_error_doc)

With gcc 4.8.2 (debian)

rtspsrc: proxy new buffer mode

jitterbuffer: add new timestamp mode

Add a new timestamp mode that assumes the local and remote clock are
synchronized. It takes the first timestamp as a base time and then uses the RTP
timestamps for the output PTS.

matroska-demux: Fix compiler warning

matroska-demux.c: In function 'gst_matroska_demux_add_stream':
matroska-demux.c:1379:7: error: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'guint64' [-Werror=format=]
       "%03u", context->uid);
       ^

videomixer: remove unneeded guint comparaison

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

osxvideosink: fix missing selector name warnings

The spaces matter in ObjC

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

y4menc: fix uninitialized variable warning

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

qtdemux: check if the end_time is defined before using it

Avoids sending EOS too soon because of overflow. Can happen on
fragmented mp4 playback.

qtdemux: use correct unref function

Events aren't GstObjects, but GstMiniObjects

qtdemux: rename chunks_are_chunks to chunks_are_samples and flip the logic

As the variable name suggests, sometimes chunks are chunks. Rename the variable
to tell what they are when they are not chunks.

qtdemux: fix typos and add more logging for unhandled parts

multiudpsink: Fix memory leak

Unmap all GstMemory of the current buffer when flushing.

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

flvmux: fix broken sample pipeline

which was muxing raw audio and video into flvmux, which won't work,
even if there were converters.

flvmux: require stream-format=raw for mpeg-2 too, but don't require framed field

raw implies that it's framed already. Fixes .. ! faac ! flvmux

souphttpsrc: do not emit EOS when connection drops

If the pipeline is stalled for too long, souphttpsrc will block and
stop fetching data from the network. This can cause the connection to
drop and souphttpsrc would handle it as an EOS. This patch makes it
persist and try to fetch more data until the end of the content length
or until receiving an error that it is beyong limits in case the content
is unknown.

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

dvdec: Don't send segment event before caps

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

dvdemux: Send stream-start, caps and segment events in the right order

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

wavenc: A-Law and Mu-Law don't have width/depth/signed caps fields

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

deinterlace: Fix handling of planar video formats in greedyh method

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

matroska: Trivial grammar fix on debug msg

matroskamux: Add context flag for WebM

WebM has a couple of specific requirements we need to handle.
Idea is to set this flag once and just rely on mux->is_webm
at run time instead of repeatedly figuring this out from
GST_MATROSKA_DOCTYPE_WEBM (which requires a strcmp()).

matroska: Do not write SegmentUID for WebM mux

WebM spec states SegmentUID is Unsupported. Files produced
with gstreamer without this change will spit an error like
this when passed to mkvalidator:

ERR201: Invalid 'SegmentUID' for profile 'webm' in Info at 192

matroskademux: make dvd palette change event sticky

So they don't get lost.

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

videoflip: Add automatic flip mode driven by image-orientation tag

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

v4l2bufferpool: O_CLOEXEC needs _GNU_SOURCE

On some systems (E.G. uClibc and older Glibc versions), O_CLOEXEC is only
defined when _GNU_SOURCE is specified, so do so.

_GNU_SOURCE needs to be defined before any system headers are included,
so move the fcntl.h section up.

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

jitterbuffer: fix race in flush-start/flush-stop

When flush-stop arrives before we process the result of the _push() in the
loop function, we might pause even though we are not flushing anymore. Fix this
race by waiting for the srcpad loop function to completely pause after doing the
flush-start.

videomixer: Update videoconvert copy

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

videomixer: Check if the pad needs reconfiguration in collected

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

jpegdec: Relax sink caps

Since jpegdec already parse the jpeg stream, the sink caps could be
relaxed. This will allow jpegdec to be selected in more case and in
particular when the jpeg typefinder does not find the width and height.

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

v4l2src: print probed caps as caps again in debug log

This got lost during refactoring.

qtdemux: Add support for the mp2v fourcc for MPEG-2 video

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

matroskademux: Fix memory leak

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

qtdemux: Add HEVC support

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

rtpgstpay: Fix memory leak

We were leaking the GList nodes of the pending buffers.

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

rtpjitterbuffer: fix race when updating the next_seqnum

If we were not waiting for the missing seqnum when we insert the lost packet
event in the jitterbuffer, we end up not updating the next_seqnum and wait
forever for the lost packets to arrive. Instead, keep track of the amount of
packets contained by the jitterbuffer item and update the next expected
seqnum only after pushing the buffer/event. This makes sure we correctly handle
GAPS in the sequence numbers.

rtpjitterbuffer: small debug improvement

rtpjitterbuffer: reset skew does not reset clock-rate

Don't reset the clock-rate when we reset the skew correction algorithm.
Reset the skew correction algorithm when we change the clock-rate.

rtpjitterbuffer: pause timer when PAUSED

Also pause the timer when we go to the PAUSED state. It is possible that we
don't have a clock or base-time in PAUSED to perform the timeouts.

rtpjitterbuffer: improve debug

mp4mux: Do not require framerate in peer video caps

Remove the framerate restriction on the caps.

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

rtspsrc: also go into the loop function after connect

When we have opened the stream, go into the loop function so that we can
receive messages from the server.

matroskademux: move the check for subtitle buffer being null terminated before validating UTF-8

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

rtpjitterbuffer: don't calculate skew without rtptime

Skip trying to calculate the skew when we don't have an rtptime.
It causes problems when lost packet events are placed in the jitterbuffer.

configure: get rid of AS_SCRUB_INCLUDE

Should not be needed any more.

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

rtspsrc: disable checks when linking pads

We know the pad links will work (and we don't check the return value
anyway).

rtpbin: avoid some pad link checks

Link pads without checks, we know it will work.

qtmux: Don't error out if downstream is not seekable for non-fragmented variants

Doing so would be a regression over 1.0 and breaks the unit test.
However the result will be most likely unusable, so let's post
a warning message on the bus.

rtpjitterbuffer: calculate some stats

rtpjitterbuffer: move send_lost_event function

Move the send_lost_event function to the do_lost_event handling, there is no
need to have a separate function.

qtdemux: add code to parse creation time earlier than 1970

Use g_date_time seconds manipulation to allow to cover the quicktime
spec for creation_time. It uses seconds since 1904.

Both paths could be done using the generic approach of seconds since
1904 with GDateTime handling, but the first path using seconds from
1970 should be more commonly found and avoids a few objects creation and
ref/unref, so keep it there for performance.

Additionally, the code for handling seconds since 1970 changed from >
to >= because having 0 seconds since 1970 is also a valid case for that
path to handle.

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

matroskademux: update stream->pos when sending buffers so that gap events are not sent unnecessarily

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

Automatic update of common submodule

From 6b03ba7 to 865aa20

configure: Actually use 1.3.0.1 as version to make configure happy

Back to development

Release 1.2.0

Update .po files

osxvideosink: fix segfault releasing the sink

show_frame is deferred to the main thread and can be called
when the sink has been released, so we need to keep an extra ref
on ObjectiveC object helper.

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

v4l2bufferpool: Restore original GstMemory in buffer if it has been changed

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

rtpmanager: update docs

docs: update docs with 1.0 element names

tests: add test for retransmission because of reordering

rtpjitterbuffer: always store lost event in jitterbuffer

Always prepare a lost event in the jitterbuffer, it is to wake up and make the
pushing thread continue. We drop the event when we are not supposed to push lost
events downstream.

rtpjitterbuffer: schedule lost event differently

Schedule the lost event by placing it inside the jitterbuffer with the seqnum
that was lost so that the pushing thread can interleave and push it properly.

tests: remove timeouts from check

Timeouts make the test unreliable and are not needed.

rtpjitterbuffer: remove list debug

rtpjitterbuffer: add type to the item

So that the upper layer can know what data is contained in the item.

rtpjitterbuffer: fix flush

Pass function to flush to properly free the queue items.

rtpjitterbuffer: append seqnum -1 packets

rtpjitterbuffer: use structure to hold packet information

Make the jitterbuffer operate on a structure containing all the packet
information. This avoids mapping the buffer multiple times just to get the RTP
information. It will also make it possible to store other miniobjects such as
events later.

rtpjitterbuffer: update expected timer when possible

When we receive a packet and we have some missing packets, we can update their
estimated arrival times based on the timestamp difference.

rtpjitterbuffer: fix order of timeout events

Improve the order of the timeout events, if there are timers with the same
timeout, we want to trigger the lowest seqnum first. For this we need to loop
over the complete array of timers to find the best one before triggering the
timeout.

rtpjitterbuffer: send lost event before signaling next buffer

First send the lost event, then update the next_seqnum counter and then
send the signal to the pushing thread that it can retry to push a buffer. This
avoids pushing out buffers before the lost event is pushed.

jitterbuffer: configure clock-rate on jitterbuffer

Add a get and setter to configure the clock-rate in the jitterbuffer instead of
passing it as an argument to the insert method.

tests: add test for packet delay and retransmission

rtpjitterbuffer: add option to reset retransmission timers

rtpjitterbuffer: stop the timer thread

The timeout code could release the lock so we need to check if we are allowed to
wait for the clock some more.

rtpjitterbuffer: unlock only once

tests: check both PTS and DTS

tests: add unit-test for multiple missing packets

Check if multiple missing packets generate retransmission events and that the
retranmission requests are canceled when the missing packet arrives.

rtpjitterbuffer: improve flush and shutdown

There is no need to unschedule the timer in flush-start, flush-stop will remove
the timers and unschedule.
Unschedule the current timer before attempting to join the timer thread.

tests: improve debug

rtpjitterbuffer: set correct expected time

When we already have a timer for a packet, skip it but don't forget to adjust
the dts to the expected dts of the next packet.