[FFmpeg-devel] [PATCH] lavf/matroska: comment a missing error check.

Nicolas George george at nsup.org
Fri Dec 23 22:39:35 EET 2016


Signed-off-by: Nicolas George <george at nsup.org>
---
 libavformat/matroskaenc.c | 1 +
 1 file changed, 1 insertion(+)


I do not have time to fix this, but I got a few segfaults here, so it needs
to be fixed. Either push the patch or, better push an actual fix.


diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c
index 827d7550c2..8e536f6774 100644
--- a/libavformat/matroskaenc.c
+++ b/libavformat/matroskaenc.c
@@ -351,6 +351,7 @@ static void end_ebml_master_crc32(AVIOContext *pb, AVIOContext **dyn_cp, Matrosk
 
     if (pb->seekable) {
         size = avio_close_dyn_buf(*dyn_cp, &buf);
+        /* FIXME missing error check */
         if (mkv->write_crc && mkv->mode != MODE_WEBM) {
             skip = 6; /* Skip reserved 6-byte long void element from the dynamic buffer. */
             AV_WL32(crc, av_crc(av_crc_get_table(AV_CRC_32_IEEE_LE), UINT32_MAX, buf + skip, size - skip) ^ UINT32_MAX);
-- 
2.11.0



More information about the ffmpeg-devel mailing list