aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Malcolm <dmalcolm@redhat.com>2022-11-22 17:29:21 -0500
committerDavid Malcolm <dmalcolm@redhat.com>2022-11-22 17:29:21 -0500
commitec7c796de020cb5cd955aa5b26c92b1da49d6076 (patch)
tree6812248cc7f2b1902be630657dfea27b4a433db6
parent64fb291c5839e1a82afb62743172b4eab1267399 (diff)
downloadgcc-ec7c796de020cb5cd955aa5b26c92b1da49d6076.zip
gcc-ec7c796de020cb5cd955aa5b26c92b1da49d6076.tar.gz
gcc-ec7c796de020cb5cd955aa5b26c92b1da49d6076.tar.bz2
analyzer: only look for named functions in root ns [PR107788]
gcc/analyzer/ChangeLog: PR analyzer/107788 * known-function-manager.cc (known_function_manager::get_match): Don't look up fndecls by name when they're not in the root namespace. gcc/testsuite/ChangeLog: PR analyzer/107788 * g++.dg/analyzer/named-functions.C: New test. Signed-off-by: David Malcolm <dmalcolm@redhat.com>
-rw-r--r--gcc/analyzer/known-function-manager.cc15
-rw-r--r--gcc/testsuite/g++.dg/analyzer/named-functions.C12
2 files changed, 24 insertions, 3 deletions
diff --git a/gcc/analyzer/known-function-manager.cc b/gcc/analyzer/known-function-manager.cc
index e17350d..c1074bc 100644
--- a/gcc/analyzer/known-function-manager.cc
+++ b/gcc/analyzer/known-function-manager.cc
@@ -91,6 +91,7 @@ known_function_manager::add (enum internal_fn ifn,
const known_function *
known_function_manager::get_match (tree fndecl, const call_details &cd) const
{
+ /* Look for a matching built-in. */
if (fndecl_built_in_p (fndecl, BUILT_IN_NORMAL))
{
if (const known_function *candidate
@@ -99,10 +100,18 @@ known_function_manager::get_match (tree fndecl, const call_details &cd) const
fndecl))
return candidate;
}
+
+ /* Look for a match by name. */
+
+ /* Reject fndecls that aren't in the root namespace. */
+ if (DECL_CONTEXT (fndecl)
+ && TREE_CODE (DECL_CONTEXT (fndecl)) != TRANSLATION_UNIT_DECL)
+ return NULL;
if (tree identifier = DECL_NAME (fndecl))
- if (const known_function *candidate = get_by_identifier (identifier))
- if (candidate->matches_call_types_p (cd))
- return candidate;
+ if (const known_function *candidate = get_by_identifier (identifier))
+ if (candidate->matches_call_types_p (cd))
+ return candidate;
+
return NULL;
}
diff --git a/gcc/testsuite/g++.dg/analyzer/named-functions.C b/gcc/testsuite/g++.dg/analyzer/named-functions.C
new file mode 100644
index 0000000..661a930
--- /dev/null
+++ b/gcc/testsuite/g++.dg/analyzer/named-functions.C
@@ -0,0 +1,12 @@
+#define NULL ((void *)0)
+
+namespace my
+{
+ int socket (int, int, int);
+};
+
+void test_my_socket ()
+{
+ /* This shouldn't match the known function "::socket". */
+ my::socket (0, 0, 0); /* { dg-bogus "leak" } */
+}