diff options
author | Giuseppe D'Angelo <giuseppe.dangelo@kdab.com> | 2022-01-27 09:47:59 +0100 |
---|---|---|
committer | Giuseppe D'Angelo <giuseppe.dangelo@kdab.com> | 2022-02-01 14:32:57 +0100 |
commit | 035babe5020cfeafdaf5e8590f4c5c3ac043a5bf (patch) | |
tree | 8ac51acfb4f1f3c402fcc9b11982451b928ae6fe /src/widgets/itemviews | |
parent | 33ad8b6fa9afbe4b8612f26c0bad42d23d94e7b2 (diff) |
QAbstractItemView: do not access invalid model indices (2/N)
Similar to the parent patch, the private selectAll() was doing
two out of bounds accesses on an empty model. Guard it.
Change-Id: If0f3ce1e6c44a152791313e47db79985e71ef955
Pick-to: 5.15 6.2 6.3
Reviewed-by: David Faure <david.faure@kdab.com>
Diffstat (limited to 'src/widgets/itemviews')
-rw-r--r-- | src/widgets/itemviews/qabstractitemview.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/widgets/itemviews/qabstractitemview.cpp b/src/widgets/itemviews/qabstractitemview.cpp index d9a64a3544..ae03f27fac 100644 --- a/src/widgets/itemviews/qabstractitemview.cpp +++ b/src/widgets/itemviews/qabstractitemview.cpp @@ -4623,6 +4623,8 @@ void QAbstractItemViewPrivate::selectAll(QItemSelectionModel::SelectionFlags com { if (!selectionModel) return; + if (!model->hasChildren(root)) + return; QItemSelection selection; QModelIndex tl = model->index(0, 0, root); |