f7f10c12b3
* Fix path_open granting more rights than requested * Add missing rights checks in: fd_fdstat_set_flags, fd_filestat_get, poll_oneoff * Fix `open_scratch_directory` not requesting any rights. * Properly request needed rights in various tests * Add some extra trace-level logging * Remove a no-op restriction of rights to the ones returned by `determine_type_rights`. It was redundant, because `FdEntry:from` internally also called `determine_type_rights` and only dropped some of them.
76 lines
2.4 KiB
Rust
76 lines
2.4 KiB
Rust
use more_asserts::assert_gt;
|
|
use std::{env, process};
|
|
use wasi_tests::open_scratch_directory;
|
|
|
|
unsafe fn test_fd_filestat_set(dir_fd: wasi::Fd) {
|
|
// Create a file in the scratch directory.
|
|
let file_fd = wasi::path_open(
|
|
dir_fd,
|
|
0,
|
|
"file",
|
|
wasi::OFLAGS_CREAT,
|
|
wasi::RIGHTS_FD_READ
|
|
| wasi::RIGHTS_FD_WRITE
|
|
| wasi::RIGHTS_FD_FILESTAT_GET
|
|
| wasi::RIGHTS_FD_FILESTAT_SET_SIZE
|
|
| wasi::RIGHTS_FD_FILESTAT_SET_TIMES,
|
|
0,
|
|
0,
|
|
)
|
|
.expect("failed to create file");
|
|
assert_gt!(
|
|
file_fd,
|
|
libc::STDERR_FILENO as wasi::Fd,
|
|
"file descriptor range check",
|
|
);
|
|
|
|
// Check file size
|
|
let stat = wasi::fd_filestat_get(file_fd).expect("failed filestat");
|
|
assert_eq!(stat.size, 0, "file size should be 0");
|
|
|
|
// Check fd_filestat_set_size
|
|
wasi::fd_filestat_set_size(file_fd, 100).expect("fd_filestat_set_size");
|
|
|
|
let stat = wasi::fd_filestat_get(file_fd).expect("failed filestat 2");
|
|
assert_eq!(stat.size, 100, "file size should be 100");
|
|
|
|
// Check fd_filestat_set_times
|
|
let old_atim = stat.atim;
|
|
let new_mtim = stat.mtim - 100;
|
|
wasi::fd_filestat_set_times(file_fd, new_mtim, new_mtim, wasi::FSTFLAGS_MTIM)
|
|
.expect("fd_filestat_set_times");
|
|
|
|
let stat = wasi::fd_filestat_get(file_fd).expect("failed filestat 3");
|
|
assert_eq!(stat.size, 100, "file size should remain unchanged at 100");
|
|
assert_eq!(stat.mtim, new_mtim, "mtim should change");
|
|
assert_eq!(stat.atim, old_atim, "atim should not change");
|
|
|
|
// let status = wasi_fd_filestat_set_times(file_fd, new_mtim, new_mtim, wasi::FILESTAT_SET_MTIM | wasi::FILESTAT_SET_MTIM_NOW);
|
|
// assert_eq!(status, wasi::EINVAL, "ATIM & ATIM_NOW can't both be set");
|
|
|
|
wasi::fd_close(file_fd).expect("failed to close fd");
|
|
wasi::path_unlink_file(dir_fd, "file").expect("failed to remove dir");
|
|
}
|
|
fn main() {
|
|
let mut args = env::args();
|
|
let prog = args.next().unwrap();
|
|
let arg = if let Some(arg) = args.next() {
|
|
arg
|
|
} else {
|
|
eprintln!("usage: {} <scratch directory>", prog);
|
|
process::exit(1);
|
|
};
|
|
|
|
// Open scratch directory
|
|
let dir_fd = match open_scratch_directory(&arg) {
|
|
Ok(dir_fd) => dir_fd,
|
|
Err(err) => {
|
|
eprintln!("{}", err);
|
|
process::exit(1)
|
|
}
|
|
};
|
|
|
|
// Run the tests.
|
|
unsafe { test_fd_filestat_set(dir_fd) }
|
|
}
|