ClangTools: Fix crash in document tool runner

Prevent removing refactor markers from already destroyed widgets.

Fixes: QTCREATORBUG-24982
Change-Id: Icf8950f8f8407fa8733a1ef30d05abb4a2b44dd4
Reviewed-by: Christian Stenger <christian.stenger@qt.io>

1 files changed, 13 insertions, 6 deletions

diff --git a/src/plugins/clangtools/documentclangtoolrunner.cpp b/src/plugins/clangtools/documentclangtoolrunner.cpp
index 9e2b388e5d..0f5e4da958 100644
--- a/src/plugins/clangtools/documentclangtoolrunner.cpp
+++ b/src/plugins/clangtools/documentclangtoolrunner.cpp
@@ -105,15 +105,21 @@ Diagnostics DocumentClangToolRunner::diagnosticsAtLine(int lineNumber) const
     return diagnostics;
 }
 
+static void removeClangToolRefactorMarkers(TextEditor::TextEditorWidget *editor)
+{
+    if (!editor)
+        return;
+    editor->setRefactorMarkers(
+        TextEditor::RefactorMarker::filterOutType(editor->refactorMarkers(),
+                                                  Constants::CLANG_TOOL_FIXIT_AVAILABLE_MARKER_ID));
+}
+
 void DocumentClangToolRunner::scheduleRun()
 {
     for (DiagnosticMark *mark : m_marks)
         mark->disable();
-    for (TextEditor::TextEditorWidget *editor : m_editorsWithMarkers) {
-        editor->setRefactorMarkers(
-            TextEditor::RefactorMarker::filterOutType(editor->refactorMarkers(),
-                                                      Constants::CLANG_TOOL_FIXIT_AVAILABLE_MARKER_ID));
-    }
+    for (TextEditor::TextEditorWidget *editor : m_editorsWithMarkers)
+        removeClangToolRefactorMarkers(editor);
     m_runTimer.start();
 }
 
@@ -325,7 +331,8 @@ void DocumentClangToolRunner::onSuccess()
     for (auto editor : TextEditor::BaseTextEditor::textEditorsForDocument(doc)) {
         if (TextEditor::TextEditorWidget *widget = editor->editorWidget()) {
             widget->setRefactorMarkers(markers + widget->refactorMarkers());
-            m_editorsWithMarkers << widget;
+            if (!m_editorsWithMarkers.contains(widget))
+                m_editorsWithMarkers << widget;
         }
     }