crashtesting: asserts on empty fill patterns

since...

commit d46c3214
Date:   Thu Jun 7 12:37:33 2018 +0200

    assert in BitmapInfoAccess if bitmap is empty or we can't read from it

Change-Id: Ic54e1d2511f2089eda9e0941f45135ae5e6f084d
Reviewed-on: https://gerrit.libreoffice.org/55686Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

svx/source/svdraw/svdetc.cxx

@@ -290,9 +290,12 @@ bool GetDraftFillColor(const SfxItemSet& rSet, Color& rCol)
             const Size aSize(aBitmap.GetSizePixel());
             const sal_uInt32 nWidth = aSize.Width();
             const sal_uInt32 nHeight = aSize.Height();
+            if (nWidth <= 0 || nHeight <= 0)
+                return bRetval;
+
             Bitmap::ScopedReadAccess pAccess(aBitmap);
 
-            if(pAccess && nWidth > 0 && nHeight > 0)
+            if (pAccess)
             {
                 sal_uInt32 nRt(0);
                 sal_uInt32 nGn(0);