Mirrors/vuinputd
1
0
Fork 0
mirror of https://github.com/joleuger/vuinputd.git synced 2026-07-17 16:36:03 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix potential race

Browse source
This commit is contained in:
Johannes Leupolz 2025-12-18 21:36:13 +00:00
parent 22ac4c23b7
commit e3094f898b
2 changed files with 4 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

1
docs/DESIGN.md
View file

@ -481,6 +481,7 @@ Under these constraints, post-exec namespace switching provides a robust and pre
at the time of `fork()`, the child process may block forever on its first allocation,
including implicit allocations inside libc or Rust runtime code.
See also [https://github.com/rust-lang/rust/blob/c1e865c/src/libstd/sys/unix/process.rs#L202
and https://systemd.io/ARCHITECTURE/ .
The chosen approach offers the best balance between correctness, portability, and operational simplicity.

9
vuinputd/src/cuse_device/vuinput_ioctl.rs
View file

@ -198,6 +198,7 @@ pub unsafe extern "C" fn vuinput_ioctl(
.dispatch(Box::new(mknod_job));
awaiter(&jobs::mknod_device_in_container_job::State::Finished);
debug!("fh {}: mknod_device in container has been finished ", fh);
fuse_lowlevel::fuse_reply_ioctl(_req, 0, std::ptr::null(), 0);
// we do not wait for the udev stuff
let emit_udev_event_job = EmitUdevEventInContainerJob::new(
@ -213,13 +214,9 @@ pub unsafe extern "C" fn vuinput_ioctl(
.lock()
.unwrap()
.dispatch(Box::new(emit_udev_event_job));
} else {
fuse_lowlevel::fuse_reply_ioctl(_req, 0, std::ptr::null(), 0);
}
// write a SYN-event (which is just zeros) just for validation
let syn_event: [u8; 24] = [0; 24];
vuinput_state.file.write_all(&syn_event).unwrap();
fuse_lowlevel::fuse_reply_ioctl(_req, 0, std::ptr::null(), 0);
}
UI_DEV_DESTROY => {
debug!("fh {}: ioctl UI_DEV_DESTROY", fh);