[Gnash-commit] gnash ChangeLog server/DynamicShape.cpp server/...

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  07/12/03 16:17:49

Modified files:
        .              : ChangeLog 
        server         : DynamicShape.cpp sprite_instance.cpp 

Log message:
        DynamicShape::clear(): set current path to NULL or would point to 
deleted memory.
        Fixes bug #21583.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.5068&r2=1.5069
http://cvs.savannah.gnu.org/viewcvs/gnash/server/DynamicShape.cpp?cvsroot=gnash&r1=1.12&r2=1.13
http://cvs.savannah.gnu.org/viewcvs/gnash/server/sprite_instance.cpp?cvsroot=gnash&r1=1.402&r2=1.403

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.5068
retrieving revision 1.5069
diff -u -b -r1.5068 -r1.5069
--- ChangeLog   3 Dec 2007 14:33:54 -0000       1.5068
+++ ChangeLog   3 Dec 2007 16:17:48 -0000       1.5069
@@ -1,5 +1,12 @@
 2007-12-03 Sandro Santilli <address@hidden>
 
+       * server/DynamicShape.cpp (clear): set current path to NULL
+         or would point to deleted memory. Fixes bug #21583.
+       * server/sprite_instance.cpp (markReachableResources): assert
+         that the tmp backup list is empty.
+
+2007-12-03 Sandro Santilli <address@hidden>
+
        * server/swf/tag_loaders.cpp (sound_stream_head_loader): ensure 
          needed bytes are available, fix long read for mp3 case.
        * server/stream.cpp (open_tag): don't trust a tag advertising

Index: server/DynamicShape.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/DynamicShape.cpp,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -b -r1.12 -r1.13
--- server/DynamicShape.cpp     7 Nov 2007 14:06:16 -0000       1.12
+++ server/DynamicShape.cpp     3 Dec 2007 16:17:49 -0000       1.13
@@ -17,7 +17,7 @@
 
 
 
-/* $Id: DynamicShape.cpp,v 1.12 2007/11/07 14:06:16 strk Exp $ */
+/* $Id: DynamicShape.cpp,v 1.13 2007/12/03 16:17:49 strk Exp $ */
 
 #include "DynamicShape.h"
 
@@ -46,6 +46,9 @@
        m_fill_styles.clear();
        m_line_styles.clear();
        m_bound.set_null();
+       _currpath=0; // or would point to invalid memory
+
+       // TODO: set _changed=true ? reset _currfill and _currline ?
 }
 
 void

Index: server/sprite_instance.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/sprite_instance.cpp,v
retrieving revision 1.402
retrieving revision 1.403
diff -u -b -r1.402 -r1.403
--- server/sprite_instance.cpp  3 Dec 2007 07:06:15 -0000       1.402
+++ server/sprite_instance.cpp  3 Dec 2007 16:17:49 -0000       1.403
@@ -3856,6 +3856,8 @@
 
        m_display_list.visitAll(marker);
 
+       assert(m_tmp_display_list.empty());
+
        _drawable->setReachable();
 
        _drawable_inst->setReachable();