d3dvideosink: Always lock the D3D surfaces in write mode

Locking them in readonly mode can give different stride to mapping
in write mode, which then causes rendering to be broken.

Happened on all (many?) NVIDIA GPUs.

Thanks to voskater15@gmail.com for hinting at the problem.

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

d3dhelpers: Swap UV planes properly for YV12 as compared to I420

If we only do it in one place colors will look funny.

amc: Properly integrate COLOR_QCOM_FormatYVU420SemiPlanar32m again

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

amcaudiodec: audio/mpeg is always MPEG 1 layer 3

Configuring e.g. layer 2 on such a decoder usually fails unless
it additionally also accepts audio/mpeg-L2.

mpegtspacketizer: Don't dereference NULL if we have no PCR yet

Can happen sometimes if the duration is requested before we
received enough data with a PCR.

tsdemux: Add locking around mpegtspacketiser groups.

Otherwise occasionally DURATION queries can crash, if they happen at
the wrong moment.

glcocoa: initalize NSApp asap when using gst-launch

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

applemedia: enable vtdec and vtenc only on >= 10.8 where VideoToolbox is public

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

vtdec: avoid segfault when output isn't configured

This can help when the callback is called during/after videodecoder
base class shut down

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

vtdec: ignore the dropped flag if buffer was received

Apparently there is some issue with VT that makes it mark the dropped
flag even though the buffer was decoded.

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

d3dvideosink: Don't leak all surfaces

This was broken when disabling the buffer pool exporting.

Also disable buffer pool a bit more efficient...

d3dvideosink: PostMessage() takes integers as last parameters, not pointers

d3dvideosink: Remove unused variable

gl/examples: gitignore generated files

gl : fix qglwtextureshare demo

watchdog: Only create a new GSource if we have a main context

We can still get OOB events while stopping the watchdog element, and while
stopping it we destroy the main context.

Also let the GSource own a reference to the element for additional safety.

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

videoaggregator: reset QoS on segment event

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

tests: h264parser: add test to identify EOSEQ / EOS NALs.

Check that end_of_seq() [EOSEQ] and end_of_stream [EOS] NAL units
are correctly parsed and the reported NAL unit size yields 1 byte,
i.e. the only NalHeaderBytes in there.

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

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

tests: h264parse: add test for byte-stream/au output.

Check that conversion to byte-stream/au formats work and that we
can effectively drop broken/invalid NAL units from the resulting
access unit buffer.

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

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

tests: h264parse: check SEI buffering_period() message is output.

If an SEI NAL unit with a buffering_period() message is inserted
between an SPS and PPS NAL unit, check that the output buffer still
contain it. i.e. make sure that this SEI message is not dropped.

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

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

h264parse: fix collection of access units to preserve config headers.

Always use a GstAdapter when collecting access units (alignment="au")
in either byte-stream or avcC format. This is required to properly
preserve config headers like SPS and PPS when invalid or broken NAL
units are subsequently parsed.

More precisely, this fixes scenario like:
<SPS> <PPS> <invalid-NAL> <slice>

where we used to reset the output frame buffer when an invalid or
broken NAL is parsed, i.e. SPS and PPS NAL units were lost, thus
preventing the next slice unit to be decoded, should this also
represent any valid data.

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

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

h264parse: improve conditions for skipping NAL units.

Carefully track cases when skipping broken or invalid NAL units is
necessary. In particular, always allow NAL units to be processed
and let that gst_h264_parse_process_nal() function decide on whether
the current NAL needs to be dropped or not.

This fixes parsing of streams with SEI NAL buffering_period() message
inserted between SPS and PPS, or SPS-Ext NAL following a traditional
SPS NAL unit, among other cases too.

Practical examples from the H.264 AVC conformance suite include
alphaconformanceG, CVSE2_Sony_B, CVSE3_Sony_H, CVSEFDFT3_Sony_E
when parsing in stream-format=byte-stream,alignment=au mode.

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

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

h264parse: introduce new state tracking variables.

Improve parser state tracking by introducing new flags reflecting
it: "got-sps", "got-pps" and "got-slice". This is an addition for
robustness purposes.

Older have_sps and have_pps variables are kept because they have
a different meaning. i.e. they are used for deciding on when to
submit updated caps or not, and rather mean "have new SPS/PPS to
be submitted?"

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

