[binutils-gdb] x86: adjust type checking constructs

Jan Beulich jbeulich@sourceware.org
Wed Dec 14 09:07:58 GMT 2022


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 == &current_templates->start);
-    (void)(end == &current_templates->end);
+    (void) sizeof (sets == &current_templates->start);
+    (void) sizeof (end == &current_templates->end);
     for (; sets < end; ++sets)
       if (str_hash_insert (op_hash, (*sets)->name, sets, 0))
 	as_fatal (_("duplicate %s"), (*sets)->name);


More information about the Binutils-cvs mailing list