Skip to content

Commit aa3fb3f

Browse files
committed
Changed set|get_sector_flag argument to uint16_t
On some platforms with very small sector size this index could overflow
1 parent d5783fe commit aa3fb3f

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

include/image.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,9 +66,9 @@ int wolfBoot_open_image(struct wolfBoot_image *img, uint8_t part);
6666
int wolfBoot_verify_integrity(struct wolfBoot_image *img);
6767
int wolfBoot_verify_authenticity(struct wolfBoot_image *img);
6868
int wolfBoot_set_partition_state(uint8_t part, uint8_t newst);
69-
int wolfBoot_set_sector_flag(uint8_t part, uint8_t sector, uint8_t newflag);
69+
int wolfBoot_set_sector_flag(uint8_t part, uint16_t sector, uint8_t newflag);
7070
int wolfBoot_get_partition_state(uint8_t part, uint8_t *st);
71-
int wolfBoot_get_sector_flag(uint8_t part, uint8_t sector, uint8_t *flag);
71+
int wolfBoot_get_sector_flag(uint8_t part, uint16_t sector, uint8_t *flag);
7272

7373
uint8_t* wolfBoot_peek_image(struct wolfBoot_image *img, uint32_t offset, uint32_t* sz);
7474

src/libwolfboot.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -207,7 +207,7 @@ int RAMFUNCTION wolfBoot_set_partition_state(uint8_t part, uint8_t newst)
207207
return 0;
208208
}
209209

210-
int RAMFUNCTION wolfBoot_set_sector_flag(uint8_t part, uint8_t sector, uint8_t newflag)
210+
int RAMFUNCTION wolfBoot_set_sector_flag(uint8_t part, uint16_t sector, uint8_t newflag)
211211
{
212212
uint32_t *magic;
213213
uint8_t *flags;
@@ -238,7 +238,7 @@ int RAMFUNCTION wolfBoot_get_partition_state(uint8_t part, uint8_t *st)
238238
return 0;
239239
}
240240

241-
int wolfBoot_get_sector_flag(uint8_t part, uint8_t sector, uint8_t *flag)
241+
int wolfBoot_get_sector_flag(uint8_t part, uint16_t sector, uint8_t *flag)
242242
{
243243
uint32_t *magic;
244244
uint8_t *flags;

0 commit comments

Comments
 (0)