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

[Jacob Garber (Code Review)]

Jacob Garber has uploaded this change for review. ( 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_indicies' 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
---
M util/romcc/romcc.c
1 file changed, 5 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/32854/1

diff --git a/util/romcc/romcc.c b/util/romcc/romcc.c
index 43be171..d1a2655 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_indicies;
+	uint64_t used_indicies;
    int i, max_index;
 #define MAX_INDICIES (sizeof(used_indicies)*CHAR_BIT)
 #define ID_BITS(X) ((X) & (TRIPLE_FLAG_LOCAL -1))
@@ -14217,11 +14217,11 @@
    	if (index >= MAX_INDICIES) {
    		internal_error(state, ins, "index unexpectedly large");
    	}
-		if (used_indicies & (1 << index)) {
+		if (used_indicies & ((uint64_t)1 << index)) {
    		internal_error(state, ins, "index previously used?");
    	}
    	/* Remember which indicies have been used */
-		used_indicies |= (1 << index);
+		used_indicies |= ((uint64_t)1 << index);
    	if (index > max_index) {
    		max_index = index;
    	}
@@ -14249,7 +14249,7 @@
    	}
    	info[ID_BITS(ins->id)].index = index;
    	/* Remember which indicies have been used */
-		used_indicies |= (1 << index);
+		used_indicies |= ((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_indicies & (1 << i)) {
+		if (used_indicies & ((uint64_t)1 << i)) {
    		for(set = vars; set; set = set->next) {
    			int index;
    			index = info[ID_BITS(set->member->id)].index;