Nico Huber has uploaded this change for review.

View Change

Fix two sign-compare issues that sneaked in

Change-Id: If36ba22606021400b385ea6083eacc7b360c20c5
Signed-off-by: Nico Huber <nico.h@gmx.de>
---
M ich_descriptors.c
M libflashrom.c
2 files changed, 2 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/00/35800/1
diff --git a/ich_descriptors.c b/ich_descriptors.c
index 6d7b020..120d3fe 100644
--- a/ich_descriptors.c
+++ b/ich_descriptors.c
@@ -421,7 +421,7 @@
for (i = 0; i < nm; i++) {
size_t j;
msg_pdbg2("%-4s", master_names[i]);
- for (j = 0; j < min(num_regions, 12); j++)
+ for (j = 0; j < (size_t)min(num_regions, 12); j++)
msg_pdbg2(" %c%c ",
desc->master.mstr[i].read & (1 << j) ? 'r' : ' ',
desc->master.mstr[i].write & (1 << j) ? 'w' : ' ');
diff --git a/libflashrom.c b/libflashrom.c
index dbc5129..1d8a9ae 100644
--- a/libflashrom.c
+++ b/libflashrom.c
@@ -139,7 +139,7 @@
*/
struct flashrom_flashchip_info *flashrom_supported_flash_chips(void)
{
- int i = 0;
+ unsigned int i = 0;
struct flashrom_flashchip_info *supported_flashchips =
malloc(flashchips_size * sizeof(*supported_flashchips));


To view, visit change 35800. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: If36ba22606021400b385ea6083eacc7b360c20c5
Gerrit-Change-Number: 35800
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h@gmx.de>
Gerrit-MessageType: newchange