Bug 26509 - UBSAN error: gas/config/tc-z80.c:3656:23:
Summary: UBSAN error: gas/config/tc-z80.c:3656:23:
Status: RESOLVED FIXED
Alias: None
Product: binutils
Classification: Unclassified
Component: binutils (show other bugs)
Version: 2.34
: P2 normal
Target Milestone: 2.36
Assignee: Alan Modra
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-08-20 12:46 UTC by Martin Liska
Modified: 2020-08-31 11:00 UTC (History)
0 users

See Also:
Host:
Target:
Build:
Last reconfirmed: 2020-08-30 00:00:00


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Liska 2020-08-20 12:46:32 UTC
The following fails when I build binutils with:
configure --build=x86_64-linux --disable-nls --disable-gdb --disable-gdbserver --disable-sim --disable-readline --disable-libdecnumber --enable-obsolete --target=z80-coff CFLAGS="-g -O2 -fsanitize=address,undefined -Wno-error" CXXLAGS="-g -O2 -fsanitize=address,undefined -Wno-error" LDFLAGS="-ldl"
Target: z80-coff
| 	exit( YY_EXIT_FAILURE );
Executing on host: sh -c {../as-new    /home/mliska/Programming/binutils/gas/testsuite/gas/all/p2425.s 2>&1}  /dev/null gas.out (timeout = 300)
/home/mliska/Programming/binutils/gas/config/tc-z80.c:3656:23: runtime error: shift exponent 32 is too large for 32-bit type 'int'
Comment 1 Sourceware Commits 2020-08-31 10:59:01 UTC
The master branch has been updated by Alan Modra <amodra@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=26e3de8e0a2d95e02b9dec46c67531755303fc4b

commit 26e3de8e0a2d95e02b9dec46c67531755303fc4b
Author: Alan Modra <amodra@gmail.com>
Date:   Mon Aug 31 08:13:19 2020 +0930

    PR26509 UBSAN: tc-z80.c:3656 shift exponent is too large
    
            PR 26509
            * config/tc-z80.c (is_overflow): Use 1UL in mask shift expression.
Comment 2 Alan Modra 2020-08-31 11:00:32 UTC
Fixed