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

Caolán McNamara (via logerrit) logerrit at kemper.freedesktop.org
Fri Apr 9 08:07:32 UTC 2021


 tools/source/generic/fract.cxx |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit ce9f23c6196b17985d14e441aa93175b31650f34
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Thu Apr 8 20:16:35 2021 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Fri Apr 9 10:06:47 2021 +0200

    ofz#33038 Floating-point-exception
    
    Change-Id: Ibe4a6667f5a14b5d94f2dbb92ad611ecba4984f4
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/113821
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/tools/source/generic/fract.cxx b/tools/source/generic/fract.cxx
index 648c91b20550..d4c4fe11c319 100644
--- a/tools/source/generic/fract.cxx
+++ b/tools/source/generic/fract.cxx
@@ -62,7 +62,8 @@ Fraction::Fraction( sal_Int64 nNum, sal_Int64 nDen ) : mnNumerator(nNum), mnDeno
         SAL_WARN( "tools.fraction", "'Fraction(" << nNum << ",0)' invalid fraction created" );
         return;
     }
-    if (nDen == -1 && nNum == std::numeric_limits<sal_Int32>::min())
+    if ((nDen == -1 && nNum == std::numeric_limits<sal_Int32>::min()) ||
+        (nNum == -1 && nDen == std::numeric_limits<sal_Int32>::min()))
     {
         mbValid = false;
         SAL_WARN("tools.fraction", "'Fraction(" << nNum << "," << nDen << ")' invalid fraction created");


More information about the Libreoffice-commits mailing list