dvb: fix compilation

gl: fix leaks

As the relevant variables are initialized to 0/NULL, we can loop
over the full range and make sure we free partial allocations
when an error happens partway through initialization.

dvb: reject too long socket paths

Coverity 206004

mxfdemux: implement simple KEY_UNIT seeking

If a KEY_UNIT seek was requested, adjust segment
start to position of the key frame.

resindvd: guard against overflow in audio subtitle streams

Turns out there was the same issue as with subtitles.

There is space for a single audio stream, but up to 255
may be used based on a uint8_t value in a struct, which may
or may not be read from the (untrusted) data.
A comment in ifo_types.h says this value is either 0 or 1, so
we can ensure this here without drawbacks.

Coverity 1139585

resindvd: guard against overflow in menu subtitle streams

There is space for a single subtitle stream, but up to 255
may be used based on a uint8_t value in a struct, which may
or may not be read from the (untrusted) data.
A comment in ifo_types.h says this value is either 0 or 1, so
we can ensure this here without drawbacks.

Coverity 1139586

mxfdemux: fix seeking, send data starting from a key unit

Fixes multiple seeking issues. When doing ACCURATE or normal
non-KEYUNIT seeks, mxfdemux would just send data from the
edit unit that covered the seek position, whether that's
a keyframe or not. Decoders would only output things from
the next keyframe then, which means there's a gap between
the start of the segment and the first decoded data in
some cases. In combination with gst-editing-services this
might result in a frozen picture for the duration of that
gap at the beginning (if videorate fixes up the first
buffer's start timestamp to cover the entire gap), or
a black frame (if no videorate is used and videomixer
fills the gap). Also fixes A/V sync issue when requesting
a KEYUNIT seek.

dtsdec: fix buffer overflows

Channels can be up to 7. Also add a guard in case more can be
returned in the future.

Coverity 1139820, 1139821

directsoundsrc: Fix critical due to missing debug category initialization

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

festival: fix fd leak

Coverity 1139831

speed: Clarify passthrough in switch statement

We do want the caps event to be forwarded downstream

CID #1139754

soundtouch: Add missing break in switch

Whoops, modifying the pitch when modifying the output rate ...

CID #1139749

rsvg: Add missing break in switch statement

This wasn't fatal before (worst case it would clear an already empty
adapter).

CID #1139758

mpeg2enc: Add missing break in switch

QUERY_CAPS is fully handled, we don't need to passthrough to the default
handler.

CID #1139756

bz2: Initialize variables

There is a small chance that we might end up in the done step without
having any output available.

Furthermore, when going through not_ready, we need to ensure gst_buffer_unmap
has a properly initialized GstMapInfo.

CID #1139923
CID #1139924
CID #1139919
CID #1139920

check: Fix mpegtsmux caps

Incoming streams need to be parsed. Doesn't fix the test completely though

mpegtsmux: Initialize helper mpegts library

We now use that library, we need to initialize it so that debug
categories (amongst other things) get properly initialized

mxf: fix stop date parsing using the wrong input

Coverity 1139645

jp2kdecimator: fix copy/paste errors in expressions

Correct formulae found in ITU T.800, B.12.1.[345]

Coverity 1139649, 1139650, 1139651

tests: fix shm test deadlock

The test was not bringing the source back to NULL, so its thread
could start creating a buffer while the pads were being shutdown.

glimagesink: Create GL context and set up window from the streaming thread

gst_gl_context_create() might need to dispatch some operations to the
application's main thread, and calling this in the change_state function
can cause deadlocks.

configure: Fix check for iOS for the OpenGL support

There is also an i386 version of iOS, which is for the simulator.
Better use our already existing HAVE_IOS check instead of relying
on the host triplet.

wildmidi: fix possible uninitialized variables

mpdparse: only add location when node existed

Actually return FALSE when the location node was non-existing
Only add the location node when it existed

mpegtsmux: Require parsed/framed input for most of the supported formats

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

d3dvideosink: Only pass a dest rectangle if set, otherwise pass NULL

Call with an uninitialized rectangle will cause errors.

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

tests: fix gl unit tests for recent API changes

gl: pass video info's by reference

gl: fix array initialization

glbumper: remove redundant check

gl/glx: use the context's display rather than asking for the window's

gl: fix assignment of temporary variables

gl: avoid adding a NULL pool to propose allocation

