Skip to content

V
vlc
Commit b1946d46 authored by Francois Cartegnie's avatar Francois Cartegnie
Browse files
Options

demux: ogg: don't double read 

wrong fix for cid #1048729
bs_read in __MAX macro forwards read pointer twice
parent f09ddd30
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 1 deletion
modules/demux/ogg.c
View file @ b1946d46
......@@ -2183,7 +2183,8 @@ static void Ogg_ReadTheoraHeader( logical_stream_t *p_stream,
bs_read( &bitstream, 8 ); /* y offset */
i_fps_numerator = bs_read( &bitstream, 32 );
i_fps_denominator = __MAX( bs_read( &bitstream, 32 ), (uint32_t) 1 );
i_fps_denominator = bs_read( &bitstream, 32 );
i_fps_denominator = __MAX( i_fps_denominator, 1 );
bs_read( &bitstream, 24 ); /* aspect_numerator */
bs_read( &bitstream, 24 ); /* aspect_denominator */
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment