[Gnash-commit] gnash libamf/amf.cpp ChangeLog

[Rob Savoye]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Rob Savoye <rsavoye>    08/04/30 03:57:01

Modified files:
        libamf         : amf.cpp 
        .              : ChangeLog 

Log message:
                * libamf/amf.cpp: Go back to a proper type cast now that the 
real
                problem is fixed. Even valgrind is happy.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/libamf/amf.cpp?cvsroot=gnash&r1=1.73&r2=1.74
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.6453&r2=1.6454

Patches:
Index: libamf/amf.cpp
===================================================================
RCS file: /sources/gnash/gnash/libamf/amf.cpp,v
retrieving revision 1.73
retrieving revision 1.74
diff -u -b -r1.73 -r1.74
--- libamf/amf.cpp      30 Apr 2008 03:35:30 -0000      1.73
+++ libamf/amf.cpp      30 Apr 2008 03:57:00 -0000      1.74
@@ -833,9 +833,11 @@
     boost::uint16_t length = 0;
     Network::byte_t len = 0;
 
-//    length = *(reinterpret_cast<boost::uint16_t *>(tmpptr));
-    length = tmpptr[1];
-//    swapBytes(&length, sizeof(boost::uint16_t));
+    length = *(reinterpret_cast<boost::uint16_t *>(tmpptr));
+//     length = tmpptr[1];             // FIXME: hack. This supports
+//                             // strings up to 256 characters
+//                             // but keep svalgrind happy.
+    swapBytes(&length, sizeof(boost::uint16_t));
     tmpptr += sizeof(boost::uint16_t);
     
     if (length <= 0) {

Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.6453
retrieving revision 1.6454
diff -u -b -r1.6453 -r1.6454
--- ChangeLog   30 Apr 2008 03:48:50 -0000      1.6453
+++ ChangeLog   30 Apr 2008 03:57:01 -0000      1.6454
@@ -1,5 +1,8 @@
 2008-04-29  Rob Savoye  <address@hidden>
 
+       * libamf/amf.cpp: Go back to a proper type cast now that the real
+       problem is fixed.
+       
        * libnet/rtmp.{h,cpp}: Use *Property instead of *Variable.
        
        * configure.ac: Don't look for mallinfo() when using jemalloc().