diff --git a/lib/checkautovariables.cpp b/lib/checkautovariables.cpp index 8dca0990636..6e4322ebfad 100644 --- a/lib/checkautovariables.cpp +++ b/lib/checkautovariables.cpp @@ -303,7 +303,7 @@ void CheckAutoVariables::autoVariables() if (v.isImpossible()) continue; if ((v.isTokValue() && (isArrayVar(v.tokvalue) || ((v.tokvalue->tokType() == Token::eString)))) || - (v.isLocalLifetimeValue() && v.lifetimeKind == ValueFlow::Value::LifetimeKind::Address)) { + (v.isLocalLifetimeValue() && v.lifetimeKind == ValueFlow::Value::LifetimeKind::Address && !Token::simpleMatch(v.tokvalue, "("))) { errorInvalidDeallocation(tok, &v); break; } diff --git a/test/testautovariables.cpp b/test/testautovariables.cpp index c4f8aa108f5..5fbe371f6a3 100644 --- a/test/testautovariables.cpp +++ b/test/testautovariables.cpp @@ -847,6 +847,12 @@ class TestAutoVariables : public TestFixture { " return 0;\n" "}\n"); ASSERT_EQUALS("", errout.str()); + + check("void f() {\n" + " S* p = &g();\n" + " delete p;\n" + "}\n"); + ASSERT_EQUALS("", errout.str()); } void testinvaliddealloc_input() {