[coreboot-gerrit] Change in ...coreboot[master]: util/romcc: Use 64 bit integers when shifting

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/32854 )

Change subject: util/romcc: Use 64 bit integers when shifting
......................................................................

util/romcc: Use 64 bit integers when shifting

'used_indices' is 64 bits wide, so use a fixed-width type to make
that clear. As such, 'index' can have a value of up to 63, so use a
64 bit integer when doing the shifts to prevent overflow.

Found-by: Coverity Scan CID 1287090
Signed-off-by: Jacob Garber 
Change-Id: Ibd089df6be60c8ea46da11e5e83cd58b2e2c54d6
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32854
Reviewed-by: Patrick Georgi 
Tested-by: build bot (Jenkins) 
---
M util/romcc/romcc.c
1 file changed, 5 insertions(+), 5 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Patrick Georgi: Looks good to me, approved

diff --git a/util/romcc/romcc.c b/util/romcc/romcc.c
index c6507dc..97cc219 100644
--- a/util/romcc/romcc.c
+++ b/util/romcc/romcc.c
@@ -14111,7 +14111,7 @@
    struct block *block;
    struct triple *old_result, *first, *ins;
    size_t count, idx;
-	unsigned long used_indices;
+	uint64_t used_indices;
    int i, max_index;
 #define MAX_INDICES (sizeof(used_indices)*CHAR_BIT)
 #define ID_BITS(X) ((X) & (TRIPLE_FLAG_LOCAL -1))
@@ -14217,11 +14217,11 @@
    	if (index >= MAX_INDICES) {
    		internal_error(state, ins, "index unexpectedly large");
    	}
-		if (used_indices & (1 << index)) {
+		if (used_indices & ((uint64_t)1 << index)) {
    		internal_error(state, ins, "index previously used?");
    	}
    	/* Remember which indices have been used */
-		used_indices |= (1 << index);
+		used_indices |= ((uint64_t)1 << index);
    	if (index > max_index) {
    		max_index = index;
    	}
@@ -14249,7 +14249,7 @@
    	}
    	info[ID_BITS(ins->id)].index = index;
    	/* Remember which indices have been used */
-		used_indices |= (1 << index);
+		used_indices |= ((uint64_t)1 << index);
    	if (index > max_index) {
    		max_index = index;
    	}
@@ -14263,7 +14263,7 @@
    for(i = 0; i <= max_index; i++) {
    	struct triple *var;
    	var = 0;
-		if (used_indices & (1 << i)) {
+		if (used_indices & ((uint64_t)1 << i)) {
    		for(set = vars; set; set = set->next) {
    			int index;
    			index = info[ID_BITS(set->member->id)].index;