Skip to content

Commit

Permalink
helix-term: document safety of libc::kill
Browse files Browse the repository at this point in the history
  • Loading branch information
cole-h committed Dec 7, 2022
1 parent 3201342 commit 6d55e3c
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions helix-term/src/application.rs
Original file line number Diff line number Diff line change
Expand Up @@ -445,6 +445,11 @@ impl Application {
.ok();
restore_term().unwrap();

// SAFETY:
//
// - helix must have permissions to send signals to all processes in its signal
// group, either by already having the requisite permission, or by having the
// user's UID / EUID / SUID match that of the receiving process(es).
let res = unsafe {
// A pid of 0 sends the signal to the entire process group, allowing the user to
// regain control of their terminal if the editor was spawned under another process
Expand Down

0 comments on commit 6d55e3c

Please sign in to comment.