gl/win32: actually build the gl plugins on win32

Regression from c8ff215b02ee178513e076e2dfbd14e9ce77ec49

gl/win32: fix detection of GL/wglext.h

Was resulting in 'present but cannot be compiled' errors

gl: fix detection of extensions with GL versions < 3

Mesa, for example returns valid pointers for glGetIntegerv and
glGetStringi even if the gl version is less than that required for
both those functions to supposedly exist.

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

mpegts: fix docs

Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>
https://bugzilla.gnome.org/show_bug.cgi?id=727356

mpegts: add dvb-t2 delivery system descriptor

Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>
https://bugzilla.gnome.org/show_bug.cgi?id=727356

mpegts: bugfix delivery system descriptors

Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>
https://bugzilla.gnome.org/show_bug.cgi?id=726979

mpegts: use GST_READ_UINT32_BE

Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>
https://bugzilla.gnome.org/show_bug.cgi?id=727187

mpegts: add parsing data broadcast descriptor

Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>
https://bugzilla.gnome.org/show_bug.cgi?id=727187

mpegts: add parsing stream identifier

Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>
https://bugzilla.gnome.org/show_bug.cgi?id=727187

mpegts: use glib alloc and free

Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>
https://bugzilla.gnome.org/show_bug.cgi?id=727159

dashdemux: use correct print format for gint64

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

hlsdemux: Handle errors when switching playlists properly

hlsdemux: Go EOS if the end of the segment is reached

hlsdemux: Store buffer end position in segment.position

mpegtspacketizer: Fix typo in debug output

h264parse: Copy over DISCONT flag from input buffers

tsdemux: Push next buffer after a discont with the DISCONT flag

tsdemux: Drain remaining data on disconts

mpegtsbase: Flush if we receive a discont buffer

tsdemux: Don't dereference NULL if flushed before a program was selected

tsdemux: Ensure that all pending data is pushed even if the stream was not activated before

gst_ts_demux_push_pending_data() will check if it now can activate the
stream and add the pad, we don't have to check that ourselves.

Fixes playback of very short MPEG TS files.

mpegtsbase: Fix pull mode scanning for PCR on small files

If a file does not contain 5 PCRs until it is EOS, or does not
contain more than 655360 bytes the PCR scanning algorithm just
aborted.

hlsdemux: Don't set the segment offset

It's causing wrong running times after seeks or bitrate
switches.

hlsdemux: Set DISCONT flag on all buffers in reverse playback mode

hlsdemux: Fix forwards and backwards searching in the files list

hlsdemux: Unset DTS of all buffers

We won't get a valid DTS from the source.

hlsdemux: Send flush start event before waiting for the tasks to finish

Otherwise we'll wait until buffers are completely processed downstream,
which might take quite some time.

hlsdemux: Implement trick modes via I-frame variant lists

hlsdemux: Implement parsing of #EXT-X-I-FRAME-STREAM-INF

These are I-frame-only variant lists that can be used
for trick mode playback.

gl: egl: fix distcheck and out of source build

build: Fix make dist

tests: fix LDADD order for insertbin test

tests: don't link codecparser tests twice to the lib

tests: fix build of gl unit tests in uninstalled setup

Put local .la files first, and add link to libgstvideo
for tests that use the libgstvideo API directly.

gl/win32: allow not building the gl plugins

Fixes build on windows if <GL/wglext.h> headers are not present.

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

eglglessink: unref last buffer on DRAIN

Similar to 093574053fae243ec5fcfd6adae8185053b1e896
in gstbasesink

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

eglglessink: fix wrong doc block for create_window

Wrong since 1ad253b9 when prop name was changed from
can_create_window to create_window.

gl: don't fail configure if we do not understand the platform

gl: fix out-of-source builds pt2

21b897de2f762bee504cbc007fd108148ee5ae24 did this for x11 and wayland.
Do it for the other backends as well.

gl: only set CAPS_FEATURE_MEMORY_EGL_IMAGE on egl platform

configure.ac: fix OSX host check

Makes the check works with (i386|x86_64)-apple-darwin* hosts.

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

gl: let the user includes itself our egl headers if needed

Forgot to address this change.

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

gl: add EGLImage support

* picked from old libgstegl:
  - GstEGLImageMemory
  - GstEGLImageAllocator
  - last_buffer management from removed GstEGLImageBufferPool

