From 59718d7c4eaf78576a2aaafb9ff2732d4ab99a60 Mon Sep 17 00:00:00 2001
From: Jarek Kobus <jaroslaw.kobus@qt.io>
Date: Wed, 27 Jan 2021 11:50:15 +0100
Subject: [PATCH] Check if the document ptr is not null

Avoid a crash when double click in hierarchy view
after the session has been changed or when the filename
changed.

Change-Id: Iebd7a99c220737262c79ade43c4ee8d11dcc037f
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
---
 src/plugins/cpptools/cppelementevaluator.cpp | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/plugins/cpptools/cppelementevaluator.cpp b/src/plugins/cpptools/cppelementevaluator.cpp
index 5cfdddcf2c0..9cb6c6f8076 100644
--- a/src/plugins/cpptools/cppelementevaluator.cpp
+++ b/src/plugins/cpptools/cppelementevaluator.cpp
@@ -548,6 +548,9 @@ public:
                     QString &expression)
     {
         doc = snapshot.document(m_fileName);
+        if (doc.isNull())
+            return false;
+
         expression = m_expression;
 
         // Fetch the expression's code
@@ -729,6 +732,8 @@ Utils::Link CppElementEvaluator::linkFromExpression(const QString &expression, c
 {
     const Snapshot &snapshot = CppModelManager::instance()->snapshot();
     Document::Ptr doc = snapshot.document(fileName);
+    if (doc.isNull())
+        return Utils::Link();
     Scope *scope = doc->globalNamespace();
 
     TypeOfExpression typeOfExpression;