[Gnash-commit] gnash ChangeLog libbase/FLVParser.cpp

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  08/05/10 16:51:07

Modified files:
        .              : ChangeLog 
        libbase        : FLVParser.cpp 

Log message:
        (parseNextTag): don't log_error if we just reached EOF outside any 
tag...

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.6579&r2=1.6580
http://cvs.savannah.gnu.org/viewcvs/gnash/libbase/FLVParser.cpp?cvsroot=gnash&r1=1.32&r2=1.33

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.6579
retrieving revision 1.6580
diff -u -b -r1.6579 -r1.6580
--- ChangeLog   10 May 2008 16:42:26 -0000      1.6579
+++ ChangeLog   10 May 2008 16:51:07 -0000      1.6580
@@ -1,5 +1,10 @@
 2008-05-10 Sandro Santilli <address@hidden>
 
+       * libbase/FLVParser.cpp (parseNextTag): don't log_error 
+         if we just reached EOF outside any tag...
+
+2008-05-10 Sandro Santilli <address@hidden>
+
        * libbase/FLVParser.{cpp,h}: rename parseNextFrame to
          parseNextTag, as that's really what the function does;
          fix parsing of audio/video flag, add parsing of FLV version.

Index: libbase/FLVParser.cpp
===================================================================
RCS file: /sources/gnash/gnash/libbase/FLVParser.cpp,v
retrieving revision 1.32
retrieving revision 1.33
diff -u -b -r1.32 -r1.33
--- libbase/FLVParser.cpp       10 May 2008 16:42:26 -0000      1.32
+++ libbase/FLVParser.cpp       10 May 2008 16:51:07 -0000      1.33
@@ -581,7 +581,9 @@
        int actuallyRead = _lt.read_bytes(tag, 12);
        if ( actuallyRead < 12 )
        {
+               if ( actuallyRead )
                log_error("FLVParser::parseNextTag: can't read tag info (needed 
12 bytes, only got %d)", actuallyRead);
+               // else { assert(_lt.get_eof(); } ?
                _parsingComplete=true;
                return false;
        }