summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorGriffinR <griffin.g.richards@gmail.com>2021-10-13 13:29:17 -0400
committerGriffinR <griffin.g.richards@gmail.com>2021-10-13 13:29:17 -0400
commit490300daab72fbc6bd5b68cc461bcfe45d531071 (patch)
tree31916b4a812d2d35ba8aa095fa416009dd654276 /src
parent1f7eb7ea010bede43d07e14b4ff21c609eaeb4ea (diff)
Disallow negative sizes in SAVEBLOCK_CHUNK
Diffstat (limited to 'src')
-rw-r--r--src/save.c11
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[] =
{