[FFmpeg-devel] [PATCH] avcodec/h264_pc: Return error if pps has FMO parameters

Andriy Gelman andriy.gelman at gmail.com
Thu Feb 13 01:39:20 EET 2020


On Sat, 08. Feb 21:30, James Almer wrote:
> On 2/8/2020 7:20 PM, Andriy Gelman wrote:
> > From: Andriy Gelman <andriy.gelman at gmail.com>
> > 
> > FMO is not supported and fields related to FMO are not parsed, meaning
> > that any fields which follow will be corrupt.
> > ---
> >  libavcodec/h264_ps.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c
> > index edbaa96b64e..80216854079 100644
> > --- a/libavcodec/h264_ps.c
> > +++ b/libavcodec/h264_ps.c
> > @@ -794,6 +794,8 @@ int ff_h264_decode_picture_parameter_set(GetBitContext *gb, AVCodecContext *avct
> >      if (pps->slice_group_count > 1) {
> >          pps->mb_slice_group_map_type = get_ue_golomb(gb);
> >          av_log(avctx, AV_LOG_ERROR, "FMO not supported\n");

> 
> Should probably be avpriv_report_missing_feature() instead. Look right
> above this chunk for an usage example of this function.
> 

Thanks, I've resent with this change.

-- 
Andriy


More information about the ffmpeg-devel mailing list