Fixes: Timeout
Fixes: 64479/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_EIGHTBPS_fuzzer-5434435386081280

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpegSigned-off-by: Michael Niedermayer <michael@niedermayer.cc>

Reviewed-by: Stefano Sabatini <stefasab@gmail.com>
Reviewed-by: Marth64 <marth64@proxyid.net>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

Signed-off-by: Marth64 <marth64@proxyid.net>

Signed-off-by: Marth64 <marth64@proxyid.net>

Signed-off-by: Marth64 <marth64@proxyid.net>

I am willing to maintain these into the future as needed.
Signed-off-by: Marth64 <marth64@proxyid.net>

Signed-off-by: Marth64 <marth64@proxyid.net>

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

Speeds up decoding and also reduces binary size.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

Improves performance by about 30%.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

Use a small LUT instead. Improves performance.
Reviewed-by: Marton Balint <cus@passwd.hu>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

Partially fixes ticket #10395.
Signed-off-by: Marton Balint <cus@passwd.hu>

Tested-by: Steven Liu <lingjiujianke@gmail.com>
Signed-off-by: James Almer <jamrial@gmail.com>

src/libavfilter/internal.h:255:45: note: passing argument to parameter 'filter' here
int ff_filter_config_links(AVFilterContext *filter);
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

This was missed in b800327f.
Signed-off-by: Martin Storsjö <martin@martin.st>

It never makes sense for this function to be called by users.

It never makes sense for this function to be called by users.

Cf. #10857
Signed-off-by: Anton Khirnov <anton@khirnov.net>

If the time code side data is overridden by the packet level, we also
make sure not to update `p->metadata` to a mismatched timecode.

Signed-off-by: Anton Khirnov <anton@khirnov.net>

We only need to consider side data types that may possibly come from the
packet.
Signed-off-by: Anton Khirnov <anton@khirnov.net>

Also fixes a memory leak where the side data was previously not properly
cleaned up on OOM.
Signed-off-by: Anton Khirnov <anton@khirnov.net>

If the time code side data is overridden by the packet level, we also
make sure not to update `out->metadata` to a mismatched timecode.
Signed-off-by: Anton Khirnov <anton@khirnov.net>

Needed by following commit.

If the time code side data is overridden by the packet level, we also
make sure not to update `out->metadata` to a mismatched timecode.

This function was already ignoring OOM errors.

The signature of these wrappers is more complicated due to a need to
distinguish between "failed allocating side data" and "side data was
already present".
Signed-off-by: Anton Khirnov <anton@khirnov.net>

This and the following commits fix #10857

Native access is from the code that declared the options, foreign access
is from code that is using the options. Forbid foreign access to
AVOption.offset/default_val, for which there is no good reason, and
which should allow us more freedom in extending their semantics in a
compatible way.

Will be useful in following commits.