From 47e646643969edf9851834bbde1e8470df2358c6 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Tue, 7 Mar 2006 23:24:55 +0000 Subject: [PATCH] make it clear that m_page is not NULL in OnBackOrNext() (coverity checked CID 20) git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@37879 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- src/generic/wizard.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/generic/wizard.cpp b/src/generic/wizard.cpp index 7054c96707..4690af5eae 100644 --- a/src/generic/wizard.cpp +++ b/src/generic/wizard.cpp @@ -780,10 +780,12 @@ void wxWizard::OnBackOrNext(wxCommandEvent& event) (event.GetEventObject() == m_btnPrev), wxT("unknown button") ); + wxCHECK_RET( m_page, _T("should have a valid current page") ); + // ask the current page first: notice that we do it before calling // GetNext/Prev() because the data transfered from the controls of the page // may change the value returned by these methods - if ( m_page && (!m_page->Validate() || !m_page->TransferDataFromWindow()) ) + if ( !m_page->Validate() || !m_page->TransferDataFromWindow() ) { // the page data is incorrect, don't do anything return; -- 2.45.2