[Gnash-commit] gnash ChangeLog server/font.cpp [release_0_7_2]

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Branch:         release_0_7_2
Changes by:     Sandro Santilli <strk>  06/11/09 14:18:45

Modified files:
        .              : ChangeLog 
        server         : font.cpp 

Log message:
                * server/font.cpp (readDefineFont2): allow seek-back in stream
                  while reading shape glyphs (fixes bug #16311).

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&only_with_tag=release_0_7_2&r1=1.1412.2.132&r2=1.1412.2.133
http://cvs.savannah.gnu.org/viewcvs/gnash/server/font.cpp?cvsroot=gnash&only_with_tag=release_0_7_2&r1=1.22&r2=1.22.2.1

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.1412.2.132
retrieving revision 1.1412.2.133
diff -u -b -r1.1412.2.132 -r1.1412.2.133
--- ChangeLog   9 Nov 2006 13:12:24 -0000       1.1412.2.132
+++ ChangeLog   9 Nov 2006 14:18:45 -0000       1.1412.2.133
@@ -1,3 +1,8 @@
+2006-11-09 Sandro Santilli <address@hidden>
+
+       * server/font.cpp (readDefineFont2): allow seek-back in stream
+         while reading shape glyphs (fixes bug #16311).
+
 2006-11-09 Bastiaan Jacques <address@hidden>
 
        * libbase/curl_adapter.cpp: Allow libcurl to follow HTTP redirects.

Index: server/font.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/font.cpp,v
retrieving revision 1.22
retrieving revision 1.22.2.1
diff -u -b -r1.22 -r1.22.2.1
--- server/font.cpp     26 Sep 2006 13:10:55 -0000      1.22
+++ server/font.cpp     9 Nov 2006 14:18:45 -0000       1.22.2.1
@@ -239,8 +239,11 @@
                        {
                                // Seek to the start of the shape data.
                                int     new_pos = table_base + offsets[i];
-                               // if we're seeking backwards, then that looks 
like a bug.
-                               assert(new_pos >= in->get_position());
+
+                               // It seems completely possible to
+                               // have such seeks-back, see bug #16311
+                               //assert(new_pos >= in->get_position());
+
                                in->set_position(new_pos);
 
                                // Create & read the shape.