avcodecmap: Don't allocate dummy codec data anymore, it causes crashes nowadays

Multithreaded encoders are going to free this dummy codec data twice, e.g.
with this pipeline
  gst-launch-1.0 videotestsrc num-buffers=40 ! \
    videoconvert ! avenc_mjpeg ! fakesink

avvidenc: Pass the correct user_data to gst_buffer_new_wrapped_full()

Fixes crash with e.g.

  gst-launch-1.0 videotestsrc num-buffers=40 ! \
    videoconvert ! avenc_huffyuv ! fakesink

libav: G_CONST_RETURN is deprecated since GLib 2.30

Add configure check for GLib for consistency with other modules

avviddec: only free config when pool doesn't take ownership

Since gst_buffer_pool_set_config() takes ownership of the config structure,
it is only necessary to free the structure before using it when the true
branch of if (gst_buffer_pool_config_validate_params) hasn't run.

gst_buffer_pool_set_config() always takes ownership of the structure
regardless of success or failure. Which means the return, checked with
if (!working_pool), has no relation to the state of the structure.

avviddec: fix memory leak

Free config before overwriting.

CID #1320708

Back to development

Release 1.6.0

Release 1.5.91

avvidec: increase default alignment to 32 bytes

Change default alignment from 16 to 32 bytes, which fixes crashes
when decoding H.265 using AVX2-based decoder code paths and when
using ximagesink/glimagesink.

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

avvideodec: ensure required mem alignment fixing avdec_h265 crashes with ximagesink/glimagesink

Make sure the alignment requirement in GstAllocationParams
matches the GstVideoAlignment requirements. This fixes
issues with avdec_h265 crashing in the avx2 code path when
used with playbin and ximagesink/glimagesink as videosink.

The internal video pool would allocate buffers with an
alignment of 15 even though GstVideoAlignment specified
a stride_align requirement of 31 (which comes from ffmpeg).

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

libav: Update to ffmpeg n2.8

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

avvid/audenc: Set some tags in the output for downstream's info

Add the codec name and bitrate into the output for informational
purposes. Bitrate in particular is now used by flvmux to set
videodatarate and audiodatarate in the resulting stream

avviddec: Fix pool reallocation logic

Some check where incorect and also unsafe. The only reliable information
in get_buffer2 is the picture width/height really. The side effect is
that the width/height of the internal pool endup padded, so when we
switch we also need to switch to the a new width/height, hence we save
the pool info.

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

Release 1.5.90

aviddec: Re-enable direct rendering

This is achieved by using a tempory internal pool. We can then switch to a
downstream pool if the downstream pool buffer have matching strides.

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

avvidenc: use template subset check for accept-caps

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

avaudenc: use pad template caps for accept-caps

use template subset check for accept-caps

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

avaudenc: remove useless getcaps method

It just calls the exact same function as the default handler

avvidenc: remove useless getcaps method

It just calls the exact same function as the default handler

avauddec: let audiodecoder call the default pad query handler

Avoids repeating the same handling in many decoders

avviddec: let videodecoder call the default pad query handler

Avoids repeating the same handling in many decoders

libav: Update to n2.7.2

avviddec: use template caps intersection for accept-caps

Avoid doing downstream caps queries when accept-caps should just
do a shallow caps check on the element itself

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

avauddec: use template caps intersection for accept-caps

Avoid doing downstream caps queries when accept-caps should just
do a shallow caps check on the element itself

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

Map ffmpeg metadata to GStreamer tags

Update to the metadata API ffmpeg has had in
place for a long time now, and reenable output
of GStreamer tags from the demuxer.

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

avviddec: Remove unused force parameter

This parameter has been always false for a long time.

avviddec: Use caps-like size in get_buffer2()

The size in the AVFrame in get_buffer2 don't match the output size,
instead they match ffmpeg's memory requirements, so we can't compare
them from the values of the output AVFrame. Those are comparable to
the values in the passed AVCodecContext.

