canvas/source/tools/pagemanager.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 0f36d4a923f37b1fd85ab024d09e84e53bf481ba
Author: Caolán McNamara <caol...@redhat.com>
Date:   Fri Sep 11 09:19:46 2015 +0100

    coverity#1323755 Using invalid iterator
    
    try and help coverity to know that aEnd and maFragments.end() are the same
    place
    
    Change-Id: I9ce4187d5ed15d0ae26b69bb360dbd47c2a6a185

diff --git a/canvas/source/tools/pagemanager.cxx 
b/canvas/source/tools/pagemanager.cxx
index 46c5716..3d08eff2 100644
--- a/canvas/source/tools/pagemanager.cxx
+++ b/canvas/source/tools/pagemanager.cxx
@@ -91,7 +91,7 @@ namespace canvas
             // TODO(F1): this is a heuristic, could
             // be designed as a policy.
             auto       aEnd( maFragments.cend() );
-            auto       aCurrMax( maFragments.end() );
+            auto       aCurrMax( aEnd );
             sal_uInt32 nCurrMaxArea = 0;
             for( auto aCurr = maFragments.begin(); aCurr != aEnd; ++aCurr )
             {
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to