[Libreoffice-commits] core.git: tools/source

Caolán McNamara caolanm at redhat.com
Fri Oct 27 14:14:15 UTC 2017


 tools/source/generic/bigint.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 91263493eac70464fa6036cf9d8676e5ed452869
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Fri Oct 27 13:28:34 2017 +0100

    ofz#3802 Integer-overflow
    
    Change-Id: Id631951119ed0a6b62ffe7d8342cd92191714114
    Reviewed-on: https://gerrit.libreoffice.org/43943
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/tools/source/generic/bigint.cxx b/tools/source/generic/bigint.cxx
index bef8de9f7a9e..a9b8eca1b97f 100644
--- a/tools/source/generic/bigint.cxx
+++ b/tools/source/generic/bigint.cxx
@@ -334,7 +334,7 @@ void BigInt::DivLong( const BigInt& rB, BigInt& rErg ) const
             nQ = (sal_uInt16)(((sal_uInt32)nTmp) / aTmpB.nNum[nLenB1]);
 
         if ( ((sal_uInt32)aTmpB.nNum[nLenB1 - 1] * nQ) >
-            ((((sal_uInt32)nTmp) - aTmpB.nNum[nLenB1] * nQ) << 16) + aTmpA.nNum[j - 2])
+            ((((sal_uInt32)nTmp) - (sal_uInt32)aTmpB.nNum[nLenB1] * nQ) << 16) + aTmpA.nNum[j - 2])
             nQ--;
         // Start division
         nK = 0;


More information about the Libreoffice-commits mailing list