Travis: Fixed false positive in lib/checkbufferoverrun. Ticket: cppcheck-opensource#4486

Author: danmar

lib/check64bit.cpp

@@ -65,7 +65,7 @@ void Check64BitPortability::pointerassignment()
             continue;
 
         for (const Token* tok = scope->classStart->next(); tok != scope->classEnd; tok = tok->next()) {
-            if (Token::Match(tok, "return %var%|%num% [;+]")) {
+            if (Token::Match(tok, "return %var%|%num% [;+]") && !Token::simpleMatch(tok, "return 0 ;")) {
                 enum { NO, INT, PTR, PTRDIFF } type = NO;
                 for (const Token *tok2 = tok->next(); tok2; tok2 = tok2->next()) {
                     if ((type == NO || type == INT) && isaddr(tok2->variable()))

test/test64bit.cpp

@@ -151,6 +151,11 @@ class Test64BitPortability : public TestFixture {
               "}\n");
         ASSERT_EQUALS("", errout.str());
 
+        check("void* foo() {\n"
+              "    return 0;\n"
+              "}\n");
+        ASSERT_EQUALS("", errout.str());
+
         check("int foo(int i) {\n"
               "    return i;\n"
               "}\n");