diff options
author | David Malcolm <dmalcolm@redhat.com> | 2020-04-28 10:52:45 -0400 |
---|---|---|
committer | David Malcolm <dmalcolm@redhat.com> | 2020-04-28 13:26:22 -0400 |
commit | 5eae0ac76dcb6aac1d1d6c4edd8852e0035792e4 (patch) | |
tree | ba5b675ec69939002165d48396d71a2b871701f4 /gcc/analyzer/ChangeLog | |
parent | 0e8e1a6d97cc44d47992e40198490f780fbbfd5a (diff) |
analyzer: fix ICE copying struct [PR 94816]
PR analyzer/94816 reports an ICE when attempting to copy a struct
containing a field for which add_region_for_type for fails (on
an OFFSET_TYPE): the region for the src field comes from
make_region_for_unexpected_tree_code which gives it a NULL type, and
then the copy calls add_region_for_type which unconditionally
dereferences the NULL type.
This patch fixes the ICE by checking for NULL types in
add_region_for_type.
gcc/analyzer/ChangeLog:
PR analyzer/94816
* engine.cc (impl_region_model_context::on_unexpected_tree_code):
Handle NULL tree.
* region-model.cc (region_model::add_region_for_type): Handle
NULL type.
* region-model.h
(test_region_model_context::on_unexpected_tree_code): Handle NULL
tree.
gcc/testsuite/ChangeLog:
PR analyzer/94816
* g++.dg/analyzer/pr94816.C: New test.
Diffstat (limited to 'gcc/analyzer/ChangeLog')
-rw-r--r-- | gcc/analyzer/ChangeLog | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/gcc/analyzer/ChangeLog b/gcc/analyzer/ChangeLog index 3c8f45883a4..5cd736385aa 100644 --- a/gcc/analyzer/ChangeLog +++ b/gcc/analyzer/ChangeLog @@ -1,5 +1,16 @@ 2020-04-28 David Malcolm <dmalcolm@redhat.com> + PR analyzer/94816 + * engine.cc (impl_region_model_context::on_unexpected_tree_code): + Handle NULL tree. + * region-model.cc (region_model::add_region_for_type): Handle + NULL type. + * region-model.h + (test_region_model_context::on_unexpected_tree_code): Handle NULL + tree. + +2020-04-28 David Malcolm <dmalcolm@redhat.com> + PR analyzer/94447 PR analyzer/94639 PR analyzer/94732 |