[Libreoffice-commits] core.git: Branch 'libreoffice-4-3' - vcl/source

Caolán McNamara caolanm at redhat.com
Tue Sep 23 07:32:37 PDT 2014


 vcl/source/gdi/regionband.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit f7abe28ec99733f80a6c6f249ef5dbddeb31ad7f
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Sun Sep 21 17:37:05 2014 +0100

    set mpFirstBand back to 0 after deleting it
    
    bff + valgrind
    
    ==30470== Invalid read of size 8
    ==30470==    at 0xA02103F: RegionBand::isSingleRectangle() const (regionband.cxx:294)
    ==30470==    by 0xA01F562: WriteRegion(SvStream&, Region const&) (region.cxx:1640)
    ==30470==    by 0x9F01FD4: MetaClipRegionAction::Write(SvStream&, ImplMetaWriteData*) (metaact.cxx:2487)
    ...
    ==30470==  Address 0x1d175980 is 0 bytes inside a block of size 56 free'd
    ==30470==    at 0x4A07991: operator delete(void*) (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
    ==30470==    by 0xA020BB1: RegionBand::implReset() (regionband.cxx:87)
    ==30470==    by 0xA020EF2: RegionBand::load(SvStream&) (regionband.cxx:249)
    
    Change-Id: I2468d43c7e73b3390f355358990f8af30b5f026e
    (cherry picked from commit 1ca892488a7a6a9c35fc99c41cd4af788fe0df6e)
    Reviewed-on: https://gerrit.libreoffice.org/11570
    Reviewed-by: Andras Timar <andras.timar at collabora.com>
    Tested-by: Andras Timar <andras.timar at collabora.com>

diff --git a/vcl/source/gdi/regionband.cxx b/vcl/source/gdi/regionband.cxx
index fa76743..adaa4b0 100644
--- a/vcl/source/gdi/regionband.cxx
+++ b/vcl/source/gdi/regionband.cxx
@@ -89,7 +89,7 @@ void RegionBand::implReset()
     }
 
     mpLastCheckedBand = 0;
-
+    mpFirstBand = 0;
 }
 
 RegionBand::~RegionBand()


More information about the Libreoffice-commits mailing list