diff options
author | GriffinR <griffin.g.richards@gmail.com> | 2021-10-13 13:29:17 -0400 |
---|---|---|
committer | GriffinR <griffin.g.richards@gmail.com> | 2021-10-13 13:29:17 -0400 |
commit | 490300daab72fbc6bd5b68cc461bcfe45d531071 (patch) | |
tree | 31916b4a812d2d35ba8aa095fa416009dd654276 /src | |
parent | 1f7eb7ea010bede43d07e14b4ff21c609eaeb4ea (diff) |
Disallow negative sizes in SAVEBLOCK_CHUNK
Diffstat (limited to 'src')
-rw-r--r-- | src/save.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/save.c b/src/save.c index 3ce4025bf..2049ca9c1 100644 --- a/src/save.c +++ b/src/save.c @@ -39,11 +39,12 @@ // (u8 *)structure was removed from the first statement of the macro in Emerald // and Fire Red/Leaf Green. This is because malloc is used to allocate addresses // so storing the raw addresses should not be done in the offsets information. -#define SAVEBLOCK_CHUNK(structure, chunkNum) \ -{ \ - chunkNum * SECTOR_DATA_SIZE, \ - min(sizeof(structure) - chunkNum * SECTOR_DATA_SIZE, SECTOR_DATA_SIZE) \ -} \ +#define SAVEBLOCK_CHUNK(structure, chunkNum) \ +{ \ + chunkNum * SECTOR_DATA_SIZE, \ + sizeof(structure) >= chunkNum * SECTOR_DATA_SIZE ? \ + min(sizeof(structure) - chunkNum * SECTOR_DATA_SIZE, SECTOR_DATA_SIZE) : 0 \ +} const struct SaveSectionOffsets gSaveSectionOffsets[] = { |