]> git.sesse.net Git - ffmpeg/commit
avcodec/movtextenc: fix writing to bytestream on BE arches
authorAndriy Gelman <andriy.gelman@gmail.com>
Wed, 14 Oct 2020 04:38:22 +0000 (00:38 -0400)
committerAndriy Gelman <andriy.gelman@gmail.com>
Fri, 16 Oct 2020 02:27:37 +0000 (22:27 -0400)
commitd4c46dc32856bd9c7c7ab29ee727676c7855fa1c
tree5f1223eebc2a6a71f7b448460b0d95d6f710ed99
parent8e58db7348b79b2f5d460632932e4f9a8e6312ed
avcodec/movtextenc: fix writing to bytestream on BE arches

Fixes fate-binsub-movtextenc on PPC64

Currently tags are written in reverse order on BE arches. This is fixed
by using MKBETAG() and AV_RB32() to be arch agnostics.

Also s->font_count is of type int. On BE arches with 32bit int,
count = AV_RB16(&s->font_count) will read two most significant bytes
instead of the least significant bytes. This is fixed by assigning
s->font_count to count first.

The final change is modifying the type of len. On BE arches
the most significant byte of the int was written instead of the least
significant byte.

Signed-off-by: Andriy Gelman <andriy.gelman@gmail.com>
libavcodec/movtextenc.c