diff options
author | David Malcolm <dmalcolm@redhat.com> | 2022-07-20 21:34:03 -0400 |
---|---|---|
committer | David Malcolm <dmalcolm@redhat.com> | 2022-07-20 21:34:03 -0400 |
commit | a6c192e80a87efbe6c0641f25a963c7bee9990fb (patch) | |
tree | 61aac71e79c36db33328a31f42f553f87f6218ea | |
parent | e7dfd8744502d6588483ec63ab7f81c2f5940267 (diff) | |
download | gcc-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.cc | 5 | ||||
-rw-r--r-- | gcc/testsuite/gcc.dg/analyzer/untracked-2.c | 7 |
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); +} |