[Gnash-commit] gnash server/parser/movie_def_impl.h ChangeLog

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  07/01/29 18:16:25

Modified files:
        server/parser  : movie_def_impl.h 
        .              : ChangeLog 

Log message:
                * server/parser/movie_def_impl.h (add_execute_tag, 
add_init_action):
                  Add an assertion to catch out-of-bounds accesses.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/server/parser/movie_def_impl.h?cvsroot=gnash&r1=1.29&r2=1.30
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.2185&r2=1.2186

Patches:
Index: server/parser/movie_def_impl.h
===================================================================
RCS file: /sources/gnash/gnash/server/parser/movie_def_impl.h,v
retrieving revision 1.29
retrieving revision 1.30
diff -u -b -r1.29 -r1.30
--- server/parser/movie_def_impl.h      10 Jan 2007 17:28:50 -0000      1.29
+++ server/parser/movie_def_impl.h      29 Jan 2007 18:16:25 -0000      1.30
@@ -452,17 +452,16 @@
        void    add_execute_tag(execute_tag* e)
        {
            assert(e);
+           if (_frames_loaded < m_playlist.size());
            m_playlist[_frames_loaded].push_back(e);
        }
 
        /// Need to execute the given tag before entering the
        /// currently-loading frame for the first time.
-       ///
-       /// @@ AFAIK, the sprite_id is totally pointless -- correct?
-       //void  add_init_action(int sprite_id, execute_tag* e)
        void    add_init_action(execute_tag* e)
        {
            assert(e);
+           assert(_frames_loaded < m_playlist.size());
            m_init_action_list[_frames_loaded].push_back(e);
        }
 

Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.2185
retrieving revision 1.2186
diff -u -b -r1.2185 -r1.2186
--- ChangeLog   29 Jan 2007 08:44:35 -0000      1.2185
+++ ChangeLog   29 Jan 2007 18:16:25 -0000      1.2186
@@ -1,3 +1,8 @@
+2007-01-29 Sandro Santilli <address@hidden>
+
+       * server/parser/movie_def_impl.h (add_execute_tag, add_init_action):
+         Add an assertion to catch out-of-bounds accesses.
+
 2007-01-29 Zou Lunkai <address@hidden>
 
        * testsuite/misc-ming.all/goto_frame_test.c: