nir/spirv: improve lseek() error handling
authorEric Engestrom <eric@engestrom.ch>
Sun, 25 Sep 2016 15:49:51 +0000 (16:49 +0100)
committerJason Ekstrand <jason.ekstrand@intel.com>
Sat, 1 Oct 2016 22:27:44 +0000 (15:27 -0700)
Signed-off-by: Eric Engestrom <eric@engestrom.ch>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
src/compiler/spirv/spirv2nir.c

index d151c2cc8285e4e9917997775f3a5f4ed18d35ae..a02402874fc3a011a397724807e6c798bcea0901 100644 (file)
@@ -39,6 +39,8 @@
 #include <unistd.h>
 #include <stdio.h>
 
+#define WORD_SIZE 4
+
 int main(int argc, char **argv)
 {
    int fd = open(argv[1], O_RDONLY);
@@ -49,9 +51,15 @@ int main(int argc, char **argv)
    }
 
    off_t len = lseek(fd, 0, SEEK_END);
+   if (len % WORD_SIZE != 0)
+   {
+      fprintf(stderr, "File length isn't a multiple of the word size\n");
+      fprintf(stderr, "Are you sure this is a valid SPIR-V shader?\n");
+      close(fd);
+      return 1;
+   }
 
-   assert(len % 4 == 0);
-   size_t word_count = len / 4;
+   size_t word_count = len / WORD_SIZE;
 
    const void *map = mmap(NULL, len, PROT_READ, MAP_PRIVATE, fd, 0);
    assert(map != NULL);