From fa12743c8e421e115f7afd90d5a84d7b44ff6ed9 Mon Sep 17 00:00:00 2001 From: Gregory Hainaut Date: Fri, 22 Jul 2016 19:02:34 +0200 Subject: [PATCH] debugger: fix another missing init ==18373== Conditional jump or move depends on uninitialised value(s) ==18373== at 0x825CC9B: CtrlMemView::render(wxDC&) (CtrlMemView.cpp:277) ==18373== by 0x825DA19: CtrlMemView::paintEvent(wxPaintEvent&) (CtrlMemView.cpp:138) ==18373== by 0x81F3A5F: Pcsx2App::HandleEvent(wxEvtHandler*, void (wxEvtHandler::*)(wxEvent&), wxEvent&) (AppMain.cpp:614) ==18373== by 0x40BC50B: wxAppConsoleBase::CallEventHandler(wxEvtHandler*, wxEventFunctor&, wxEvent&) const (in /usr/lib/i386-linux-gnu/libwx_baseu-3.0.so.0.2.0) ==18373== by 0x426C654: wxEvtHandler::ProcessEventIfMatchesId(wxEventTableEntryBase const&, wxEvtHandler*, wxEvent&) (in /usr/lib/i386-linux-gnu/libwx_baseu-3.0.so.0.2.0) ==18373== by 0x426C752: wxEventHashTable::HandleEvent(wxEvent&, wxEvtHandler*) (in /usr/lib/i386-linux-gnu/libwx_baseu-3.0.so.0.2.0) Reported by gcc too --- pcsx2/gui/Debugger/CtrlMemView.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pcsx2/gui/Debugger/CtrlMemView.cpp b/pcsx2/gui/Debugger/CtrlMemView.cpp index 68e690eeac..5e746ca603 100644 --- a/pcsx2/gui/Debugger/CtrlMemView.cpp +++ b/pcsx2/gui/Debugger/CtrlMemView.cpp @@ -201,9 +201,9 @@ void CtrlMemView::render(wxDC& dc) for (int j = 0; j < rowSize; j++) { u32 byteAddress = rowAddress + j; - u8 byteCurrent; - bool byteValid; - + u8 byteCurrent = 0; + bool byteValid = false; + try { byteValid = validCpu && cpu->isValidAddress(byteAddress);