rev 12457 - in trunk/packages/kdelibs/debian: . patches

[Raúl Sánchez Siles]

Author: kebianizao-guest
Date: 2008-10-22 22:34:33 +0000 (Wed, 22 Oct 2008)
New Revision: 12457

Added:
   trunk/packages/kdelibs/debian/patches/63_fixed-layout-table.diff
Modified:
   trunk/packages/kdelibs/debian/changelog
Log:
konqueror: Crash on eBay page (Closes: #502459) with recently added
63_fixed-layout-table.diff

Modified: trunk/packages/kdelibs/debian/changelog
===================================================================
--- trunk/packages/kdelibs/debian/changelog	2008-10-21 21:17:40 UTC (rev 12456)
+++ trunk/packages/kdelibs/debian/changelog	2008-10-22 22:34:33 UTC (rev 12457)
@@ -21,6 +21,8 @@
   * Fixed wrong http header parsing, added 61_httpheader_backport.diff 
   * Fixed wrong Google Maps rendering, added 62_fix_googlemaps_backport.diff
   * Change dependencies from obsolete libcupsys2-dev to libcups2-dev.
+  * konqueror: Crash on eBay page (Closes: #502459) with recently added
+    63_fixed-layout-table.diff
 
  -- Debian Qt/KDE Maintainers <debian-qt-kde at lists.debian.org>  Mon, 15 Sep 2008 02:41:12 +0200
 

Added: trunk/packages/kdelibs/debian/patches/63_fixed-layout-table.diff
===================================================================
--- trunk/packages/kdelibs/debian/patches/63_fixed-layout-table.diff	                        (rev 0)
+++ trunk/packages/kdelibs/debian/patches/63_fixed-layout-table.diff	2008-10-22 22:34:33 UTC (rev 12457)
@@ -0,0 +1,24 @@
+Index: branches/KDE/3.5/kdelibs/khtml/rendering/table_layout.cpp
+===================================================================
+--- branches/KDE/3.5/kdelibs/khtml/rendering/table_layout.cpp	(revisión: 874967)
++++ branches/KDE/3.5/kdelibs/khtml/rendering/table_layout.cpp	(revisión: 874968)
+@@ -297,7 +297,8 @@
+ #endif
+         for ( int i = 0; available > 0 && i < nEffCols; i++ ) {
+             if ( width[i].isPercent() ) {
+-                int w = base * width[i].value() / totalPercent;
++                // totalPercent may be 0 below if all %-width specifed are 0%. (#172557)
++                int w = totalPercent ? base * width[i].value() / totalPercent : 0;
+                 available -= w;
+                 calcWidth[i] = w;
+             }
+@@ -313,7 +314,8 @@
+ 
+         for ( int i = 0; available > 0 && i < nEffCols; i++ ) {
+             if ( width[i].isVariable() ) {
+-                int w = available / totalVariable;
++                // totalVariable may be 0 below if all the variable widths specified are 0.
++                int w = totalVariable ? available / totalVariable : 0;
+                 available -= w;
+                 calcWidth[i] = w;
+ 		totalVariable--;