avviddec: Reset current_dr on close and when not chosing it

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

avviddec: Disable direct rendering before a frame has been produces

ffmpeg doesn't provide the final's image width & height in the get_buffer2()
callback, so it's not possible to create an output state for GstVideoDecoder
at this stage. So only try to do direct rendering if the buffer pool has already
been negotiated based on the final decoded size.

This partially reverts the effects of 2e621f8db

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

Revert "For static builds, add the ffmpeg library directories to the linker flags"

This reverts commit ac3437153dc6cb3d82cdda0d1f60ffc5600fdf67.

Doesn't actually make sense as it will put the (uninstalled) library paths
into the installed .la files. How does this all work?

For static builds, add the ffmpeg library directories to the linker flags

avviddec: Get code out of loop

Code was executed only on the first iteration, so just pull it out
of the loop entirely. This makes it clear it has nothing to do with the loop.

avviddec: Only create the AVBuffer once the stride is validated

If it is created earlier and the stride is invalid, then the frame
will be freed and it won't be possible to use it in the fallback path.
Not doing this causes a segfault because it will try to use
already freed memory.

avviddec: Re-enable direct rendering

avauddec: Read channels from the channels field

If there is no layout, just read the channel count from the
channels field.

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

avviddec: Ignore negotiation error on shutdown

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

avauddec: Access field using accessors

Those fields are documented to only be safe to access using
accessors as their position is not part of the ABI.

Add support for musepack (mpc) sv8 audio

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

avviddec: fix slice_offset memory leak

In case of real videos, slice_offset is being allocated,
but the same is not being freed.

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

Update mailing list in doap file

Disable iconv support to fix Windows build

It's needed only for subtitle charset conversion, and we don't
use the ffmpeg subtitle support anyway.

Also disable d3d11va and dxva2 support, we don't use the hardware
codec support.

Automatic update of common submodule

From f74b2df to 9aed1d7

Fix OSX build: Explicitly disable VDA h264 decoder

VideoDecodeAcceleration framework is deprecated in 10.11, and currently
cuases linker errors when compiling on OSX. Oddly, --disable-hwaccels
did not also disable h264_vda already.

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

Make the LGPL build the default and require --enable-gpl to build a GPL licensed version

avmux: Blacklist some subtitle and adaptive streaming muxers

Switch from libav v11.4 to ffmpeg n2.7.1

ffmpeg seems to be the one of the two forks, which is most widely used by
Linux distributions and in general. Also Google is using it for e.g. Chrome
and has engineers working on finding and fixing security issues in it.

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

avaudenc: Remember how many samples the codec consumed for telling the base class

avauddec: Ensure input buffers have FF_INPUT_BUFFER_PADDING_SIZE padding, which is required by avcodec_decode_audio4 ()

avmux: Ignore stream_segment,ssegment muxer that is provided by ffmpeg

The comma confuses our code, e.g. it's not a valid element name or structure
name.

avvidenc: Fix compiler warning

gstavvidenc.c: In function 'gst_ffmpegvidenc_flush_buffers':
gstavvidenc.c:733:7: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement]
       GstFFMpegVidEncClass *oclass =
       ^
cc1: all warnings being treated as errors

avvidenc: Set AVFrame flags for interlacing per frame and not in set_format()

They will get overridden by av_frame_unref() with the defaults.

av{aud,vid}enc: Create wrapped output buffers with GST_MEMORY_FLAG_READONLY

libav might still have references to the buffers itself and uses a writability
system similar to ours based on reference counts.

avvidenc: Properly wrap and free the output AVPackets generated during draining

avvidenc: Make sure to keep input data alive until libav is done with it

avaudenc: Make sure to keep input data alive until libav is done with it

av{aud,vid}enc: Stop using deprecated AVFrame APIs

avauddec: Remove custom buffer allocation function

libav always uses planar audio formats nowadays, not much use in
us trying to allocate anything here until we add support for planar
aka non-interleaved audio formats at least in audioconvert.