* add-ons:
  - GstEGLImageMemory now old a reference on GstGLContext
    so that it can delete the EGLImage and its gltexture source
    while having the associated gl context being current.
  - add EGLImage support for GstVideoGLTextureUploadMeta which
    mainly call EGLImageTargetTexture2D
  - GstGLBufferPool now supports GstEGLImageAllocator
  - glimagesink / glfilters / etc.. now propose GstEGLImageAllocator
    to upstream

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

gl: deploy egl headers in gst/gl/egl instead of gst/gl

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

gl: remove commented and unsued code in x11 Makefile.am

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

pkgconfig: add gstreamer-gl

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

gl: fix crash if _build_extension_string is not called

On GLES2 then (gl->GetIntegerv && gl->GetStringi) is false
regression introduced by cc6df204e2f58fffda5cbe90f3450aeba95889c4

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

gl: Only enabled cocoa backend on osx (and not ios)

ios requires using EAGL

rtmpsink: Remove URL check for valid playpath.

The playpath is an optional component of the URL - don't require it.

srtpenc: unlock object lock before posting error

Posting an error with the object lock will deadlock. Rework the code
a bit to make things more simple.

codecparsers: cosmetics: drop TAB characters.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

dvbsuboverlay: Clarify confusing debug message

The message is describing the number of bytes remaining for
parsing, not 'missing'.

tsdemux: Fix mpegts_packetizer_set_current_pcr_offset()

When adjusting the PCR offset of groups, adjust the correct group
entry from the list - not the current group repeatedly.

dvbsuboverlay: Remove warning for 2-bit and 8-bit subpictures

Both 2-bit and 8-bit mode now seem fine, so remove the warning
messages about them

dvbsuboverlay: Fix 8-bit subpicture rendering.

Fix 2 small flaws handling 8-bit subpictures that makes
my one test file work.

codecparsers: h264: fix slice_header() parsing for MVC.

The idr_pic_id syntax element depends on IdrPicFlag, which is a calculated
value that does not only depend on NAL unit type (IDR), but possibly also
on MVC non_idr_flag syntax element.

The computed idr_pic_flag is already stored in GstH264NalUnit structure.

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

Signed-off-by: Li Xiaowei <xiaowei.a.li@intel.com>
Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: h264: add gst_h264_parse_nalu_header() helper.

Add helper to parse the NALU header. Move bounds checking to there.

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

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: h264: add gst_h264_parse_sps_data() helper.

Split seq_parameter_set_data() parsing off gst_h264_parse_sps() so
that it could be re-used later on.

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

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: h264: complete set of NAL unit types.

Add missing NAL unit types. They are mostly related to alpha blending,
scalable video coding extensions (SVC, Annex.G), and multiview video
coding extensions (MVC, Annex.H).

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: h264: clean-up gst_h264_parser_parse_sei_message().

Fix build when GST_DISABLE_GST_DEBUG is not defined. Use a switch
statement to dispatch to the various SEI payload handlers.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: h264: fix skipping of unsupported SEI messages.

The payloadSize does not account for emulation prevention bytes. So,
just use nal_reader_skip() for skipping payload_size bits. It should
be possible to further optimize this code since the NAL reader shall
be aligned to byte boundary already.

Kill the now unused nal_reader_skip_to_next_byte() function.

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

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: h264: fix SEI buffering_period() parsing.

Fix parsing of buffering_period() SEI messages. The number of bits
used to express {nal,vcl}_initial_cpb_removal_delay{,_offset} syntax
elements is not 5 but 1 + initial_cpb_removal_delay_length_minus1.

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

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: h264: fix rbsp_more_data() implementation.

Account for trailing zero bits when checking for rbsp_more_data().

In particular, fix an hypothetical stream whereby rbsp_more_data()
is called in the following conditions for PPS header: NalReader
reached position 20, 12 bits are remaining and trailing data at
current byte position is c8 00.

rbsp_more_data() used to return TRUE whereas it should obviously
return FALSE because x8 00 represents a valid rbsp_trailing_bits()
structure.

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

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

gl: fix out-of-source builds.

Always use the locally generated <gst/gl/gstglconfig.h> file.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

gl/download: add support for the bt709 color matrix

gl/upload: add support for the bt709 color matrix

gl: remove explicit reference to X_CFLAGS

They are in GL_CFLAGS if required