aboutsummaryrefslogtreecommitdiffstats
path: root/tests
diff options
context:
space:
mode:
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2019-12-12 16:53:17 +0100
committerMichael Niedermayer <michael@niedermayer.cc>2019-12-27 12:38:08 +0100
commit9bde6c6be0ea568f6ce646690616a0d4274095d4 (patch)
tree362e374b93ee0072be3469e478afc23a5a4524a6 /tests
parentb0d0d7e4d0581ccfeea37cc3fd0d3ca9d999fbfc (diff)
downloadffmpeg-9bde6c6be0ea568f6ce646690616a0d4274095d4.tar.gz
avformat/microdvd: Use \n instead of \0 to end file header
Up until now, the microdvd demuxer uses av_strdup() to allocate the extradata from a string; its length is set to strlen() + 1, i.e. including the \0 at the end. Upon remuxing, the muxer would simply copy the extradata at the beginning, including the \0. This commit changes this by not adding the \0 to the size of the extradata; the muxer now delimits extradata by inserting a \n. This required to change the subtitles-microdvd-remux FATE-test. Furthermore, the extradata is now allocated with zeroed padding. The microdvd decoder is not affected by this, as it didn't use the size of the extradata at all, but treated it as a C-string. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Reviewed-by: Paul B Mahol <onemda@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'tests')
-rw-r--r--tests/ref/fate/sub-microdvd-remuxbin416 -> 416 bytes
1 files changed, 0 insertions, 0 deletions
diff --git a/tests/ref/fate/sub-microdvd-remux b/tests/ref/fate/sub-microdvd-remux
index a71da99031..92ff233f56 100644
--- a/tests/ref/fate/sub-microdvd-remux
+++ b/tests/ref/fate/sub-microdvd-remux
Binary files differ