From 98c48a4779e834e11ff568e0e6741cefa8e59570 Mon Sep 17 00:00:00 2001 From: hjk Date: Fri, 18 Aug 2017 09:21:44 +0200 Subject: [PATCH] BareMetal: Don't crash if no GdbServerProvider is found Change-Id: I78bca87f9e24d23f8faf26340e3dd585300efe86 Reviewed-by: Christian Kandeler --- src/plugins/baremetal/baremetaldebugsupport.cpp | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/src/plugins/baremetal/baremetaldebugsupport.cpp b/src/plugins/baremetal/baremetaldebugsupport.cpp index 278e018f1cc..c4615daa6e3 100644 --- a/src/plugins/baremetal/baremetaldebugsupport.cpp +++ b/src/plugins/baremetal/baremetaldebugsupport.cpp @@ -62,7 +62,14 @@ BareMetalDebugSupport::BareMetalDebugSupport(RunControl *runControl) return; } - const GdbServerProvider *p = GdbServerProviderManager::findProvider(dev->gdbServerProviderId()); + const QString providerId = dev->gdbServerProviderId(); + const GdbServerProvider *p = GdbServerProviderManager::findProvider(providerId); + if (!p) { + // FIXME: Translate. + reportFailure(QString("No GDB server provider found for %1").arg(providerId)); + return; + } + if (p->startupMode() == GdbServerProvider::StartupOnNetwork) { StandardRunnable r; r.executable = p->executable();