From d3a1c80edf4bbcf4c581670fa4884ccc7894afe9 Mon Sep 17 00:00:00 2001 From: Magne Sjaastad Date: Mon, 13 Nov 2023 15:54:55 +0100 Subject: [PATCH] Guard null pointer use --- ApplicationLibCode/UserInterface/RiuMainWindowBase.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ApplicationLibCode/UserInterface/RiuMainWindowBase.cpp b/ApplicationLibCode/UserInterface/RiuMainWindowBase.cpp index 569d15d07c..8f32f714f2 100644 --- a/ApplicationLibCode/UserInterface/RiuMainWindowBase.cpp +++ b/ApplicationLibCode/UserInterface/RiuMainWindowBase.cpp @@ -354,11 +354,12 @@ bool RiuMainWindowBase::isBlockingViewSelectionOnSubWindowActivated() const void RiuMainWindowBase::removeViewerFromMdiArea( RiuMdiArea* mdiArea, QWidget* viewer ) { bool removedSubWindowWasActive = false; + bool wasMaximized = true; - QMdiSubWindow* subWindowBeingClosed = findMdiSubWindow( viewer ); - bool wasMaximized = subWindowBeingClosed->isMaximized(); - if ( subWindowBeingClosed ) + if ( QMdiSubWindow* subWindowBeingClosed = findMdiSubWindow( viewer ) ) { + wasMaximized = subWindowBeingClosed->isMaximized(); + if ( subWindowBeingClosed->isActiveWindow() ) { // If we are removing the active window, we will need a new active window