Aren/toaruos
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

tty: support setting controlling session

Browse Source 
This is... kinda backwards? But it works sufficiently for us.
Every TTY has at most one controlling session - every session
is supposed have at most one controlling terminal, too, but we
don't implement that part yet.
This commit is contained in:
K. Lange 2024-01-19 20:12:34 +09:00
parent 0d28bf0904
commit fa8a6e20ab
1 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
kernel/vfs/tty.c
View File

@ -320,6 +320,24 @@ int pty_ioctl(pty_t * pty, unsigned long request, void * argp) {
validate(argp); validate(argp);
*(pid_t *)argp = pty->fg_proc; *(pid_t *)argp = pty->fg_proc;
return 0; return 0;
case TIOCSCTTY:
/* If this is already the control session, quietly ignore. */
if (this_core->current_process->session == this_core->current_process->id &&
pty->ct_proc == this_core->current_process->session) {
return 0;
}
/* If we aren't a session leader, we can't do this. */
if (this_core->current_process->session != this_core->current_process->id) {
return -EPERM;
}
/* If there's already a control session, only root can steal control, and only if *argp is 1
* (on Linux, that's "if argp is 1", but we kinda messed this up by checking ioctl argp stuff
* for bounds validity in the system call layer, so instead we use a pointer to 1... */
if (pty->ct_proc && (!argp || (*(int*)argp != 1) || this_core->current_process->user != 0)) {
return -EPERM;
}
pty->ct_proc = this_core->current_process->session;
return 0;
case TCSETS: case TCSETS:
case TCSETSW: case TCSETSW:
if (!argp) return -EINVAL; if (!argp) return -EINVAL;