[1.16.2-1] Fix ASAN build error

Change-Id: I66d40c3a03ea13f52d64f60f9272e09a5a81b066
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>

avaudioenc: Fix build error for tv_profile

Change-Id: I18e05fcb24b6fd7d6974e317e94efbd54e0937ea

Merge branch 'tizen_gst_1.16.2' into tizen

Change-Id: I7329ca1d34ab878f5a2b78fdeb945520397ff93f

Merge branch 'upstream/1.16' into tizen_gst_1.16.2

Change-Id: I181a882310f8cd7315616dd623dbfb4c49fcd86e

arm: Fix SIGBUS on ARM when compiled with binutils 2.29

In binutils 2.29, the behavior of the ADR instruction changed so that 1 is
added to the address of a Thumb function (previously nothing was added). This
allows the loaded address to be passed to a BLX instruction and the correct
mode change will occur.

See: https://sourceware.org/bugzilla/show_bug.cgi?id=21458

By using adr with a label that isn't annotated as a thumb function,
we avoid the new behaviour in binutils 2.29 and get the same behaviour
as in prior releases, and as in other assemblers (ms armasm.exe,
clang's built in assembler) - an idea that Janne Grunau came up with.

Signed-off-by: Martin Storsjö <martin@martin.st>
[tizen] Fix crash in libgstlibav.so after toolchain upgrade

Change-Id: I5fe79ed4f9b37968421d3522677180c1984f9421
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>

Release 1.16.2

Update docs

avvidenc: Fix error propagation

Instead of returning the default return value (GST_FLOW_OK), actually
return an error one (res vs ret).

avdemux: Fix segmentation fault if long_name is NULL

Some plugins (like libcdio) registers empty long_name field. Calling strncmp on this field leads to a segmentation fault.

Signed-off-by: Kevin Joly <joly.kevin25@gmail.com>

avviddec: Fix huge leak caused by circular reference

AVBufferRef -> GstFFMpegVideoDecVideoFrame -> GstVideoCodecFrame -> AVBufferRef

Instead of holding additional ref there, set read-only which would not be
reused by ff_reget_buffer()

Fixes: https://gitlab.freedesktop.org/gstreamer/gst-libav/issues/63

avviddec: Enforce allocate new AVFrame per input frame

... if ffmpeg would reuse the allocated AVBuffer. Reused AVFrame by
the ffmpeg seems to break our decoding flow since the reused AVFrame
holds the initial opaque data (GstVideoCodecFrame in this case), so
we couldn't trace the our in/out frames.
To enforce get_buffer() call per output frame, hold another reference
to the AVBuffer in order to mark the AVBuffer as not writable.

Fixes: https://gitlab.freedesktop.org/gstreamer/gst-libav/issues/62

Release 1.16.1

Update docs

docs: don't include the type hierarchy

gtk-doc can't generate a type hierarchy when scanning gst-libav, and gtk-doc
1.30 onwards doesn't write a file if there is no type hierarchy (unlike previous
releases, which wrote an empty file).  This results in the build failing with
gtk-doc 1.30 onwards, so remove the type hierarchy section from the
documentation as it doesn't serve any purpose.

Fixes https://gitlab.freedesktop.org/gstreamer/gst-libav/issues/57

avvidenc: Correctly signal interlaced input to ffmpeg when the input caps are interlaced

Regression from 1e4529ced2dadbfed4ac10d639a45dbcb0660129

avvidenc: Also set the repeat_pict flag correctly and take the TFF flag from caps if available

autotools: add bcrypt to win32 libs

The included libav requires it now. Otherwise the builds fails with:

  CCLD     libgstlibav.la
build-i686-w64-mingw32/gst-libs/ext/.libs/libavutil.a(random_seed.o): In function `av_get_random_seed':
gst-libav-1.16.0/gst-libs/ext/libav/libavutil/random_seed.c:126: undefined reference to `BCryptOpenAlgorithmProvider@16'
gst-libav-1.16.0/gst-libs/ext/libav/libavutil/random_seed.c:129: undefined reference to `BCryptGenRandom@16'
gst-libav-1.16.0/gst-libs/ext/libav/libavutil/random_seed.c:130: undefined reference to `BCryptCloseAlgorithmProvider@8'
collect2.exe: error: ld returned 1 exit status

gstav: Use libavcodec util function for version check

