aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Malcolm <dmalcolm@redhat.com>2022-07-20 21:34:03 -0400
committerDavid Malcolm <dmalcolm@redhat.com>2022-07-20 21:34:03 -0400
commita6c192e80a87efbe6c0641f25a963c7bee9990fb (patch)
tree61aac71e79c36db33328a31f42f553f87f6218ea
parente7dfd8744502d6588483ec63ab7f81c2f5940267 (diff)
downloadgcc-a6c192e80a87efbe6c0641f25a963c7bee9990fb.zip
gcc-a6c192e80a87efbe6c0641f25a963c7bee9990fb.tar.gz
gcc-a6c192e80a87efbe6c0641f25a963c7bee9990fb.tar.bz2
analyzer: fix ICE on untracked decl_regions [PR106374]
gcc/analyzer/ChangeLog: PR analyzer/106374 * region.cc (decl_region::get_svalue_for_initializer): Bail out on untracked regions. gcc/testsuite/ChangeLog: PR analyzer/106374 * gcc.dg/analyzer/untracked-2.c: New test. Signed-off-by: David Malcolm <dmalcolm@redhat.com>
-rw-r--r--gcc/analyzer/region.cc5
-rw-r--r--gcc/testsuite/gcc.dg/analyzer/untracked-2.c7
2 files changed, 12 insertions, 0 deletions
diff --git a/gcc/analyzer/region.cc b/gcc/analyzer/region.cc
index a8d1ae9..b78bf4e 100644
--- a/gcc/analyzer/region.cc
+++ b/gcc/analyzer/region.cc
@@ -1152,6 +1152,11 @@ decl_region::get_svalue_for_initializer (region_model_manager *mgr) const
if (binding->symbolic_p ())
return NULL;
+ /* If we don't care about tracking the content of this region, then
+ it's unused, and the value doesn't matter. */
+ if (!tracked_p ())
+ return NULL;
+
binding_cluster c (this);
c.zero_fill_region (mgr->get_store_manager (), this);
return mgr->get_or_create_compound_svalue (TREE_TYPE (m_decl),
diff --git a/gcc/testsuite/gcc.dg/analyzer/untracked-2.c b/gcc/testsuite/gcc.dg/analyzer/untracked-2.c
new file mode 100644
index 0000000..565a9cc
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/analyzer/untracked-2.c
@@ -0,0 +1,7 @@
+typedef unsigned char u8;
+extern int foo(const u8 *key, unsigned int keylen);
+int test (void)
+{
+ static const u8 default_salt[64];
+ return foo(default_salt, 64);
+}