From ef1b0f999df0c7e817f30f1d80406b1d4471c6a6 Mon Sep 17 00:00:00 2001 From: hjk Date: Thu, 17 Aug 2017 10:15:46 +0200 Subject: [PATCH] Debugger: Fix assert handling for devices without signalOperation() That's not a full solution, but fixes the reported crash. The core of the problem is that BareMetal devices do not provide signalOperation() (which is ok) and this code path should not have been taken to start with. Change-Id: Ib903b8d7f0728c5b3cb217b567247e66b751c5c3 Task-number: QTCREATORBUG-18694 Reviewed-by: Christian Kandeler --- src/plugins/debugger/gdb/gdbengine.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/plugins/debugger/gdb/gdbengine.cpp b/src/plugins/debugger/gdb/gdbengine.cpp index fd3f8892f3d..a49f469b875 100644 --- a/src/plugins/debugger/gdb/gdbengine.cpp +++ b/src/plugins/debugger/gdb/gdbengine.cpp @@ -800,7 +800,7 @@ void GdbEngine::interruptInferior() QTC_ASSERT(state() == InferiorStopRequested, qDebug() << state(); notifyInferiorStopFailed()); QTC_ASSERT(!m_signalOperation, notifyInferiorStopFailed()); m_signalOperation = runTool()->device()->signalOperation(); - QTC_ASSERT(m_signalOperation, notifyInferiorStopFailed()); + QTC_ASSERT(m_signalOperation, notifyInferiorStopFailed(); return); connect(m_signalOperation.data(), &DeviceProcessSignalOperation::finished, this, &GdbEngine::handleInterruptDeviceInferior);