diff options
author | Mete Balci <metebalci@gmail.com> | 2019-03-30 12:51:03 +0100 |
---|---|---|
committer | Tomas Vanek <vanekt@fbl.cz> | 2019-04-10 16:37:51 +0100 |
commit | 5b263d7b0c71d2560af5dabbeec051b807d9e6a3 (patch) | |
tree | d307a73f7272dc9e490dd51088537c02d9061ec2 /src | |
parent | d5936dc688bedf54848a29b7c171ef47deb2bf91 (diff) | |
download | riscv-openocd-5b263d7b0c71d2560af5dabbeec051b807d9e6a3.zip riscv-openocd-5b263d7b0c71d2560af5dabbeec051b807d9e6a3.tar.gz riscv-openocd-5b263d7b0c71d2560af5dabbeec051b807d9e6a3.tar.bz2 |
fix for sanitizer errors in left shifts
The modified lines cause "runtime error: left shift of <X> by <Y>
places cannot be represented in type 'int'", because integer
literal is cast to int by default.
Change-Id: Ie38119b5eb46ee470e0d149959e523b48ac4d66d
Signed-off-by: Mete Balci <metebalci@gmail.com>
Reviewed-on: http://openocd.zylin.com/5005
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Diffstat (limited to 'src')
-rw-r--r-- | src/helper/binarybuffer.h | 2 | ||||
-rw-r--r-- | src/jtag/core.c | 2 | ||||
-rw-r--r-- | src/target/armv8.c | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/src/helper/binarybuffer.h b/src/helper/binarybuffer.h index f1da8c4..7ac221e 100644 --- a/src/helper/binarybuffer.h +++ b/src/helper/binarybuffer.h @@ -118,7 +118,7 @@ static inline uint32_t buf_get_u32(const uint8_t *_buffer, uint32_t result = 0; for (unsigned i = first; i < first + num; i++) { if (((buffer[i / 8] >> (i % 8)) & 1) == 1) - result |= 1 << (i - first); + result |= 1U << (i - first); } return result; } diff --git a/src/jtag/core.c b/src/jtag/core.c index f90ae99..e57f62d 100644 --- a/src/jtag/core.c +++ b/src/jtag/core.c @@ -1007,7 +1007,7 @@ static bool jtag_examine_chain_match_tap(const struct jtag_tap *tap) return true; /* optionally ignore the JTAG version field - bits 28-31 of IDCODE */ - uint32_t mask = tap->ignore_version ? ~(0xf << 28) : ~0; + uint32_t mask = tap->ignore_version ? ~(0xfU << 28) : ~0U; uint32_t idcode = tap->idcode & mask; /* Loop over the expected identification codes and test for a match */ diff --git a/src/target/armv8.c b/src/target/armv8.c index c8cfcae..e271c1a 100644 --- a/src/target/armv8.c +++ b/src/target/armv8.c @@ -646,7 +646,7 @@ int armv8_read_mpidr(struct armv8_common *armv8) retval = dpm->instr_read_data_r0(dpm, armv8_opcode(armv8, READ_REG_MPIDR), &mpidr); if (retval != ERROR_OK) goto done; - if (mpidr & 1<<31) { + if (mpidr & 1U<<31) { armv8->multi_processor_system = (mpidr >> 30) & 1; armv8->cluster_id = (mpidr >> 8) & 0xf; armv8->cpu_id = mpidr & 0x3; |