From 831690f89b178d7bc47a25f1794bfd9633cad1ec Mon Sep 17 00:00:00 2001
From: Paul <pfultz2@yahoo.com>
Date: Thu, 16 Jul 2020 14:33:39 -0500
Subject: [PATCH] Use parseDecl instead

---
 lib/valueflow.cpp |  5 +++--
 test/teststl.cpp  | 22 ++++++++++++++++++++++
 2 files changed, 25 insertions(+), 2 deletions(-)

diff --git a/lib/valueflow.cpp b/lib/valueflow.cpp
index 57dadab82..e06bada7c 100644
--- a/lib/valueflow.cpp
+++ b/lib/valueflow.cpp
@@ -3625,8 +3625,9 @@ static void valueFlowLifetime(TokenList *tokenlist, SymbolDatabase*, ErrorLogger
 
             bool isContainerOfPointers = true;
             const Token* containerTypeToken = tok->valueType()->containerTypeToken;
-            if (containerTypeToken && Token::simpleMatch(containerTypeToken->previous(), "<") && containerTypeToken->previous()->link()) {
-                isContainerOfPointers = Token::simpleMatch(containerTypeToken->previous()->link()->previous(), "*");
+            if (containerTypeToken) {
+                ValueType vt = ValueType::parseDecl(containerTypeToken, settings);
+                isContainerOfPointers = vt.pointer > 0;
             }
 
             LifetimeStore ls;
diff --git a/test/teststl.cpp b/test/teststl.cpp
index ca054dfaf..d243f0cd7 100644
--- a/test/teststl.cpp
+++ b/test/teststl.cpp
@@ -4176,6 +4176,28 @@ private:
               "    delete b;\n"
               "}\n" ,true);
         ASSERT_EQUALS("", errout.str());
+
+        check("struct A {};\n"
+              "void f() {\n"
+              "    std::vector<A *, std::allocator<A*>> v;\n"
+              "    A *a = new A();\n"
+              "    v.push_back(a);\n"
+              "    A *b = v.back();\n"
+              "    v.pop_back();\n"
+              "    delete b;\n"
+              "}\n" ,true);
+        ASSERT_EQUALS("", errout.str());
+
+        check("struct A {};\n"
+              "void f() {\n"
+              "    std::vector<std::shared_ptr<A>> v;\n"
+              "    std::shared_ptr<A> a = std::make_shared<A>();\n"
+              "    v.push_back(a);\n"
+              "    std::shared_ptr<A> b = v.back();\n"
+              "    v.pop_back();\n"
+              "    delete b;\n"
+              "}\n" ,true);
+        ASSERT_EQUALS("", errout.str());
     }
 
     void invalidContainerLoop() {