Since fdshare() and fdinit() only operate on filedescs, make them

take pointers to filedesc structures instead of threads.  This makes
it more clear that they do not do any voodoo with the thread/proc
or anything other than the filedesc passed in or returned.

Remove some XXX KSE's as this resolves the issue.

1 files changed, 10 insertions, 10 deletions

diff --git a/sys/kern/kern_descrip.c b/sys/kern/kern_descrip.c
index dba34c50735c..fbc12e433ffd 100644
--- a/sys/kern/kern_descrip.c
+++ b/sys/kern/kern_descrip.c
@@ -1202,14 +1202,14 @@ ffree(fp)
 }
 
 /*
- * Build a new filedesc structure.
+ * Build a new filedesc structure from another.
+ * Copy the current, root, and jail root vnode references.
  */
 struct filedesc *
-fdinit(td)
-	struct thread *td;
+fdinit(fdp)
+	struct filedesc *fdp;
 {
 	register struct filedesc0 *newfdp;
-	register struct filedesc *fdp = td->td_proc->p_fd;
 
 	MALLOC(newfdp, struct filedesc0 *, sizeof(struct filedesc0),
 	    M_FILEDESC, M_WAITOK | M_ZERO);
@@ -1238,13 +1238,13 @@ fdinit(td)
  * Share a filedesc structure.
  */
 struct filedesc *
-fdshare(p)
-	struct proc *p;
+fdshare(fdp)
+	struct filedesc *fdp;
 {
-	FILEDESC_LOCK(p->p_fd);
-	p->p_fd->fd_refcnt++;
-	FILEDESC_UNLOCK(p->p_fd);
-	return (p->p_fd);
+	FILEDESC_LOCK(fdp);
+	fdp->fd_refcnt++;
+	FILEDESC_UNLOCK(fdp);
+	return (fdp);
 }
 
 /*