The version of libavutil is printed in the log instead of libavcodec
because avutil_version() returns LIBAVUTIL_VERSION_INT. This can be confusing,
so we should be replace it with avcodec_version().

Disable hevc decoder

[Version] 1.12.2-1
[Profile] Common
[Issue Type] Update

Change-Id: Ibec96e900bfbcbfa6f87c32155209bac72ea53dc
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>

ci: use template from 1.16 branch

Release 1.16.0

Update docs

libav: Update to ffmpeg n4.1.3

Release 1.15.90

Update docs

avcfg: Override type of bitrate property from int64 to int

See https://gitlab.freedesktop.org/gstreamer/gst-libav/issues/41#note_142808

The switch to the new ffmpeg property system changed the type of the
bitrate property from int to int64, which potentially breaks many
existing applications at runtime as properties are usually set via
g_object_set().

As such, override the type to int until GStreamer 2.0.

avviddec: do not add 708 caption meta if already exists

(this is only used for CEA 708 raw data). another element
such as mpegvideoparse may have already added the meta.

avvidenc: pass Closed Caption metadata to libav

avauddec: fix decoding of APE and Cook audio

.. and other formats where ffmpeg gives us multiple
subframes per input frame.

Since we now support non-interleaved audio, we can't
just concat buffers any more. Also, audio metas won't
be combined when buffers are merged, so when we push
out the combined buffer we'll look at the meta describing
only the first subframe and think it covers the whole
frame leading to stutter/gaps in the output.

We could fix this by copying the output data into a new
buffer when we merge buffers, but that's suboptimal, so
let's add some API to GstAudioDecoder to push out subframes
and use that instead.

https://gitlab.freedesktop.org/gstreamer/gst-libav/issues/49

Back to development

Release 1.15.2

Update docs

avdemux: fix negative pts if start_time is bigger than the ts

The start time is supposed to be the ts of the first frame.
FFmpeg uses fractions to represent timestamps and the start time may use a
different base than the frame pts. So we may end up having the start
time bigger than the pts because of rounding when converting to gst ts.

See https://gitlab.freedesktop.org/gstreamer/gst-libav/issues/51
for details.

configure: Only disable dxva2 once

Update to ffmpeg n4.1.1

Release 1.15.1

Update docs

avcfg: Fix AVOptionRanges leak

It must be freed with av_opt_freep_ranges as documented.

Automatic update of common submodule

From ed78bee to 59cb678

libav: Update to ffmpeg n4.1

Add Gitlab CI configuration

This commit adds a .gitlab-ci.yml file, which uses a feature
to fetch the config from a centralized repository. The intent is
to have all the gstreamer modules use the same configuration.

The configuration is currently hosted at the gst-ci repository
under the gitlab/ci_template.yml path.

Part of https://gitlab.freedesktop.org/gstreamer/gstreamer-project/issues/29

codecmap: Add mapping for Voxware metasound

Allows actually decoding such streams :)

libav: Update to ffmpeg n4.0.3

Update common submodule location

Remove the git directory

Clone the code from gitlab

This fixes https://gitlab.freedesktop.org/gstreamer/gst-libav/issues/43

meson: Remove leftover debug logging

avcfg: Ensure that ternary operator always evaluates to int64

When building with MSVC, if the 3rd operator is a double, the entire
expression always promoted double, and is then cast to int64.

When TRUE, this evaluates to (gint64) (gdouble) (INT64_MAX)
which overflows to INT64_MIN on MSVC, but not on C99 compilers.

This causes us to fail the g_return_if_fail inside g_param_spec_int64
when built with MSVC.

avdemux: Expose IFF container support

This exposes support for the "iff" demuxer. This is a general purpose
format, and the reason to expose this now is to allow reading DSD data
(which is supported via a variant called DSDIFF).

More information at:
  https://wiki.multimedia.cx/index.php?title=IFF
  https://www.loc.gov/preservation/digital/formats/fdd/fdd000245.shtml

meson: Generate a pkgconfig file for gstlibav

This matches all other plugins in the other gstreamer repos. This is
also necessary for generating the correct libtool archive (.la) files
in Cerbero which are needed for static linking on Android and iOS.

meson: use new 'python' module instead of deprecated 'python3' one

https://github.com/mesonbuild/meson/pull/4169

meson: Don't export symbols from linked static libraries

We don't want to export any symbols from the ffmpeg static libraries
we link to when building inside Cerbero. In the Autotools build, we
pass -export-symbols-regex to libtool which ensures this for us.

