[FFmpeg-devel] [PATCH v2 2/2] avcodec/libx264: return error if unknown picture type encountered

[Limin Wang]

On Fri, Jan 17, 2020 at 10:09:47PM +0100, Michael Niedermayer wrote:
> On Fri, Jan 17, 2020 at 12:48:48PM +0800, lance.lmwang at gmail.com wrote:
> > From: Limin Wang <lance.lmwang at gmail.com>
> > 
> > Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
> > ---
> >  libavcodec/libx264.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c
> > index ca8f6c0..4fef0be 100644
> > --- a/libavcodec/libx264.c
> > +++ b/libavcodec/libx264.c
> > @@ -471,7 +471,8 @@ static int X264_frame(AVCodecContext *ctx, AVPacket *pkt, const AVFrame *frame,
> >          pict_type = AV_PICTURE_TYPE_B;
> >          break;
> >      default:
> > -        pict_type = AV_PICTURE_TYPE_NONE;
> > +        av_log(avctx, AV_LOG_ERROR, "Unknown picture type encountered.\n");
> > +        return AVERROR_EXTERNAL;
> >      }
> 
> its ctx not avctx, this wont build
yes, it's my fault, I realized that my test branch did not compile the libx264 library.

> 
> [...]
> 
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> You can kill me, but you cannot change the truth.



> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


-- 
Thanks,
Limin Wang