diff options
| author | mjg <mjg@FreeBSD.org> | 2014-10-22 00:23:43 +0000 |
|---|---|---|
| committer | mjg <mjg@FreeBSD.org> | 2014-10-22 00:23:43 +0000 |
| commit | 2ebe66c2905234e9e3b5057b47542a9b61d7144c (patch) | |
| tree | 6a36b43cbe03e6686b72f3132ca176603d8ccb7d /sys/kern/kern_exec.c | |
| parent | 4386bf043d1b3107a4a480ad90128e1246fdb07c (diff) | |
| download | FreeBSD-src-2ebe66c2905234e9e3b5057b47542a9b61d7144c.zip FreeBSD-src-2ebe66c2905234e9e3b5057b47542a9b61d7144c.tar.gz | |
filedesc: cleanup setugidsafety a little
Rename it to fdsetugidsafety for consistency with other functions.
There is no need to take filedesc lock if not closing any files.
The loop has to verify each file and we are guaranteed fdtable has space
for at least 20 fds. As such there is no need to check fd_lastfile.
While here tidy up is_unsafe.
Diffstat (limited to 'sys/kern/kern_exec.c')
| -rw-r--r-- | sys/kern/kern_exec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/kern/kern_exec.c b/sys/kern/kern_exec.c index f2bbdaa..09212c8 100644 --- a/sys/kern/kern_exec.c +++ b/sys/kern/kern_exec.c @@ -695,7 +695,7 @@ interpret: */ PROC_UNLOCK(p); VOP_UNLOCK(imgp->vp, 0); - setugidsafety(td); + fdsetugidsafety(td); error = fdcheckstd(td); if (error != 0) goto done1; |
