* [binutils-gdb] x86: adjust type checking constructs
@ 2022-12-14 9:07 Jan Beulich
0 siblings, 0 replies; only message in thread
From: Jan Beulich @ 2022-12-14 9:07 UTC (permalink / raw)
To: bfd-cvs
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=eb99386180bb187f3ef3a59b222f16c8dfe3b81e
commit eb99386180bb187f3ef3a59b222f16c8dfe3b81e
Author: Jan Beulich <jbeulich@suse.com>
Date: Wed Dec 14 10:07:44 2022 +0100
x86: adjust type checking constructs
As Alan points out, ASAN takes issue with these constructs, for
current_templates being NULL. Wrap them in sizeof(), so the expressions
aren't actually evaluated.
Diff:
---
gas/config/tc-i386.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index ffe97d9d10e..990f4b9fec2 100644
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -2985,8 +2985,8 @@ md_begin (void)
/* Type checks to compensate for the conversion through void * which
occurs during hash table insertion / lookup. */
- (void)(sets == ¤t_templates->start);
- (void)(end == ¤t_templates->end);
+ (void) sizeof (sets == ¤t_templates->start);
+ (void) sizeof (end == ¤t_templates->end);
for (; sets < end; ++sets)
if (str_hash_insert (op_hash, (*sets)->name, sets, 0))
as_fatal (_("duplicate %s"), (*sets)->name);
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2022-12-14 9:07 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-14 9:07 [binutils-gdb] x86: adjust type checking constructs Jan Beulich
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).