UBSan-detected overflow when nWidthPixel is LONG_MAX and rCurrentZoom is 1

...from the call to

SetColumnWidth( nId, LONG_MAX );

in BrowseBox::AutoSizeLastColumn(), e.g. happens when opening the Gallery in
Draw.

Change-Id: I151ae557d9d2bec52ecb0bd92b870fb0b91d7242

1 files changed, 15 insertions, 6 deletions

diff --git a/svtools/source/brwbox/datwin.cxx b/svtools/source/brwbox/datwin.cxx
index a3dd14049979..e56ef6b70de6 100644
--- a/svtools/source/brwbox/datwin.cxx
+++ b/svtools/source/brwbox/datwin.cxx
@@ -124,12 +124,21 @@ BrowserColumn::~BrowserColumn()
 void BrowserColumn::SetWidth(sal_uLong nNewWidthPixel, const Fraction& rCurrentZoom)
 {
     _nWidth = nNewWidthPixel;
-    double n = (double)_nWidth;
-    n *= (double)rCurrentZoom.GetDenominator();
-    if (!rCurrentZoom.GetNumerator())
-        throw o3tl::divide_by_zero();
-    n /= (double)rCurrentZoom.GetNumerator();
-    _nOriginalWidth = n>0 ? (long)(n+0.5) : -(long)(-n+0.5);
+    // Avoid overflow when called with LONG_MAX from
+    // BrowseBox::AutoSizeLastColumn:
+    if (_nWidth == LONG_MAX)
+    {
+        _nOriginalWidth = _nWidth;
+    }
+    else
+    {
+        double n = (double)_nWidth;
+        n *= (double)rCurrentZoom.GetDenominator();
+        if (!rCurrentZoom.GetNumerator())
+            throw o3tl::divide_by_zero();
+        n /= (double)rCurrentZoom.GetNumerator();
+        _nOriginalWidth = n>0 ? (long)(n+0.5) : -(long)(-n+0.5);
+    }
 }
 
 void BrowserColumn::Draw( BrowseBox& rBox, OutputDevice& rDev, const Point& rPos, bool bCurs  )