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

[Michael Stahl]

 vcl/source/gdi/bmpfast.cxx |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 2ca79052d0fab159b017cacd3d7145d8012de850
Author: Michael Stahl <mstahl at redhat.com>
Date:   Wed Jan 29 14:55:47 2014 +0100

    vcl: try to fix sizes in ImplBlendToBitmap()
    
    Since commit 2f84b420e0428693fb2d45b68afdc9023dfc08bc,
    sw_unoapi test sw.SwAccessibleEndnoteView crashes while trying to blend
    a 1107x8 Bitmap into a 97x8 Bitmap; it's not obvious that any of the
    parameters passed in is wrong and it's odd that ImplBlendToBitmap()
    would use rSrcBuffer.mnHeight and rDstBuffer.mnWidth; presumably
    it should use both height and width of the same buffer.
    
    Change-Id: I40b98ba56e703323df91f6fd7773addcfd69a402

diff --git a/vcl/source/gdi/bmpfast.cxx b/vcl/source/gdi/bmpfast.cxx
index 0fe79ea..30c1ccf 100644
--- a/vcl/source/gdi/bmpfast.cxx
+++ b/vcl/source/gdi/bmpfast.cxx
@@ -608,7 +608,8 @@ bool ImplBlendToBitmap( TrueColorPixelPtr<SRCFMT>& rSrcLine,
         nDstLinestep = -nDstLinestep;
     }
 
-    for( int y = rSrcBuffer.mnHeight; --y >= 0; )
+    assert(rDstBuffer.mnHeight <= rSrcBuffer.mnHeight); // not sure about that?
+    for (int y = rDstBuffer.mnHeight; --y >= 0;)
     {
         ImplBlendLines<8>( aDstLine, rSrcLine, aMskLine, rDstBuffer.mnWidth );
         aDstLine.AddByteOffset( nDstLinestep );