From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Wed, 9 Jul 2003 21:58:24 +0000 (+0000)
Subject: check index in GetString()
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/17a6a662f7d7932983fc29892854fad56a82057b

check index in GetString()


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

diff --git a/src/mac/carbon/choice.cpp b/src/mac/carbon/choice.cpp
index fdd9b43da6..c187d9ecc7 100644
--- a/src/mac/carbon/choice.cpp
+++ b/src/mac/carbon/choice.cpp
@@ -177,8 +177,12 @@ void wxChoice::SetString(int n, const wxString& s)
 
 wxString wxChoice::GetString(int n) const
 {
+    wxCHECK_MSG( n >= 0 && (size_t)n < m_strings.GetCount(), _T(""),
+                    _T("wxChoice::GetString(): invalid index") );
+
     return m_strings[n] ;
 }
+
 // ----------------------------------------------------------------------------
 // client data
 // ----------------------------------------------------------------------------
diff --git a/src/mac/choice.cpp b/src/mac/choice.cpp
index fdd9b43da6..c187d9ecc7 100644
--- a/src/mac/choice.cpp
+++ b/src/mac/choice.cpp
@@ -177,8 +177,12 @@ void wxChoice::SetString(int n, const wxString& s)
 
 wxString wxChoice::GetString(int n) const
 {
+    wxCHECK_MSG( n >= 0 && (size_t)n < m_strings.GetCount(), _T(""),
+                    _T("wxChoice::GetString(): invalid index") );
+
     return m_strings[n] ;
 }
+
 // ----------------------------------------------------------------------------
 // client data
 // ----------------------------------------------------------------------------