h264parse: fix and optimize NAL collection function.

Use gst_h264_parser_identify_nalu_unchecked() to identify the next
NAL unit. We don't want to parse the full NAL unit, but only the
header bytes and possibly the first RBSP byte for identifying the
first_mb_in_slice syntax element.

Also fix check for failure when returning from that function. The
only success condition for that is GST_H264_PARSER_OK, so use it.

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

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

codecparsers: h264: clarifications and documentation fixes.

Fix documentation for GstH264NalUnit. The @ref_idc part was totally
unbalanced. Also add a note about @offset and @size fields to remind
that this is relative to the start of the NAL unit, thus including
the header bytes.

codecparsers: h264: fix identification of EOSEQ and EOS NALs.

An end_of_seq() [EOSEQ] or end_of_stream() [EOS] NAL unit is really
one byte long because this shall include the NalHeaderBytes (1) too.
The NALU.offset starts from the first byte of the header.

This is the proper fix to commit d37f842. In practice, this fixes
parsing of FRExt1_Panasonic_D and FRExt2_Panasonic_C, that include
additional frames after an EOSEQ.

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

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

rtmpsrc: Report limited bandwidth

Makes uridecodebin treat this source as a stream source,
allowing timeshifting.

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

mpegts: Expose GstMpegtsDescriptor free function

Nothing earth shattering, but avoids people having to use g_boxed_free()

eagl: Disable usage of R and RG textures on iOS

They don't work currently and just render zeroes, while the
fallback code path with LUM and LUM_ALPHA textures still works.

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

faceblur: release the haar cascade

aggregator: Avoid destroying sources we do not own

+ Unref the maincontext in a new dispose function
+ Make sure to remove all sources on dispose

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

vtenc: fix memory leak

The pixel buffer release callback is called if the void *
dataPtr given to the CVPixelBufferCreateWithPlanarBytes
is not NULL.

According to the documentation dataPtr is supposed to be a
"plane description block" but no specific type is given.

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

segmentation: Release memory storage in finalize

textoverlay: Release memory storage and resize image when caps change

templatematch: Release cvImages when reloading templates are changing caps

pyramidsegment: Release image when changing caps or reusing the element

handdetect: Release memory storage

faceblur: Release memory storage

And also release cvImages and memory storage when changing caps or reusing the
element.

edgedetect: Don't leak cvImages when caps are changing or the element is reused

pyramidsegment: Release memory storage when finalizing

schroenc: Remove obsolete FIXME comment

schroenc: Don't leak all input buffers

h264parse: Fix memory leak in unit test

textoverlay: Free text buffer in finalize

glimagesink: Chain up to the parent class' finalize function

glimagesink: Make sure to always unref the display

Even if we didn't create a context yet.

intersubsrc: basesrc is already creating a pad for us, no need to do it twice

yadif: basetransform is creating pads for us already, don't do it twice

glupload: Fix memory leak in unit test

glcontext: Fix memory leaks in unit test

mpegts: Fix memory leaks in the test and the library

Currently uses g_boxed_free() in the unit test, this needs to
be fixed properly when cleaning up the API.

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

assrender: Make static caps actually static

Otherwise we leak once caps instance on every function call.

uvch264src: Free property strings when disposing the instance

intersubsink/src: Free channel string when finalizing the instance

uvch264src: Fix compiler warning when compiling with G_DISABLE_ASSERT

vc1parse: Fix compiler warnings when compiling with G_DISABLE_ASSERT

kate: Use G_GSIZE_FORMAT instead of unportable %zu

badvideo: Update dist generated ORC files

badvideo: Rename videoconvert functions to prevent conflicts with static linking

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

tests: aggregator: fix various leaks in the tests

compositor: tests: Fix pad leak

Remember to unref requested pad

dataurisrc: fix leak as gst_buffer_replace adds its own ref

So unref the buffer after that otherwise it leaks

aggregator: always store or unref the buffer on the _chain function

Otherwise it leaks, and it is very common to go to flushing when the
pipeline is stopping, leaking a buffer.

aggregator: always unref the buffer on _finish function

Otherwise the user doesn't know if it was unref'd or not

aggregator: add dispose/finalize functions

Add functions to be able to cleanup the mutex/cond and pending buffers
on the aggregator and on its pad

Release 1.3.90

