On 2/24/12 6:28 PM, Ronald S. Bultje wrote:
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.


Ok.

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to