diff options
| author | Morten Johan Sorvig <morten.sorvig@digia.com> | 2012-12-13 10:43:36 +0100 |
|---|---|---|
| committer | The Qt Project <gerrit-noreply@qt-project.org> | 2012-12-14 17:12:44 +0100 |
| commit | abf95b7f77a5d698f05dacdf86a6b23aa3ab8260 (patch) | |
| tree | a14e5b60e4acdbf57b004271cdb14a30dc1c92ed /src | |
| parent | 9a6366cead1c787f0c29b721bdbf07585199e9ca (diff) | |
Prevent crash in childAt when there are 0 children
Fix off-by-one error. QAccessibleInterface::child()
is 0-based, start the iteration at childCount() - 1.
Change-Id: I0c841b692adab0aae4e8ac44c7d308cd7a176ac8
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/plugins/accessible/widgets/qaccessiblemenu.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/accessible/widgets/qaccessiblemenu.cpp b/src/plugins/accessible/widgets/qaccessiblemenu.cpp index 5f7d9a9c30..c7dcee78f0 100644 --- a/src/plugins/accessible/widgets/qaccessiblemenu.cpp +++ b/src/plugins/accessible/widgets/qaccessiblemenu.cpp @@ -165,7 +165,7 @@ QAccessibleMenuItem::~QAccessibleMenuItem() QAccessibleInterface *QAccessibleMenuItem::childAt(int x, int y ) const { - for (int i = childCount(); i >= 0; --i) { + for (int i = childCount() - 1; i >= 0; --i) { QAccessibleInterface *childInterface = child(i); if (childInterface->rect().contains(x,y)) { return childInterface; |