[Libreoffice-commits] core.git: Branch 'libreoffice-6-1' - drawinglayer/source

[Caolán McNamara]

 drawinglayer/source/texture/texture3d.cxx |   11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

New commits:
commit f532decefb7881eb5f56864db9fcfd5d2ba85ee4
Author: Caolán McNamara <caol...@redhat.com>
Date:   Mon May 28 21:36:16 2018 +0100

    forcepoint#41 null deref
    
    Change-Id: I16e9e083811c6e14861da1ba1df7d46e8c8771d7
    Reviewed-on: https://gerrit.libreoffice.org/54973
    Reviewed-by: Caolán McNamara <caol...@redhat.com>
    Tested-by: Caolán McNamara <caol...@redhat.com>

diff --git a/drawinglayer/source/texture/texture3d.cxx 
b/drawinglayer/source/texture/texture3d.cxx
index fc8750199c8f..0dac447ccf50 100644
--- a/drawinglayer/source/texture/texture3d.cxx
+++ b/drawinglayer/source/texture/texture3d.cxx
@@ -74,8 +74,6 @@ namespace drawinglayer
         {
             // #121194# Todo: use alpha channel, too (for 3d)
             maBitmap = maBitmapEx.GetBitmap();
-            mpReadBitmap = Bitmap::ScopedReadAccess(maBitmap);
-            OSL_ENSURE(mpReadBitmap, "GeoTexSvxBitmapEx: Got no read access to 
Bitmap (!)");
 
             if(mbIsTransparent)
             {
@@ -92,8 +90,13 @@ namespace drawinglayer
                 mpReadTransparence = Bitmap::ScopedReadAccess(maTransparence);
             }
 
-            mfMulX = static_cast<double>(mpReadBitmap->Width()) / 
maSize.getX();
-            mfMulY = static_cast<double>(mpReadBitmap->Height()) / 
maSize.getY();
+            mpReadBitmap = Bitmap::ScopedReadAccess(maBitmap);
+            SAL_WARN_IF(!mpReadBitmap, "drawinglayer", "GeoTexSvxBitmapEx: Got 
no read access to Bitmap");
+            if (mpReadBitmap)
+            {
+                mfMulX = static_cast<double>(mpReadBitmap->Width()) / 
maSize.getX();
+                mfMulY = static_cast<double>(mpReadBitmap->Height()) / 
maSize.getY();
+            }
 
             if(maSize.getX() <= 1.0)
             {
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits