derek/gem5
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

sim-se: Use tgt_stat64 instead of tgt_stat in newfstatatFunc (#283)

Browse Source 
The syscall emulation of newfstatat incorrectly treated the output stat
buffer to be of type `OS::tgt_stat`, not `OS::tgt_stat64`, causing the
invalid output stat buffer in the application to hold invalid data.

This patch fixes the bug by simply substituting the type `OS::tgt_stat`
with `OS::tgt_stat64` in `newstatatFunc()`.

GitHub issue: https://github.com/gem5/gem5/issues/281
This commit is contained in:
Bobby R. Bruce
2023-09-12 08:33:42 -07:00
committed by GitHub
parent 94e5a0cccf 259a5d6272
commit 1bebf6a3cc
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/sim/syscall_emul.hh
View File

@@ -1375,7 +1375,7 @@ statFunc(SyscallDesc *desc, ThreadContext *tc,
template <class OS>
SyscallReturn
newfstatatFunc(SyscallDesc *desc, ThreadContext *tc, int dirfd,
VPtr<> pathname, VPtr<typename OS::tgt_stat> tgt_stat,
VPtr<> pathname, VPtr<typename OS::tgt_stat64> tgt_stat,
int flags)
{
std::string path;
@@ -1405,7 +1405,7 @@ newfstatatFunc(SyscallDesc *desc, ThreadContext *tc, int dirfd,
if (result < 0)
return -errno;
copyOutStatBuf<OS>(tgt_stat, &host_buf);
copyOutStat64Buf<OS>(tgt_stat, &host_buf);
return 0;
}