From: Alan Modra <amodra@gmail.com>
Date: Tue, 10 Dec 2019 22:22:22 +0000 (+1030)
Subject: ubsan: tic6x: shift left of int
X-Git-Url: http://git.efficios.com/?a=commitdiff_plain;h=b84f6152ee8bb141acae181b34a51c3e58da8930;p=deliverable%2Fbinutils-gdb.git

ubsan: tic6x: shift left of int

	* tic6x-dis.c (tic6x_extract_32): Avoid signed overflow.
---

diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index 3ae93c1641..3ba885c48b 100644
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,7 @@
+2019-12-11  Alan Modra  <amodra@gmail.com>
+
+	* tic6x-dis.c (tic6x_extract_32): Avoid signed overflow.
+
 2019-12-11  Alan Modra  <amodra@gmail.com>
 
 	* tic4x-dis.c (tic4x_print_register): Formatting.  Don't segfault
diff --git a/opcodes/tic6x-dis.c b/opcodes/tic6x-dis.c
index 36075da115..45a1e98f2a 100644
--- a/opcodes/tic6x-dis.c
+++ b/opcodes/tic6x-dis.c
@@ -163,9 +163,9 @@ static unsigned int
 tic6x_extract_32 (unsigned char *p, struct disassemble_info *info)
 {
   if (info->endian == BFD_ENDIAN_LITTLE)
-    return (p[0]) | (p[1] << 8) | (p[2] << 16) | (p[3] << 24);
+    return p[0] | (p[1] << 8) | (p[2] << 16) | ((unsigned) p[3] << 24);
   else
-    return (p[3]) | (p[2] << 8) | (p[1] << 16) | (p[0] << 24);
+    return p[3] | (p[2] << 8) | (p[1] << 16) | ((unsigned) p[0] << 24);
 }
 
 /* Extract a 16-bit value read from the instruction stream.  */