Update .po files

interaudiosrc: Fail in set_caps() if the caps can't be parsed

tests: camerabin: fix caps leak in test

The gst_query_set_caps_result doesn't take ownership of caps

audiovisualizer: fix caps leaks

Fix leak of caps event and of caps objects when setting caps on
sink and src pads

kate: unref events when freeing events list

Instead of just feeing the queue structs

kateenc: fix segment event leak

gst_event_replace increments the refcount

mpegts: atsc: avoid calling g_convert with 0-sized input

Avoids an assertion. The cached string will be assigned an empty string
as its value when size is 0.

codecparsers: h264: fix memory leak in GstH264PPS.

The gst_h264_parse_pps() function dynamically allocates the slice
group ids map array, so that needs to be cleared before parsing a
new PPS NAL unit again, or when it is no longer needed.

Likewise, a clean copy to the internal NAL parser state needs to be
performed so that to avoid a double-free corruption.

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

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

codecparsers: h264: fix typo in GstH264VUIParams description.

codecparsers: h264: add support for Recovery Point SEI message.

The recovery point SEI message helps a decoder in determining if the
decoding process would produce acceptable pictures for display after
the decoder initiates random access or after the encoder indicates
a broken link in the coded video sequence.

This is not used in the h264parse element, but it could help debugging.

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

codecparsers: h264: add nal_reader_skip_long() helper.

Add nal_reader_skip_long() helper function to allow an arbitrary number
of bits to be skipped. The former nal_reader_skip() function is too
limited to the actual cache size.

Use this new function to simplify gst_h264_parser_parse_sei_message()
default case, that skips unsupported payloads.

v2: made args consistent from header to source file.

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

videoaggregator: fix a refcount error when keeping the buffer

We take a ref on the pad's buffer at the beginning so we need to
unref when we are done in all cases.

codecparsers: vp8: move up built-in range decoder private data.

Use the first _gst_reserved[] slot to hold the built-in range decoder
private data. The first slot was formerly the buffer size, which was
then promoted to semi-public namespace when it got integrated as git
commit 2940ac6.

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

glcolorconvert: free pixel swizzling information

glmixer: remove redundant/unused lock

Use the GstObject lock instead

glmixer: remove redundant creation and free of GPtrArrays

Also plugs a memory leak

glmixer: plug a memory leak for the caps

gst_query_set_caps_result() takes a ref on the caps

aggregator: plug a memory leak of the srccaps

glmixer: intersect with the filtercaps if available

gl: update gles2 compatability header

mpegts: Unify API : GstMpegts / gst_mpegts_ / GST_*_MPEGTS_

It was previously a mix and match of both variants, introducing just too much
confusion.

The prefix are from now on:
* GstMpegts for structures and type names (and not GstMpegTs)
* gst_mpegts_ for functions (and not gst_mpeg_ts_)
* GST_MPEGTS_ for enums/flags (and not GST_MPEG_TS_)
* GST_TYPE_MPEGTS_ for types (and not GST_TYPE_MPEG_TS_)

The rationale for chosing that is:
* the namespace is shorter/direct (it's mpegts, not mpeg_ts nor mpeg-ts)
* the namespace is one word under Gst
* it's shorter (yah)

mpegtssection: Free data in error cases

Data is marked as "transfer full", if we have an error we need to
free it.

glfilter: pass through the allocation query when in passthrough mode

Otherwise two GL elements on either side will fail to use the same
GL context.

glfilter: prefer passthrough for non-sysmem caps

glshader: enable glshader on GLES2

glcolorscale: do passthrough on same caps

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

glfilter: handle NULL decide_query which means passthrough

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

glfilter: prepend intersection to src caps

Prefer to stay in the same memory space if possible.
Also it let a chance to do passthrough.

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

gl: enable glvideomixer on GLES2

dvbsrc: add copyright/license stub to header

dvbsrc: documentation fixes

glvideomixer: bas output width/height on the pad properties

Allows automatic negotiation of the size in the following case:
gst-launch-1.0 glvideomixer name=m sink_0::xpos=0 sink_1::xpos=320 ! glimagesink \
    videotestsrc ! m. \
    videotestsrc pattern=1 ! m.

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

glvideomixer: don't clobber already allocated shader

curls?ftpsink: Fix memory leaks due to new error handling

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