Mercurial > libavformat.hg

changeset 5094:7d589bcd9b1f libavformat

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Check for packet_length 0, it is already treated as invalid by the padding check, but that resulted in a confusing/wrong error message.
author reimar
date Mon, 06 Jul 2009 17:08:32 +0000
parents d4cce49ac1f1
children 7f17519bfa37
files asfdec.c
diffstat 1 files changed, 1 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/asfdec.c	Sat Jul 04 08:02:01 2009 +0000
+++ b/asfdec.c	Mon Jul 06 17:08:32 2009 +0000
@@ -633,7 +633,7 @@
     DO_2BITS(asf->packet_flags >> 3, padsize, 0); // padding length
 
     //the following checks prevent overflows and infinite loops
-    if(packet_length >= (1U<<29)){
+    if(!packet_length || packet_length >= (1U<<29)){
         av_log(s, AV_LOG_ERROR, "invalid packet_length %d at:%"PRId64"\n", packet_length, url_ftell(pb));
         return -1;
     }