[FFmpeg-devel] [PATCH 2/2] avformat/mxfenc: support XAVC long gop
Hendrik Leppkes
h.leppkes at gmail.com
Wed Apr 10 02:05:55 EEST 2019
On Wed, Apr 10, 2019 at 12:21 AM Baptiste Coudurier
<baptiste.coudurier at gmail.com> wrote:
> + return 0;
> + }
> + init_get_bits(&gb, tmp, tmp_size*8);
> + ret = avpriv_h264_decode_seq_parameter_set(&gb, (AVCodecContext*)s, &ps, 0);
The AVCodecContext cast looks like a recipe for disaster. The function
is internal to the H264 decoder, so if it at some point decides to
actually access the AVCodecContext it receives, or even worse, the
priv_data element in it, expecting a H264Context, then everything goes
to hell.
Just another sign why exporting functions internal from a decoder is
just an overall bad idea...
- Hendrik
More information about the ffmpeg-devel
mailing list