avviddec: Remove unneeded check

frame is always valid in this function (but wasn't before the
refactoring from a few months ago).

CID #1439540

avcfg: Simplify code

The existence of 'opt' is checked, the remainder of the code can therefore
rely on it being valid.

CID #1439537

libav: update internal snapshot to ffmpeg n4.0.2

decoders: fix draining

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

avmux: Place pva case after generic case

In the function gst_ffmpeg_formatid_get_codecids() in the if / else if
construct the special case !strcmp (format_name, "pva") should be
handled before the generic case (plugin->audio_codec !=
AV_CODEC_ID_NONE) || (plugin->video_codec != AV_CODEC_ID_NONE)
This patch fixes the ordering.

I stumbled accorss this issue while adding a new format to
gst_ffmpeg_formatid_get_codecids()

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

avauddec: add support for decoding in non-interleaved layout

This removes the internal interleave loop and always negotiates
the native output layout of the libav decoder. Users can use
audioconvert to interleave if necessary.

Special care has been taken to leave the encoder unaffected by
the changes in avcodecmap, since GstAudioEncoder doesn't support
the non-interleaved layout yet.

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

libav: Fix symbol redefine build error

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

Remove obsolete #if 0-ed code

Remove swscale plugin code

This has been disabled for 5.5 years, time to remove it.

libav: Update to ffmpeg release/4.0 HEAD

avcfg: Properly initalize GValue

.. to the expected property value type.

avcfg: actually remove call to g_object_getv

(Sorry about that)

av*dec: Ignore decoding errors

We were ignoring these before the port to 4.0, interpreting them
as GST_FLOW_ERROR / GST_ELEMENT_ERROR causes check failures.

We should start using GST_*_DECODER_ERROR in latter commits,
for now simply restore the previous behaviour.

avcfg: use g_object_get_property, not g_object_getv

I simply confused those two, we didn't need g_object_getv
here anyway.

avcfg: ignore some generic properties

We expose profile, level and colorimetry / colorspaces through
caps.

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

avcfg: add skip support to overrides system

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

meson: add FFmpeg meson as a fallback

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

auddec: fix luck-based sinkpad access

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

Update ffmpeg version requirements

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

Remove obsolete scale and resample sources

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

configure: update libav configure args

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

libav: Update to n4.0.1 release

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

meson: stop ignoring deprecation warnings!

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

avvidenc: port to send_frame / receive_packet

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

avviddec: fix invalid alignment calculations.

avcodec_align_dimensions2 uses context->pix_fmt to make its
calculations, we thus need to make sure it is adequately set
when calling it.

Fixes:

gst-launch-1.0 videotestsrc ! video/x-raw, width=1920, height=1080 \
! avenc_mpeg4 ! avdec_mpeg4 ! xvimagesink

This showed invalid writes under valgrind, then segfault.

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

avaudenc: port to send_frame / receive_packet

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

avmux: fix remaining warnings

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

avauddec: fix remaining warnings

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

avauddec: port to new decoding API

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

avaudenc: further cleanup

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

avaudenc: install options generically

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

avviddec: fix signed/unsigned comparisons

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

avvidec: port to new decoding API

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

gstav: stop calling deprecated registration methods

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

auddec: stop using deprecated getters

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

av_oformat_next (deprecated) -> av_muxer_iterate

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

av_iformat_next (deprecated) -> av_demuxer_iterate

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

avauddec: remove obsolete version check

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

av_codec_next (deprecated) -> av_codec_iterate

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

avdemux: port to AVCodecParameters

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

avmux: port to AVCodecParameters

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

Remove all uses of AVPicture

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

avaudenc: remove obsolete setting of rc_strategy

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

Refactor avcfg / avvidenc

We were previously installing hardcoded properties for all
video encoders, refactor to instead use FFmpeg's AVOption API.

avvidenc still exposes a few properties related to the pass
mechanism: while the AVOption API allows specifying both passes
as flags at the same time, this is not practical in GStreamer's
context, where passes need to be run separately using a stats file.

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

avviddec: Stop using deperated EDGE API

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

Use AV_ namespace for all CODEC_ macro

The remaining use of CODEC_ are codec flags that has been moved into the
new codec private properties or have been deperated. Will be fixed in
later patches.

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

Use AV_ namespace for INPUT_BUFFER_PADDING_SIZE

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

Don't include deprecated avfiltergraph.h header

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

avdemux: fix memory leaks

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