If realloc fails, the pointer is overwritten and the previously allocated
buffer is leaked, which goes against the expected behavior of keeping the
packet unchanged in case of error.

Michael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NJames Almer <jamrial@gmail.com>

This makes av_stream_add_side_data() consistent with av_packet_add_side_data().
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NJames Almer <jamrial@gmail.com>

Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NJames Almer <jamrial@gmail.com>

Signed-off-by: NJames Almer <jamrial@gmail.com>

Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NJames Almer <jamrial@gmail.com>

Functionally similar to av_packet_add_side_data(). Allows the use of an
already allocated buffer as stream side data.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NJames Almer <jamrial@gmail.com>

when parsing keyframe index metadata, list the message by trace log
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

Fixes compilation with Yasm 1.1.0 and older.
Signed-off-by: NJames Almer <jamrial@gmail.com>

Video doesn't exit ffmpeg on error anymore, and audio now prints an
error.

It makes no sense for this code to be inside the video filter loop, and
the frame sample aspect ratio is even set unconditionally a few lines
above.

This avoids unneeded computations
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Fixes remuxing apng streams coming from the apng demuxer, which sends extradata
during init.
Signed-off-by: NJames Almer <jamrial@gmail.com>

This reverts commit e0c6b320.

Said commit changed the behavior of the demuxer and decoder in a non
backwards compatible way.
Demuxers should make extradata available at init if possible, and send
new extradata as side data within a packet if needed.

A better fix for the remuxing crash will follow.
Signed-off-by: NJames Almer <jamrial@gmail.com>

* commit '8d07e941':
  FATE: add a test of H.264 SEI recovery in an intra refresh stream

Our H264 decoder drops 3 frames from the beginning of the stream, but
all frames after those match, hence the difference in the fate test.
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '46278ec9':
  mp3enc: write trailing padding

Noop, we have our own implementation for mp3 gapless.
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit 'd60c2d52':
  mp3dec: read the initial/trailing padding from the LAME tag

Noop, we have our own implementation for mp3 gapless tags.
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '7a763714':
  libopenh264enc: Simplify init by setting FF_CODEC_CAP_INIT_CLEANUP
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '2d097c16':
  libopenh264enc: Return a more sensible error code in some init failure paths
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '36b380dc':
  libopenh264dec: Simplify the init thanks to FF_CODEC_CAP_INIT_CLEANUP being set
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit 'd0b1e604':
  libopenh264dec: Fix cleanup if the init failed early
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '61cb9fac':
  mov: fix stream extradata_size allocation
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '0b1bd1b2':
  lavd: Drop unneeded av_init_packet()s
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '390b95b8':
  fate: Add a mixed NAL coding sample
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

- Restored alphabetical order.
- Enhanced sections aac_adtstoasc, dca_core, h264_mp4toannexb.
- Added sections hevc_mp4toannexb and vp9_superframe.
- Renamed (if required) and filled previously empty sections
  mjpegadump, mov2textsub/text2movsub, mp3decomp, and
  remove_extra.
- Fixes ticket #3198.
Signed-off-by: NMoritz Barsnick <barsnick@gmx.net>
Signed-off-by: NLou Logan <lou@lrcd.com>

Fix trac issue #5957.

Fixes division by 0
This is similar to how avg_frame_rate is checked elsewhere
Fixes: 6d24add0455f41b1b45b7ba615cd46f3/asan_generic_dc34c3_5480_0a2ef411cae999b9871ed71a2e481b71.mov

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

This matches the other branch
Fixes out of array read
Fixes: 4d142ca76d39fe685effcf5017098723/asan_heap-oob_31ae824_8611_348fdb64f9009b63c8a8eae9a0e497c5.mkv

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

* commit 'cbbb4040':
  fate: Restore order of h264 entries
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '61bd0ed7':
  h264: Log more information about invalid NALu size
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '7b1ae0e7':
  checkasm/arm: preserve the stack alignment checkasm_checked_call
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '80fbb7be':
  checkasm: vp8.mc: initialize the full src buffer after ec325742Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '17c99b61':
  h2645_parse: handle embedded Annex B NAL units in size prefixed NAL units

This commit is a noop, see a9bb4cf8Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit 'a8cbe5a0':
  h264_ps: export actual height in MBs as SPS.mb_height
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '99cf9433':
  d3d11va: don't keep the context lock while waiting for a frame
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '2866d108':
  vp8dsp: Remove the comment saying that the height is equal to the width
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>

* commit '8c816c0c':
  checkasm/arm: align the clobber check data properly for ldrd
Merged-by: NHendrik Leppkes <h.leppkes@gmail.com>