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

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  08/05/06 15:28:31

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

Log message:
        (ActionFscommand2): fix ensureStack calls. Fixes bug #23181.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&r1=1.6528&r2=1.6529
http://cvs.savannah.gnu.org/viewcvs/gnash/server/vm/ASHandlers.cpp?cvsroot=gnash&r1=1.233&r2=1.234

Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/gnash/gnash/ChangeLog,v
retrieving revision 1.6528
retrieving revision 1.6529
diff -u -b -r1.6528 -r1.6529
--- ChangeLog   6 May 2008 14:55:24 -0000       1.6528
+++ ChangeLog   6 May 2008 15:28:29 -0000       1.6529
@@ -1,5 +1,10 @@
 2008-05-06 Sandro Santilli <address@hidden>
 
+       * server/vm/ASHandlers.cpp (ActionFscommand2): fix ensureStack
+         calls. Fixes bug #23181.
+
+2008-05-06 Sandro Santilli <address@hidden>
+
        * server/PropertyList.cpp (setValue): don't let prop flags
          prevent setting values of destructive properties.
          set_member doesn't use this function, so user-requested

Index: server/vm/ASHandlers.cpp
===================================================================
RCS file: /sources/gnash/gnash/server/vm/ASHandlers.cpp,v
retrieving revision 1.233
retrieving revision 1.234
diff -u -b -r1.233 -r1.234
--- server/vm/ASHandlers.cpp    28 Apr 2008 07:51:04 -0000      1.233
+++ server/vm/ASHandlers.cpp    6 May 2008 15:28:31 -0000       1.234
@@ -1514,7 +1514,7 @@
        thread.ensureStack(1); // nargs
        unsigned int nargs = env.top(off++).to_int();
 
-       thread.ensureStack(nargs); // nargs, cmdname
+       thread.ensureStack(off+nargs+1); // nargs, cmdname, actual args
        std::string cmd = env.top(off++).to_string();
 
        std::stringstream ss;
@@ -1523,11 +1523,11 @@
        {
                as_value arg = env.top(off++);
                if ( i>1 ) ss << ", ";
-               ss << arg.to_debug_string();
+               ss << arg;
        }
        ss << ")";
 
-       log_unimpl("fscommand2:%s", ss.str().c_str());
+       LOG_ONCE( log_unimpl("fscommand2:%s", ss.str()) );
 
        // TODO: check wheter or not we should drop anything from
        //       the stack, some reports and the Canonical tests