From: Karsten Ballüder <ballueder@usa.net>
Date: Thu, 25 Feb 1999 10:32:48 +0000 (+0000)
Subject: GetNextItem() now checks for item < GetItemCount() to avoid illegal
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/02da33e27d0eee3440a1ea83c8551ec99ab93f9f

GetNextItem() now checks for item < GetItemCount() to avoid illegal
wxList access beyond end of list.


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@1785 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/generic/listctrl.cpp b/src/generic/listctrl.cpp
index 01afffd3fe..600a0fccd6 100644
--- a/src/generic/listctrl.cpp
+++ b/src/generic/listctrl.cpp
@@ -2033,6 +2033,7 @@ long wxListMainWindow::GetNextItem( long item, int WXUNUSED(geometry), int state
 {
     long ret = 0;
     if (item > 0) ret = item;
+    if(item >= GetItemCount()) return -1;
     wxNode *node = m_lines.Nth( ret );
     while (node)
     {