soc_index is not only used for comparisons, but also as index into the
socs[] array. However, the array starts with SOC_TYPE_LS1046A and the
enum with SOC_TYPE_INVALID leading to socs[SOC_TYPE_LS1046A].soc
== SOC_TYPE_LS1028A and socs[SOC_TYPE_LS1028A] to overflow the array.
This broke LS1046A boot and LS1028A seems to have only worked by chance,
because the memory after socs[] happened to be zeroed.
Fix this by reordering the enum and while at it remove the redundancy in
having both the index and the soc member be the same value.
Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx>
---
scripts/pblimage.c | 14 ++++++--------
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/scripts/pblimage.c b/scripts/pblimage.c
index 8cb473d5bcf2..ef09b0f96084 100644
--- a/scripts/pblimage.c
+++ b/scripts/pblimage.c
@@ -57,30 +57,28 @@ static uint32_t pbi_crc_cmd1;
static uint32_t pbi_crc_cmd2;
enum soc_type {
- SOC_TYPE_INVALID,
+ SOC_TYPE_INVALID = -1,
SOC_TYPE_LS1046A,
SOC_TYPE_LS1028A,
};
struct soc_type_entry {
const char *name;
- enum soc_type soc;
bool big_endian;
};
static struct soc_type_entry socs[] = {
- {
+ [SOC_TYPE_LS1046A] = {
.name = "ls1046a",
- .soc = SOC_TYPE_LS1046A,
.big_endian = true,
- }, {
+ },
+ [SOC_TYPE_LS1028A] = {
.name = "ls1028a",
- .soc = SOC_TYPE_LS1028A,
.big_endian = false,
},
};
-static enum soc_type soc_type;
+static enum soc_type soc_type = SOC_TYPE_INVALID;
static char *rcwfile;
static char *pbifile;
@@ -413,7 +411,7 @@ int main(int argc, char *argv[])
for (i = 0; i < ARRAY_SIZE(socs); i++) {
if (!strcmp(socs[i].name, cputypestr)) {
- soc_type = socs[i].soc;
+ soc_type = i;
break;
}
}
--
2.39.2
