fabi/Xenia-Canary
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[Base/Posix] mapped_memory portability and correctness fixes

Browse Source 
Replace Linux-specific fstat64/ftruncate64 with portable POSIX equivalents.
Fix mmap error check to use MAP_FAILED instead of null pointer comparison.
Remove unconditional ftruncate after mmap that could corrupt files.
Add zero-length guard.
This commit is contained in:
Herman S.
2026-03-26 23:04:27 +09:00
parent 19e9d0c2de
commit cf013fdc15
1 changed files with 15 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
src/xenia/base/mapped_memory_posix.cc
View File

@@ -46,20 +46,26 @@ class PosixMappedMemory : public MappedMemory {
break;
}
struct stat file_stat;
if (fstat(file_descriptor, &file_stat)) {
close(file_descriptor);
return nullptr;
}
uint64_t file_size = uint64_t(file_stat.st_size);
size_t map_length = length;
if (!length) {
struct stat64 file_stat;
if (fstat64(file_descriptor, &file_stat)) {
close(file_descriptor);
return nullptr;
}
map_length = size_t(file_stat.st_size);
map_length = size_t(file_size);
}
if (!map_length) {
close(file_descriptor);
return nullptr;
}
void* data =
mmap(0, map_length, protection, MAP_SHARED, file_descriptor, offset);
ftruncate(file_descriptor, map_length);
if (!data) {
if (data == MAP_FAILED) {
close(file_descriptor);
return nullptr;
}
@@ -75,7 +81,7 @@ class PosixMappedMemory : public MappedMemory {
}
if (file_descriptor_ >= 0) {
if (truncate_size) {
ftruncate64(file_descriptor_, off64_t(truncate_size));
ftruncate(file_descriptor_, truncate_size);
}
close(file_descriptor_);
file_descriptor_ = -1;