[vlc-commits] svcdsub: Don't store unused values; skip over them

[Hugo Beauzée-Luyssen]

vlc | branch: master | Hugo Beauzée-Luyssen <hugo at beauzee.fr> | Thu Nov 23 16:37:45 2017 +0100| [b2b4c42ccbe54759ea325f914fecc4cd42985869] | committer: Hugo Beauzée-Luyssen

svcdsub: Don't store unused values; skip over them

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=b2b4c42ccbe54759ea325f914fecc4cd42985869
---

 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 */