[Gnash-commit] gnash ChangeLog backend/render_handler_agg.cpp [release_0_7_2]

[Udo Giacomozzi]

CVSROOT:        /cvsroot/gnash
Module name:    gnash
Branch:         release_0_7_2
Changes by:     Udo Giacomozzi <udog>   06/11/07 12:30:55

Modified files:
        .              : ChangeLog 
        backend        : render_handler_agg.cpp 

Log message:
        backporting...

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/ChangeLog?cvsroot=gnash&only_with_tag=release_0_7_2&r1=1.1412.2.106&r2=1.1412.2.107
http://cvs.savannah.gnu.org/viewcvs/gnash/backend/render_handler_agg.cpp?cvsroot=gnash&only_with_tag=release_0_7_2&r1=1.29.2.13&r2=1.29.2.14

Patches:
Index: ChangeLog
===================================================================
RCS file: /cvsroot/gnash/gnash/ChangeLog,v
retrieving revision 1.1412.2.106
retrieving revision 1.1412.2.107
diff -u -b -r1.1412.2.106 -r1.1412.2.107
--- ChangeLog   7 Nov 2006 11:47:21 -0000       1.1412.2.106
+++ ChangeLog   7 Nov 2006 12:30:55 -0000       1.1412.2.107
@@ -1,3 +1,8 @@
+2006-11-07 Udo Giacomozzi <address@hidden>
+
+       * backend/render_handler_agg.cpp: Protected clear_framebuffer()
+       against negative width and height arguments
+
 2006-11-07 Sandro Santilli <address@hidden>
 
        * server/action.cpp, server/swf/ASHandlers.cpp: turn

Index: backend/render_handler_agg.cpp
===================================================================
RCS file: /cvsroot/gnash/gnash/backend/render_handler_agg.cpp,v
retrieving revision 1.29.2.13
retrieving revision 1.29.2.14
diff -u -b -r1.29.2.13 -r1.29.2.14
--- backend/render_handler_agg.cpp      4 Nov 2006 12:43:50 -0000       
1.29.2.13
+++ backend/render_handler_agg.cpp      7 Nov 2006 12:30:55 -0000       
1.29.2.14
@@ -16,7 +16,7 @@
 
  
 
-/* $Id: render_handler_agg.cpp,v 1.29.2.13 2006/11/04 12:43:50 udog Exp $ */
+/* $Id: render_handler_agg.cpp,v 1.29.2.14 2006/11/07 12:30:55 udog Exp $ */
 
 // Original version by Udo Giacomozzi and Hannes Mayr, 
 // INDUNET GmbH (www.indunet.it)
@@ -520,10 +520,17 @@
        /// still correct, but slower. 
   /// This function clears only a certain portion of the screen, while /not/ 
   /// being notably slower for a fullscreen clear. 
-       void clear_framebuffer(unsigned int left, unsigned int top, 
-    unsigned int width, unsigned int height, agg::rgba8 color) {
+       void clear_framebuffer(int left, int top, int width, int height, 
agg::rgba8 color) {
     
-         if (!width) return;
+               if (width<1) {
+                       log_msg("warning: clear_framebuffer() called with 
width=%d", width);
+                       return;
+               }
+        
+               if (height<1) {
+                       log_msg("warning: clear_framebuffer() called with 
height=%d", height);
+                       return;
+               }
          
          unsigned int y;
          const unsigned int max_y = top+height; // to be exact, it's one off 
the max.