From 64a0a72a544c68ffdbf39d5d7b49dfae24c9477d Mon Sep 17 00:00:00 2001 From: Erik Verbruggen Date: Thu, 11 Sep 2014 10:43:54 +0200 Subject: [PATCH] Beautifier: change QTC_CHECK to QTC_ASSERT. Return when the check fails, so there won't be a nullptr deref 2 lines down. Change-Id: I4dd1ba13798a57700c486e4b1f24b45afb2837ad Reviewed-by: Lorenz Haas Reviewed-by: Erik Verbruggen --- src/plugins/beautifier/configurationeditor.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/plugins/beautifier/configurationeditor.cpp b/src/plugins/beautifier/configurationeditor.cpp index 39f24e33b95..046c45fd7bc 100644 --- a/src/plugins/beautifier/configurationeditor.cpp +++ b/src/plugins/beautifier/configurationeditor.cpp @@ -118,7 +118,7 @@ ConfigurationEditor::ConfigurationEditor(QWidget *parent) void ConfigurationEditor::setSettings(AbstractSettings *settings) { - QTC_CHECK(settings); + QTC_ASSERT(settings, return); m_settings = settings; QStringList keywords = m_settings->options();