FIX: Fix possible division by zero

- closes https://github.com/assimp/assimp/issues/4819
pull/4820/head
Kim Kulling 2022-12-02 09:24:51 +01:00 committed by GitHub
parent b0987f4513
commit 62486c6e7c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 5 deletions

View File

@ -405,6 +405,7 @@ void MDLImporter::InternReadFile_Quake1() {
} }
// go to the end of the skin section / the beginning of the next skin // go to the end of the skin section / the beginning of the next skin
bool overflow = false; bool overflow = false;
if (pcHeader->skinwidth != 0 || pcHeader->skinheight != 0) {
if ((pcHeader->skinheight > INT_MAX / pcHeader->skinwidth) || (pcHeader->skinwidth > INT_MAX / pcHeader->skinheight)){ if ((pcHeader->skinheight > INT_MAX / pcHeader->skinwidth) || (pcHeader->skinwidth > INT_MAX / pcHeader->skinheight)){
overflow = true; overflow = true;
} }
@ -412,6 +413,7 @@ void MDLImporter::InternReadFile_Quake1() {
szCurrent += pcHeader->skinheight * pcHeader->skinwidth +sizeof(float) * iNumImages; szCurrent += pcHeader->skinheight * pcHeader->skinwidth +sizeof(float) * iNumImages;
} }
} }
}
} else { } else {
szCurrent += sizeof(uint32_t); szCurrent += sizeof(uint32_t);
unsigned int iSkip = i ? UINT_MAX : 0; unsigned int iSkip = i ? UINT_MAX : 0;