svn commit: r274007 - head/sys/kern
Mateusz Guzik
mjg at FreeBSD.org
Mon Nov 3 05:12:18 UTC 2014
Author: mjg
Date: Mon Nov 3 05:12:17 2014
New Revision: 274007
URL: https://svnweb.freebsd.org/changeset/base/274007
Log:
filedesc: plug unnecessary fdp NULL checks in fdescfreee and fdcopy
Anything reaching these functions has fd table.
Modified:
head/sys/kern/kern_descrip.c
Modified: head/sys/kern/kern_descrip.c
==============================================================================
--- head/sys/kern/kern_descrip.c Mon Nov 3 04:41:29 2014 (r274006)
+++ head/sys/kern/kern_descrip.c Mon Nov 3 05:12:17 2014 (r274007)
@@ -76,6 +76,7 @@ __FBSDID("$FreeBSD$");
#include <sys/unistd.h>
#include <sys/user.h>
#include <sys/vnode.h>
+#include <sys/kdb.h>
#ifdef KTRACE
#include <sys/ktrace.h>
#endif
@@ -1910,9 +1911,7 @@ fdcopy(struct filedesc *fdp)
struct filedescent *nfde, *ofde;
int i;
- /* Certain daemons might not have file descriptors. */
- if (fdp == NULL)
- return (NULL);
+ MPASS(fdp != NULL);
newfdp = fdinit(fdp);
/* copy all passable descriptors (i.e. not kqueue) */
@@ -2034,10 +2033,8 @@ fdescfree(struct thread *td)
struct vnode *cdir, *jdir, *rdir;
int i;
- /* Certain daemons might not have file descriptors. */
fdp = td->td_proc->p_fd;
- if (fdp == NULL)
- return;
+ MPASS(fdp != NULL);
#ifdef RACCT
PROC_LOCK(td->td_proc);
More information about the svn-src-head
mailing list