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

Caolán McNamara caolanm at redhat.com
Tue Oct 24 16:22:45 UTC 2017


 emfio/source/reader/mtftools.cxx |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 6a880bbc34790b619986602a921d734995fdf1c0
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Tue Oct 24 10:37:39 2017 +0100

    ofz Integer-overflow
    
    Change-Id: I8219dfa79565601681bc236789b0b18886c4f311
    Reviewed-on: https://gerrit.libreoffice.org/43745
    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/emfio/source/reader/mtftools.cxx b/emfio/source/reader/mtftools.cxx
index 96558d882ca4..399c453c0a9a 100644
--- a/emfio/source/reader/mtftools.cxx
+++ b/emfio/source/reader/mtftools.cxx
@@ -31,6 +31,7 @@
 #include <rtl/tencinfo.h>
 #include <vcl/virdev.hxx>
 #include <o3tl/make_unique.hxx>
+#include <o3tl/safeint.hxx>
 #include <officecfg/Setup.hxx>
 #include <officecfg/Office/Linguistic.hxx>
 #include <unotools/configmgr.hxx>
@@ -518,7 +519,9 @@ namespace emfio
 
         rFont.SetFontSize( aFontSize );
 
-        if( ( mnWinExtX * mnWinExtY ) < 0 )
+        sal_Int32 nResult;
+        const bool bFail = o3tl::checked_multiply(mnWinExtX, mnWinExtY, nResult);
+        if (!bFail && nResult < 0)
             rFont.SetOrientation( 3600 - rFont.GetOrientation() );
     }
 


More information about the Libreoffice-commits mailing list