mpo/VVVVVV
1
0
Fork 0
mirror of https://github.com/TerryCavanagh/VVVVVV.git synced 2024-12-22 17:49:43 +01:00
Code Issues Releases Wiki Activity

Handle cases where less bytes are read than expected

Browse source 
This checks the return value of PHYSFS_readBytes() in all cases, and
uses a wrapper to not duplicate common logic. If the read fails, then it
will vlog an error, else if the amount of bytes read was less than
expected, it will vlog a warning.

Additionally, the space allocated for a file in
FILESYSTEM_loadFileToMemory is SDL_calloc()ed instead of SDL_malloc()ed
so if there are less bytes than expected, the memory will at least be
zeroed. This also means we don't have to do the null termination,
because the last byte will already be zeroed.

The return value of PHYSFS_readBytes() when reading the headers of
binary blobs now assigns to `header->size`, so the call has been placed
after the increment to `offset` because `offset` needs the correct (i.e.
intended) size of the header.
This commit is contained in:
Misa 2023-05-15 17:37:07 -07:00
parent 90c1a39231
commit 8036471e76
1 changed files with 39 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

46
desktop_version/src/FileSystemUtils.cpp
View file

@ -725,6 +725,38 @@ static void load_stdin(void)
stdin_length = pos - 1; stdin_length = pos - 1;
} }
static PHYSFS_sint64 read_bytes(
const char* name, PHYSFS_File* handle, void* buffer,
const PHYSFS_uint64 length
) {
const PHYSFS_sint64 bytes_read = PHYSFS_readBytes(handle, buffer, length);
if (bytes_read < 0)
{
vlog_error(
"Could not read bytes from file %s: %s",
name,
PHYSFS_getErrorByCode(PHYSFS_getLastErrorCode())
);
}
else if ((unsigned) bytes_read != length)
{
const char* reason;
if (PHYSFS_eof(handle))
{
reason = "Unexpected EOF";
}
else
{
reason = PHYSFS_getErrorByCode(PHYSFS_getLastErrorCode());
}
vlog_warn(
"Partially read file %s: Expected %lli bytes, got %lli: %s",
name, length, bytes_read, reason
);
}
return bytes_read;
}
void FILESYSTEM_loadFileToMemory( void FILESYSTEM_loadFileToMemory(
const char *name, const char *name,
unsigned char **mem, unsigned char **mem,
@ -732,7 +764,7 @@ void FILESYSTEM_loadFileToMemory(
) { ) {
PHYSFS_File *handle; PHYSFS_File *handle;
PHYSFS_sint64 length; PHYSFS_sint64 length;
PHYSFS_sint64 success; PHYSFS_sint64 bytes_read;
if (name == NULL || mem == NULL) if (name == NULL || mem == NULL)
{ {
@ -784,15 +816,14 @@ void FILESYSTEM_loadFileToMemory(
*len = length; *len = length;
} }
*mem = (unsigned char *) SDL_malloc(length + 1); *mem = (unsigned char *) SDL_calloc(length + 1, 1);
if (*mem == NULL) if (*mem == NULL)
{ {
VVV_exit(1); VVV_exit(1);
} }
(*mem)[length] = 0;
success = PHYSFS_readBytes(handle, *mem, length); bytes_read = read_bytes(name, handle, *mem, length);
if (success == -1) if (bytes_read < 0)
{ {
VVV_free(*mem); VVV_free(*mem);
} }
@ -850,7 +881,8 @@ bool FILESYSTEM_loadBinaryBlob(binaryBlob* blob, const char* filename)
size = PHYSFS_fileLength(handle); size = PHYSFS_fileLength(handle);
PHYSFS_readBytes( read_bytes(
filename,
handle, handle,
&blob->m_headers, &blob->m_headers,
sizeof(blob->m_headers) sizeof(blob->m_headers)
@ -887,8 +919,8 @@ bool FILESYSTEM_loadBinaryBlob(binaryBlob* blob, const char* filename)
{ {
VVV_exit(1); /* Oh god we're out of memory, just bail */ VVV_exit(1); /* Oh god we're out of memory, just bail */
} }
PHYSFS_readBytes(handle, *memblock, header->size);
offset += header->size; offset += header->size;
header->size = read_bytes(filename, handle, *memblock, header->size);
valid += 1; valid += 1;
continue; continue;