]> cloud.milkyroute.net Git - dolphin.git/commitdiff
Fix possible out of range error inKItemListSizeHintResolver::itemsMoved
authorFrank Reininghaus <frank78ac@googlemail.com>
Sun, 4 Aug 2013 19:46:08 +0000 (21:46 +0200)
committerFrank Reininghaus <frank78ac@googlemail.com>
Sun, 4 Aug 2013 19:46:08 +0000 (21:46 +0200)
The function assumes implicitly that the moved range always starts with
the index 0. This is indeed the case at the moment, but it might make
sense to change that in the future. This commit prevents that we get an
out of range problem then.

Thanks to Emmanuel Pescosta for finding this problem, see
https://git.reviewboard.kde.org/r/111808/

src/kitemviews/private/kitemlistsizehintresolver.cpp

index e44630243b1c148325f26689a2a03d093563c68a..c86ce2bf859866f85b53fea607a38f7abf840e04 100644 (file)
@@ -120,7 +120,7 @@ void KItemListSizeHintResolver::itemsMoved(const KItemRange& range, const QList<
 
     const int movedRangeEnd = range.index + range.count;
     for (int i = range.index; i < movedRangeEnd; ++i) {
-        const int newIndex = movedToIndexes.at(i);
+        const int newIndex = movedToIndexes.at(i - range.index);
         newSizeHintCache[newIndex] = m_sizeHintCache.at(i);
     }