diff mbox series

[FFmpeg-devel,09/11] avformat/segafilmenc: Simplify writing two bytes

Message ID 20200718001928.10603-8-andreas.rheinhardt@gmail.com
State Accepted
Commit 37ec33f42c42f0058b3271d79096dbc215b1902e
Headers show
Series [FFmpeg-devel,01/11] avformat/webmdashenc: Fix segfault when no filename is given when live | expand

Checks

Context Check Description
andriy/default pending
andriy/make success Make finished
andriy/make_fate success Make fate finished

Commit Message

Andreas Rheinhardt July 18, 2020, 12:19 a.m. UTC 
Use avio_wb16() instead of avio_write() to write two zero bytes.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
---
 libavformat/segafilmenc.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/libavformat/segafilmenc.c b/libavformat/segafilmenc.c
index e144da4e68..d935caf00d 100644
--- a/libavformat/segafilmenc.c
+++ b/libavformat/segafilmenc.c
@@ -119,7 +119,6 @@  static int film_write_packet(AVFormatContext *format_context, AVPacket *pkt)
         if (encoded_buf_size != pkt->size && (pkt->size % encoded_buf_size) != 0) {
             avio_write(pb, pkt->data, pkt->size);
         } else {
-            uint8_t padding[2] = {0, 0};
             /* In Sega Cinepak, the reported size in the Cinepak header is
              * 8 bytes too short. However, the size in the STAB section of the header
              * is correct, taking into account the extra two bytes. */
@@ -127,7 +126,7 @@  static int film_write_packet(AVFormatContext *format_context, AVPacket *pkt)
             metadata->size += 2;
 
             avio_write(pb, pkt->data, 10);
-            avio_write(pb, padding, 2);
+            avio_wb16(pb, 0);
             avio_write(pb, &pkt->data[10], pkt->size - 10);
         }
     } else {