From 985c1ee3e9dc0298405e9400f7016d869ec4676e Mon Sep 17 00:00:00 2001 From: Christian Stenger Date: Wed, 10 May 2023 12:32:05 +0200 Subject: [PATCH] Perforce: Do not wait for process if not set up The process could be not set up correctly or at all if we exit the start function early. Avoids a soft assert. Change-Id: I9f0d71acc4a440b85dbb26fa656e0b551ae685f4 Reviewed-by: hjk --- src/plugins/perforce/perforcechecker.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/plugins/perforce/perforcechecker.cpp b/src/plugins/perforce/perforcechecker.cpp index f9a5b5df38b..cf63d9e6913 100644 --- a/src/plugins/perforce/perforcechecker.cpp +++ b/src/plugins/perforce/perforcechecker.cpp @@ -26,8 +26,10 @@ PerforceChecker::PerforceChecker(QObject *parent) : QObject(parent) PerforceChecker::~PerforceChecker() { - m_process.kill(); - m_process.waitForFinished(); + if (m_process.isRunning()) { + m_process.kill(); + m_process.waitForFinished(); + } resetOverrideCursor(); }