diff options
| author | Patrick Georgi <pgeorgi@chromium.org> | 2017-01-02 18:47:50 +0100 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-01-06 18:40:04 +0100 |
| commit | 766c3fec2d8cfc8d0d7bcfd94f98bace8d3309e9 (patch) | |
| tree | dc3bd03d8d0e9697c0a242d563316b3e864ebdd9 | |
| parent | 6d0c65ebc6c0f1e8b8d012dcfd42512b7d281515 (diff) | |
util/romcc: avoid shifting more than the variable's width
That's undefined behavior in C
Change-Id: I671ed8abf02e57a7cc993d1a85354e905f51717d
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Found-by: Coverity Scan #1229557
Reviewed-on: https://review.coreboot.org/18014
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
| -rw-r--r-- | util/romcc/romcc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/util/romcc/romcc.c b/util/romcc/romcc.c index b08f8e888a..aa41ccf2ce 100644 --- a/util/romcc/romcc.c +++ b/util/romcc/romcc.c @@ -10021,13 +10021,13 @@ static void simplify_copy(struct compile_state *state, struct triple *ins) /* Ensure I am properly sign extended */ if (size_of(state, right->type) < size_of(state, ins->type) && is_signed(right->type)) { - long_t val; + uint64_t val; int shift; shift = SIZEOF_LONG - size_of(state, right->type); val = left; val <<= shift; val >>= shift; - left = val; + left = (ulong_t)val; } mkconst(state, ins, left); break; |