On 10/11/14 08:10, Vittorio Giovara wrote:
On Sun, Nov 9, 2014 at 12:29 PM, Luca Barbato <[email protected]> wrote:
On 09/11/14 08:48, Vittorio Giovara wrote:

From: Michael Niedermayer <[email protected]>

CC: [email protected]
Bug-Id: CID 733706
---
   libavcodec/tak_parser.c | 3 ++-
   1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libavcodec/tak_parser.c b/libavcodec/tak_parser.c
index 295df24..5c03ec1 100644
--- a/libavcodec/tak_parser.c
+++ b/libavcodec/tak_parser.c
@@ -67,7 +67,8 @@ static int tak_parse(AVCodecParserContext *s,
AVCodecContext *avctx,
                                              buf_size);
               const uint8_t *tmp_buf = buf;

-            ff_combine_frame(pc, END_NOT_FOUND, &tmp_buf, &tmp_buf_size);
+            if (ff_combine_frame(pc, END_NOT_FOUND, &tmp_buf,
&tmp_buf_size) != -1)
+                return AVERROR(ENOMEM);
               consumed += tmp_buf_size;
               buf      += tmp_buf_size;
               buf_size -= tmp_buf_size;


It is used again in the file and the common usage of that function seems
different.

So what you propose?

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

Reply via email to