fix: overflow in int before cast to size_t (#31)

sparky-game · May 25, 2024 · 38480fc · 38480fc
1 parent 7d7154f
commit 38480fc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/.github/workflows/build-linux.yaml b/.github/workflows/build-linux.yaml
@@ -7,6 +7,7 @@ on:
       - 'include/**'
       - 'src/**'
       - 'test/**'
+      - 'tools/**'
       - 'vendor/**'
       - 'Cargo.toml'
       - 'Cargo.lock'
@@ -20,6 +21,7 @@ on:
       - 'include/**'
       - 'src/**'
       - 'test/**'
+      - 'tools/**'
       - 'vendor/**'
       - 'Cargo.toml'
       - 'Cargo.lock'

diff --git a/tools/skap/src/skap.c b/tools/skap/src/skap.c
@@ -64,9 +64,9 @@ int main(void) {
     }
   }
   for (usz i = 0; i < ARRLEN(imgs_paths); ++i) {
-    skap_idx_image_link_blob(&img_idxs[i], ftell(fd), imgs[i].width * imgs[i].height);
+    skap_idx_image_link_blob(&img_idxs[i], ftell(fd), (usz) imgs[i].width * imgs[i].height);
     printf("  WRITE   %s >> " SKAP_FILENAME "\n", imgs_paths[i]);
-    if (fwrite(imgs[i].data, sizeof(Color), imgs[i].width * imgs[i].height, fd) != (usz) (imgs[i].width * imgs[i].height)) {
+    if (fwrite(imgs[i].data, sizeof(Color), (usz) imgs[i].width * imgs[i].height, fd) != (usz) imgs[i].width * imgs[i].height) {
       fprintf(stderr, "ERROR: unable to write to file\n");
       skap_file_destroy(fd);
       unload_imgs();