From 38af865560c87df2c3eec2432163cf11b8236b38 Mon Sep 17 00:00:00 2001
From: PKEuS <philipp.kloke@web.de>
Date: Mon, 29 Sep 2014 16:49:20 +0200
Subject: [PATCH] Fixed false positive: Member variable not initialized in the
 constructor with array and (*this) (#5754)

---
 lib/checkclass.cpp        |  4 +++-
 test/testconstructors.cpp | 11 +++++++++++
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/lib/checkclass.cpp b/lib/checkclass.cpp
index 891714cc8..ffecf5748 100644
--- a/lib/checkclass.cpp
+++ b/lib/checkclass.cpp
@@ -528,7 +528,7 @@ void CheckClass::initializeVarList(const Function &func, std::list<const Functio
         }
 
         if (!Token::Match(ftok->next(), "::| %var%") &&
-            !Token::Match(ftok->next(), "this . %var%") &&
+            !Token::Match(ftok->next(), "*| this . %var%") &&
             !Token::Match(ftok->next(), "* %var% =") &&
             !Token::Match(ftok->next(), "( * this ) . %var%"))
             continue;
@@ -700,6 +700,8 @@ void CheckClass::initializeVarList(const Function &func, std::list<const Functio
         // Assignment of array item of member variable?
         else if (Token::Match(ftok, "* %var% =")) {
             assignVar(ftok->next()->str(), scope, usage);
+        } else if (Token::Match(ftok, "* this . %var% =")) {
+            assignVar(ftok->strAt(3), scope, usage);
         }
 
         // The functions 'clear' and 'Clear' are supposed to initialize variable.
diff --git a/test/testconstructors.cpp b/test/testconstructors.cpp
index 3b601be5f..c76f4c85c 100644
--- a/test/testconstructors.cpp
+++ b/test/testconstructors.cpp
@@ -2171,6 +2171,17 @@ private:
               "    char name[255];\n"
               "};");
         ASSERT_EQUALS("", errout.str());
+
+        // #5754
+        check("class John\n"
+              "{\n"
+              "public:\n"
+              "    John() {*this->name = '\\0';}\n"
+              "\n"
+              "private:\n"
+              "    char name[255];\n"
+              "};");
+        ASSERT_EQUALS("", errout.str());
     }
 
     void uninitVarArray3() {