From b2b4c42ccbe54759ea325f914fecc4cd42985869 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Hugo=20Beauz=C3=A9e-Luyssen?= Date: Thu, 23 Nov 2017 16:37:45 +0100 Subject: [PATCH] svcdsub: Don't store unused values; skip over them --- modules/codec/svcdsub.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/modules/codec/svcdsub.c b/modules/codec/svcdsub.c index dcd545520d..e509effdbc 100644 --- a/modules/codec/svcdsub.c +++ b/modules/codec/svcdsub.c @@ -362,12 +362,13 @@ static void ParseHeader( decoder_t *p_dec, block_t *p_block ) { decoder_sys_t *p_sys = p_dec->p_sys; uint8_t *p = p_block->p_buffer; - uint8_t i_options, i_options2, i_cmd, i_cmd_arg; + uint8_t i_options, i_cmd; int i; p_sys->i_spu_size = GETINT16(p); i_options = *p++; - i_options2 = *p++; + // Skip over unused value + p++; if( i_options & 0x08 ) { p_sys->i_duration = GETINT32(p); } else p_sys->i_duration = 0; /* Ephemer subtitle */ @@ -388,7 +389,7 @@ static void ParseHeader( decoder_t *p_dec, block_t *p_block ) i_cmd = *p++; /* We do not really know this, FIXME */ - if( i_cmd ) {i_cmd_arg = GETINT32(p);} + if( i_cmd ) { p += 4; } /* Actually, this is measured against a different origin, so we have to * adjust it */