[vlc-commits] demux: ogg: fix integer overflow

Francois Cartegnie git at videolan.org
Sun Oct 13 15:55:59 CEST 2013


vlc/vlc-2.1 | branch: master | Francois Cartegnie <fcvlcdev at free.fr> | Fri Oct 11 13:08:31 2013 +0200| [0d9790599fbabffe5440efd3b578f9864be25eb7] | committer: Jean-Baptiste Kempf

demux: ogg: fix integer overflow

(cherry picked from commit 546da0729ccdd84cb0914a80703a276e448569f9)
Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

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

 modules/demux/oggseek.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/modules/demux/oggseek.c b/modules/demux/oggseek.c
index 3c65e5f..56f2635 100644
--- a/modules/demux/oggseek.c
+++ b/modules/demux/oggseek.c
@@ -857,8 +857,8 @@ int64_t oggseek_read_page( demux_t *p_demux )
     demux_sys_t *p_ogg = p_demux->p_sys  ;
     uint8_t header[PAGE_HEADER_BYTES+255];
     int i_nsegs;
-    int i_in_pos;
     int i;
+    int64_t i_in_pos;
     int64_t i_result;
     int i_page_size;
     char *buf;
@@ -912,8 +912,8 @@ int64_t oggseek_read_page( demux_t *p_demux )
 
     if ( ogg_sync_pageout( &p_ogg->oy, &p_ogg->current_page ) != 1 )
     {
-        msg_Err( p_demux , "Got invalid packet, read %"PRId64" of %i: %s",i_result,i_page_size,
-                 buf );
+        msg_Err( p_demux , "Got invalid packet, read %"PRId64" of %i: %s %"PRId64,
+                 i_result, i_page_size, buf, i_in_pos );
         return 0;
     }
 



More information about the vlc-commits mailing list