avauddec: Use undeprecated AVFrame API

avviddec: Free frame before the codec is freed

Who knows, maybe freeing the frame calls cleanup inside the codec.

avviddec: Use undeprecated AVFrame API and unref frame once we're done with it

avauddec: Negotiate based on the AVFrame instead of just the AVCodecContext

avviddec: Use the new get_buffer2() API instead of the deprecated APIs

And do the frame reference counting by adding AVBufferRefs to the frame.

avviddec: Also proxy the data/size pointers for our wrapper AVBufferRef

codecmap: add vp9 mapping

avviddec: Wrap the original AVBufferRef in our own buffer for the destroy function

Just adding a dummy buffer at the very end might not be enough as there
already might be too many buffers.

avviddec: libav will already copy the reordered_opaque pointer for us

If we do it ourselves, it might get the wrong value if our assumptions are
broken by libav at a later time.

avviddec: Negotiate based on the AVFrame information, not the context information

The context contains the information from the latest input frame, we're
however interested in the information from the latest output frame. As we have
to negotiate for the buffer that is about to come next.

This should fix some crashes that happened when both information got out of
sync. If that happens now, we will do fallback allocation until the output
is renegotiated too.

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

Back to development

Release 1.5.2

Automatic update of common submodule

From 6015d26 to f74b2df

avcodec: Check against codec format list

There exist few formats (deprecated though) used by mjpeg decoder
and encoder that maps to the same GStreamer format. To properly
pick the right format, also lookup each Codec list before accepting
the format. This fixes error when trying to use mjpeg encoder.

Note that this may results in faded colors. In fact, these special
format are meant to specify that this is full range YUV. Colorimetry
in gst-libav is not yet implemented, hence is ignored in general. So
I think it's fine to first fix the issue before addressing the missing
feature.

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

viddec: Replicate any 3D/multiview info from input caps onto output

cruft: add the obsolete tmpl dir to cruft-dirs

Automatic update of common submodule

From d9a3353 to 6015d26

Automatic update of common submodule

From d37af32 to d9a3353

Automatic update of common submodule

From 21ba2e5 to d37af32

Automatic update of common submodule

From c408583 to 21ba2e5

docs: remove variables that we define in the snippet from common

This is syncing our Makefile.am with upstream gtkdoc.

Automatic update of common submodule

From c8fb372 to c408583

Back to development

Release 1.5.1

avviddec: Release stream lock while calling avcodec_decode_video2()

It might call back into us from another thread and try to take the stream lock
again, e.g. to allocate a buffer.

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

avviddec: Post error message before returning a flow error

This is required.

check: Use GST_CHECK_MAIN () macro everywhere

Makes source code smaller, and ensures we go through common initialization
path (like the one that sets up XML unit test output ...)

Make top-level 'check-torture' and 'check-valgrind' targets functional

avcodecmap: Use video/x-cavs mime type instead of video/x-gst-av-cavs

More generic mime-type which can be used by other decoders than libav.

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

configure: don't disable Chinese AVS (cavs) decoder

It enables GStreamer to decode Chinese AVS video streams.
Also fix indentation.

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

avauddev: Unref decoded AVFrame after we're done with it

Otherwise we might leak some memory, like all compressed data when using
avdec_ac3.

libav: Update to v11.4

Update .gitignore

configure: bump automake requirement to 1.14 and autoconf to 2.69

This is only required for builds from git, people can still
build tarballs if they only have older autotools.

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

Automatic update of common submodule

From bc76a8b to c8fb372

avdemux: resurrect some flow return handling

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

libav: Update to v11.3

avdemux: fix double semicolon

libav: Add support for 10 bit planar AYUV formats

libav: Change from deprecated PIX_FMT_* to AV_PIX_FMT_*

avviddec: Don't copy padding if handling a frame failed

We will return an error right afterwards anyway.

avviddec: If a flow return during draining is not OK, don't drain any further