[Gnash-commit] gnash ChangeLog server/swf/ASHandlers.cpp

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  06/06/29 00:02:32

Modified files:
        .              : ChangeLog 
        server/swf     : ASHandlers.cpp 

Log message:
        * server/ASHandlers.cpp (ActionWaitForFrame): added stack size check.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.438&r2=1.439
http://cvs.savannah.gnu.org/viewcvs/gnash/server/swf/ASHandlers.cpp?cvsroot=gnash&r1=1.17&r2=1.18

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.438
retrieving revision 1.439
diff -u -b -r1.438 -r1.439
--- ChangeLog   28 Jun 2006 23:27:44 -0000      1.438
+++ ChangeLog   29 Jun 2006 00:02:31 -0000      1.439
@@ -1,5 +1,6 @@
 2006-06-29 Sandro Santilli <address@hidden>
 
+       * server/ASHandlers.cpp (ActionWaitForFrame): added stack size check.
        * server/ASSound.cpp, server/ActionExec.cpp, server/Function.cpp,
        server/Global.cpp, server/MovieClipLoader.cpp, server/action.cpp,
        server/as_environment.cpp, server/as_environment.h,

Index: server/swf/ASHandlers.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/swf/ASHandlers.cpp,v
retrieving revision 1.17
retrieving revision 1.18
diff -u -b -r1.17 -r1.18
--- server/swf/ASHandlers.cpp   28 Jun 2006 23:27:45 -0000      1.17
+++ server/swf/ASHandlers.cpp   29 Jun 2006 00:02:31 -0000      1.18
@@ -613,6 +613,13 @@
        //
        // Since we don't load incrementally, just ignore this opcode.
 
+       if ( env.stack_size() < 1 )
+       {
+               log_error("Empty stack on ActionWaitForFrame. Bogus SWF?");
+               // yeah.. as if gnash was bugfree :)
+               return;
+       }
+
        env.drop(1);
 
        dbglogfile << __PRETTY_FUNCTION__