d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't use curthread to resolve file descriptor. Request may be queued, so

Browse Source 
thread will be different. Instead require sender to send process ID
together with file descriptor.
This commit is contained in:
Alexander Motin 2008-11-08 06:25:57 +00:00
parent 85ae3122e9
commit 45056382d0
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=184762
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
sys/netgraph/ng_tty.c
View File

@ -73,6 +73,7 @@
#include <sys/syslog.h>
#include <sys/tty.h>
#include <sys/ttycom.h>
#include <sys/proc.h>
#include <net/if.h>
#include <net/if_var.h>
@ -250,7 +251,8 @@ ngt_shutdown(node_p node)
static int
ngt_rcvmsg(node_p node, item_p item, hook_p lasthook)
{
struct thread *td = curthread; /* XXX */
struct proc *p;
struct thread *td;
const sc_p sc = NG_NODE_PRIVATE(node);
struct ng_mesg *msg, *resp = NULL;
int error = 0;
@ -262,8 +264,14 @@ ngt_rcvmsg(node_p node, item_p item, hook_p lasthook)
case NGM_TTY_SET_TTY:
if (sc->tp != NULL)
return (EBUSY);
error = ttyhook_register(&sc->tp, td, *(int *)msg->data,
p = pfind(((int *)msg->data)[0]);
if (p == NULL)
return (ESRCH);
td = FIRST_THREAD_IN_PROC(p);
error = ttyhook_register(&sc->tp, td, ((int *)msg->data)[1],
&ngt_hook, sc);
PROC_UNLOCK(p);
if (error != 0)
return (error);
break;