From: Jakub Jelinek <jakub@redhat.com>
Date: Wed, 16 Oct 2024 15:45:19 +0000 (+0200)
Subject: c: Fix up uninitialized next.original_type use in #embed optimization
X-Git-Tag: basepoints/gcc-16~5164
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f5224caf53a4f17b190497c00c505977d358bef9;p=thirdparty%2Fgcc.git

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.
---

diff --git a/gcc/c/c-parser.cc b/gcc/c/c-parser.cc
index 120f2b289c0..e4381044e5c 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