Hi, On Fri, Feb 24, 2012 at 9:24 AM, Luca Barbato <[email protected]> wrote: > On 2/22/12 6:27 PM, Ronald S. Bultje wrote: >> On Fri, Feb 17, 2012 at 2:49 PM, Ronald S. Bultje<[email protected]> >> wrote: >>> >>> From: "Ronald S. Bultje"<[email protected]> >>> >>> --- >>> libavcodec/utils.c | 2 ++ >>> 1 files changed, 2 insertions(+), 0 deletions(-) >>> >>> diff --git a/libavcodec/utils.c b/libavcodec/utils.c >>> index 255406f..e0d2b3d 100644 >>> --- a/libavcodec/utils.c >>> +++ b/libavcodec/utils.c >>> @@ -553,6 +553,8 @@ int avcodec_default_reget_buffer(AVCodecContext *s, >>> AVFrame *pic){ >>> return s->get_buffer(s, pic); >>> } >>> >>> + assert(s->pix_fmt == pic->pix_fmt); >> >> Ping. > > Is it really impossible to get there?
Maybe, maybe not. If it happens, it crashes anyway (you can't copy a RGB24 source to a YUV420P dest; plane[1] is NULL). The assert just clarifies an assumption that is implicit. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
