diff --git a/lib/checkbufferoverrun.cpp b/lib/checkbufferoverrun.cpp
index cdb0da490..6b7cb8637 100644
--- a/lib/checkbufferoverrun.cpp
+++ b/lib/checkbufferoverrun.cpp
@@ -514,7 +514,7 @@ void CheckBufferOverrun::parse_for_body(const Token *tok2, const ArrayInfo &arra
 
 
 
-void CheckBufferOverrun::checkFunctionParameter(const Token &tok, unsigned int par, const ArrayInfo &arrayInfo, std::list<std::string> functionNames)
+void CheckBufferOverrun::checkFunctionParameter(const Token &tok, unsigned int par, const ArrayInfo &arrayInfo)
 {
     // total_size : which parameter in function call takes the total size?
     std::map<std::string, unsigned int> total_size;
@@ -677,31 +677,21 @@ void CheckBufferOverrun::checkFunctionParameter(const Token &tok, unsigned int p
                         }
                     }
                 }
-
-                // Calling function..
-                if (Token::Match(ftok, "%var% (")) {
-                    ArrayInfo ai(arrayInfo);
-                    ai.varid(parameterVarId);
-                    checkFunctionCall(ftok, ai, functionNames);
-                }
             }
         }
     }
 }
 
 
-void CheckBufferOverrun::checkFunctionCall(const Token *tok, const ArrayInfo &arrayInfo, std::list<std::string> functionNames)
+void CheckBufferOverrun::checkFunctionCall(const Token *tok, const ArrayInfo &arrayInfo)
 {
-    if (std::find(functionNames.begin(), functionNames.end(), tok->str()) != functionNames.end())
-        return;
-    functionNames.push_back(tok->str());
 
     // 1st parameter..
     if (Token::Match(tok->tokAt(2), "%varid% ,|)", arrayInfo.varid()))
-        checkFunctionParameter(*tok, 1, arrayInfo, functionNames);
+        checkFunctionParameter(*tok, 1, arrayInfo);
     else if (Token::Match(tok->tokAt(2), "%varid% + %num% ,|)", arrayInfo.varid())) {
         const ArrayInfo ai(arrayInfo.limit(MathLib::toLongNumber(tok->strAt(4))));
-        checkFunctionParameter(*tok, 1, ai, functionNames);
+        checkFunctionParameter(*tok, 1, ai);
     }
 
     // goto 2nd parameter and check it..
@@ -714,10 +704,10 @@ void CheckBufferOverrun::checkFunctionCall(const Token *tok, const ArrayInfo &ar
             break;
         if (tok2->str() == ",") {
             if (Token::Match(tok2, ", %varid% ,|)", arrayInfo.varid()))
-                checkFunctionParameter(*tok, 2, arrayInfo, functionNames);
+                checkFunctionParameter(*tok, 2, arrayInfo);
             else if (Token::Match(tok2, ", %varid% + %num% ,|)", arrayInfo.varid())) {
                 const ArrayInfo ai(arrayInfo.limit(MathLib::toLongNumber(tok2->strAt(3))));
-                checkFunctionParameter(*tok, 2, ai, functionNames);
+                checkFunctionParameter(*tok, 2, ai);
             }
             break;
         }
@@ -911,9 +901,9 @@ void CheckBufferOverrun::checkScope(const Token *tok, const std::vector<std::str
         // memset, memcmp, memcpy, strncpy, fgets..
         if (varid == 0 && size > 0) {
             if (Token::Match(tok, ("%var% ( " + varnames + " ,").c_str()))
-                checkFunctionParameter(*tok, 1, arrayInfo, std::list<std::string>());
+                checkFunctionParameter(*tok, 1, arrayInfo);
             if (Token::Match(tok, ("%var% ( %var% , " + varnames + " ,").c_str()))
-                checkFunctionParameter(*tok, 2, arrayInfo, std::list<std::string>());
+                checkFunctionParameter(*tok, 2, arrayInfo);
         }
 
         // Loop..
@@ -985,8 +975,7 @@ void CheckBufferOverrun::checkScope(const Token *tok, const std::vector<std::str
                 continue;
 
             const ArrayInfo arrayInfo1(varid, varnames, total_size / size, size);
-            const std::list<std::string> functionNames;
-            checkFunctionCall(tok, arrayInfo1, functionNames);
+            checkFunctionCall(tok, arrayInfo1);
         }
 
         // undefined behaviour: result of pointer arithmetic is out of bounds
@@ -1110,7 +1099,7 @@ void CheckBufferOverrun::checkScope(const Token *tok, const ArrayInfo &arrayInfo
 
         // Check function call..
         if (Token::Match(tok, "%var% (")) {
-            checkFunctionCall(tok, arrayInfo, std::list<std::string>());
+            checkFunctionCall(tok, arrayInfo);
         }
 
         if (Token::Match(tok, "strncpy|memcpy|memmove ( %varid% , %str% , %num% )", arrayInfo.varid())) {
diff --git a/lib/checkbufferoverrun.h b/lib/checkbufferoverrun.h
index 1739aeda7..0b4539669 100644
--- a/lib/checkbufferoverrun.h
+++ b/lib/checkbufferoverrun.h
@@ -203,17 +203,15 @@ public:
      * \param tok token for the function name
      * \param par on what parameter is the array used
      * \param arrayInfo the array information
-     * \param functionNames function names (to prevent endless recursion)
      */
-    void checkFunctionParameter(const Token &tok, const unsigned int par, const ArrayInfo &arrayInfo, std::list<std::string> functionNames);
+    void checkFunctionParameter(const Token &tok, const unsigned int par, const ArrayInfo &arrayInfo);
 
     /**
      * Helper function that checks if the array is used and if so calls the checkFunctionCall
      * @param tok token that matches "%var% ("
      * @param arrayInfo the array information
-     * \param functionNames function names (to prevent endless recursion)
      */
-    void checkFunctionCall(const Token *tok, const ArrayInfo &arrayInfo, std::list<std::string> functionNames);
+    void checkFunctionCall(const Token *tok, const ArrayInfo &arrayInfo);
 
     void arrayIndexOutOfBoundsError(const Token *tok, const ArrayInfo &arrayInfo, const std::vector<MathLib::bigint> &index);
     void arrayIndexOutOfBoundsError(const std::list<const Token *> &callstack, const ArrayInfo &arrayInfo, const std::vector<MathLib::bigint> &index);
diff --git a/test/testbufferoverrun.cpp b/test/testbufferoverrun.cpp
index 85054fcb5..202ac5147 100644
--- a/test/testbufferoverrun.cpp
+++ b/test/testbufferoverrun.cpp
@@ -33,6 +33,8 @@ public:
 
 private:
 
+
+
     void check(const char code[], bool experimental = true, const std::string &filename="test.cpp") {
         // Clear the error buffer..
         errout.str("");
@@ -722,14 +724,6 @@ private:
               "    foo(p+1);\n"
               "}\n");
         ASSERT_EQUALS("", errout.str());
-
-        // #3168
-        check("void a(char *p) { memset(p,0,100); }\n"
-              "void b() {\n"
-              "    char buf[10];\n"
-              "    a(buf);"
-              "}");
-        ASSERT_EQUALS("[test.cpp:1]: (error) Buffer access out-of-bounds: buf\n", errout.str());
     }