T0b1/wasmtime
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix the path_filestat test on Linux (#706)

Browse Source 
Only very recently in #700 did we actually start running wasi tests
again (they weren't running by accident). Just before that landed we
also landed #688 which had some refactorings. Unfortunately #688 had a
minor issue in it which wasn't caught because tests weren't run. This
means that the bug in #688 slipped in and is now being caught by #700
now that both are landed on master.

This commit fixes the small issue introduced and should get our CI green
again!
This commit is contained in:
Alex Crichton
2019-12-12 15:19:58 -08:00
committed by GitHub
parent a5c30d7ffe
commit 054b79427e
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
crates/wasi-common/src/old/snapshot_0/sys/unix/linux/filetime.rs
View File

@@ -36,7 +36,7 @@ pub(crate) fn utimensat(
static INVALID: AtomicBool = AtomicBool::new(false); static INVALID: AtomicBool = AtomicBool::new(false);
if !INVALID.load(Relaxed) { if !INVALID.load(Relaxed) {
let p = CString::new(path.as_bytes())?; let p = CString::new(path.as_bytes())?;
let times = [to_timespec(&atime), to_timespec(&mtime)]; let times = [to_timespec(&atime)?, to_timespec(&mtime)?];
let rc = unsafe { let rc = unsafe {
libc::syscall( libc::syscall(
libc::SYS_utimensat, libc::SYS_utimensat,

2
crates/wasi-common/src/sys/unix/linux/filetime.rs
View File

@@ -36,7 +36,7 @@ pub(crate) fn utimensat(
static INVALID: AtomicBool = AtomicBool::new(false); static INVALID: AtomicBool = AtomicBool::new(false);
if !INVALID.load(Relaxed) { if !INVALID.load(Relaxed) {
let p = CString::new(path.as_bytes())?; let p = CString::new(path.as_bytes())?;
let times = [to_timespec(&atime), to_timespec(&mtime)]; let times = [to_timespec(&atime)?, to_timespec(&mtime)?];
let rc = unsafe { let rc = unsafe {
libc::syscall( libc::syscall(
libc::SYS_utimensat, libc::SYS_utimensat,