diff options
author | Jakub Jelinek <jakub@redhat.com> | 2024-10-16 17:45:19 +0200 |
---|---|---|
committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2024-10-16 17:45:19 +0200 |
commit | f5224caf53a4f17b190497c00c505977d358bef9 (patch) | |
tree | 712d5c336c093ae24c07e5f7a3ac9adb7dd09ca8 /gcc | |
parent | ee4fdda70f1080bba5e49cadebc44333e19edeb4 (diff) | |
download | gcc-f5224caf53a4f17b190497c00c505977d358bef9.zip gcc-f5224caf53a4f17b190497c00c505977d358bef9.tar.gz gcc-f5224caf53a4f17b190497c00c505977d358bef9.tar.bz2 |
c: Fix up uninitialized next.original_type use in #embed optimization
Jonathan pointed me at a diagnostic from an unnamed static analyzer
which found that next.original_type isn't initialized for the CPP_EMBED
case when it is parsed in a comma expression, yet
expr.original_type = next.original_type;
is done a few lines later and the expr is returned.
2024-10-16 Jakub Jelinek <jakub@redhat.com>
* c-parser.cc (c_parser_expression): Initialize next.original_type
to integer_type_node for the CPP_EMBED case.
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/c/c-parser.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/gcc/c/c-parser.cc b/gcc/c/c-parser.cc index 120f2b2..e438104 100644 --- a/gcc/c/c-parser.cc +++ b/gcc/c/c-parser.cc @@ -13299,6 +13299,7 @@ c_parser_expression (c_parser *parser) next.value = build_int_cst (TREE_TYPE (val), ((const unsigned char *) RAW_DATA_POINTER (val))[last]); + next.original_type = integer_type_node; c_parser_consume_token (parser); } else |