freebsd-skq/etc/mtree/BSD.tests.dist
jhb 6d687d5919 Properly do a deep copy of the ioctls capability array for fget_cap().
fget_cap() tries to do a cheaper snapshot of a file descriptor without
holding the file descriptor lock.  This snapshot does not do a deep
copy of the ioctls capability array, but instead uses a different
return value to inform the caller to retry the copy with the lock
held.  However, filecaps_copy() was returning 1 to indicate that a
retry was required, and fget_cap() was checking for 0 (actually
'!filecaps_copy()').  As a result, fget_cap() did not do a deep copy
of the ioctls array and just reused the original pointer.  This cause
multiple file descriptor entries to think they owned the same pointer
and eventually resulted in duplicate frees.

The only code path that I'm aware of that triggers this is to create a
listen socket that has a restricted list of ioctls and then call
accept() which calls fget_cap() with a valid filecaps structure from
getsock_cap().

To fix, change the return value of filecaps_copy() to return true if
it succeeds in copying the caps and false if it fails because the lock
is required.  I find this more intuitive than fixing the caller in
this case.  While here, change the return type from 'int' to 'bool'.

Finally, make filecaps_copy() more robust in the failure case by not
copying any of the source filecaps structure over.  This avoids the
possibility of leaking a pointer into a structure if a similar future
caller doesn't properly handle the return value from filecaps_copy()
at the expense of one more branch.

I also added a test case that panics before this change and now passes.

Reviewed by:	kib
Discussed with:	mjg (not a fan of the extra branch)
MFC after:	1 week
Differential Revision:	https://reviews.freebsd.org/D15047
2018-04-17 18:07:40 +00:00

1039 lines
20 KiB
Plaintext

# $FreeBSD$
#
# Please see the file src/etc/mtree/README before making changes to this file.
#
/set type=dir uname=root gname=wheel mode=0755
.
bin
cat
..
chflags
..
chmod
..
date
..
dd
..
echo
..
expr
..
ln
..
ls
..
mkdir
..
mv
..
pax
..
pkill
..
pwait
..
rmdir
..
sh
builtins
..
errors
..
execution
..
expansion
..
invocation
..
parameters
..
parser
..
set-e
..
..
sleep
..
test
..
..
cddl
lib
..
sbin
..
usr.bin
ctfconvert
..
..
usr.sbin
dtrace
common
aggs
..
arithmetic
..
arrays
..
assocs
..
begin
..
bitfields
..
buffering
..
builtinvar
..
cg
..
clauses
..
cpc
..
decls
..
drops
..
dtraceUtil
..
end
..
env
..
enum
..
error
..
exit
..
fbtprovider
..
funcs
..
grammar
..
include
..
inline
..
io
..
ip
..
java_api
..
json
..
lexer
..
llquantize
..
mdb
..
mib
..
misc
..
multiaggs
..
offsetof
..
operators
..
pid
..
plockstat
..
pointers
..
pragma
..
predicates
..
preprocessor
..
print
..
printa
..
printf
..
privs
..
probes
..
proc
..
profile-n
..
providers
..
raise
..
rates
..
safety
..
scalars
..
sched
..
scripting
..
sdt
..
sizeof
..
speculation
..
stability
..
stack
..
stackdepth
..
stop
..
strlen
..
strtoll
..
struct
..
sugar
..
syscall
..
sysevent
..
tick-n
..
trace
..
tracemem
..
translators
..
typedef
..
types
..
uctf
..
union
..
usdt
..
ustack
..
vars
..
version
..
..
..
zfsd
..
..
..
etc
rc.d
..
..
games
..
gnu
lib
..
usr.bin
diff
..
..
..
lib
atf
libatf-c
detail
..
..
libatf-c++
detail
..
..
test-programs
..
..
libarchive
..
libc
c063
..
db
..
gen
execve
..
posix_spawn
..
..
hash
data
..
..
iconv
..
inet
..
locale
..
net
getaddrinfo
data
..
..
..
nss
..
regex
data
..
..
resolv
..
rpc
..
ssp
..
setjmp
..
stdio
..
stdlib
..
string
..
sys
..
time
..
tls
dso
..
..
termios
..
ttyio
..
..
libcam
..
libcasper
services
cap_dns
..
cap_grp
..
cap_pwd
..
cap_sysctl
..
..
..
libcrypt
..
libdevdctl
..
libkvm
..
libmp
..
libnv
..
libproc
..
libregex
data
..
..
librt
..
libsbuf
..
libthr
dlopen
..
..
libutil
..
libxo
..
msun
..
..
libexec
atf
atf-check
..
atf-sh
..
..
rtld-elf
..
tftpd
..
..
sbin
dhclient
..
devd
..
growfs
..
ifconfig
..
mdconfig
..
pfctl
files
..
..
..
secure
lib
..
libexec
..
usr.bin
..
usr.sbin
..
..
share
examples
tests
atf
..
plain
..
tap
..
..
..
zoneinfo
..
..
sys
acl
..
aio
..
capsicum
..
cddl
zfs
bin
..
include
..
tests
acl
cifs
..
nontrivial
..
trivial
..
..
atime
..
bootfs
..
cache
..
cachefile
..
clean_mirror
..
cli_root
zfs_upgrade
..
zfs_promote
..
zfs_clone
..
zfs_property
..
zfs_destroy
..
zpool_create
..
zpool_history
..
zpool_expand
..
zpool_remove
..
zfs_mount
..
zfs_unshare
..
zdb
..
zpool_online
..
zpool_get
..
zpool_export
..
zfs_copies
..
zfs_get
..
zfs
..
zpool_clear
..
zpool_import
blockfiles
..
..
zpool
..
zpool_offline
..
zpool_replace
..
zfs_rollback
..
zpool_set
..
zfs_send
..
zfs_set
..
zpool_detach
..
zfs_diff
..
zpool_scrub
..
zfs_inherit
..
zfs_snapshot
..
zfs_share
..
zpool_destroy
..
zpool_status
..
zfs_unmount
..
zfs_receive
..
zfs_create
..
zpool_upgrade
blockfiles
..
..
zpool_add
..
zfs_rename
..
zpool_attach
..
zfs_reservation
..
..
cli_user
misc
..
zfs_list
..
zpool_iostat
..
zpool_list
..
..
compression
..
ctime
..
delegate
..
devices
..
exec
..
grow_pool
..
grow_replicas
..
history
..
hotplug
..
hotspare
..
inheritance
..
interop
..
inuse
..
iscsi
..
large_files
..
largest_pool
..
link_count
..
migration
..
mmap
..
mount
..
mv_files
..
nestedfs
..
no_space
..
online_offline
..
pool_names
..
poolversion
..
quota
..
redundancy
..
refquota
..
refreserv
..
rename_dirs
..
replacement
..
reservation
..
rootpool
..
rsend
..
scrub_mirror
..
slog
..
snapshot
..
snapused
..
sparse
..
threadsappend
..
truncate
..
txg_integrity
..
userquota
..
utils_test
..
write_dirs
..
xattr
..
zfsd
..
zil
..
zinject
..
zones
..
zvol
zvol_ENOSPC
..
zvol_cli
..
zvol_misc
..
zvol_swap
..
..
zvol_thrash
..
..
..
..
dtrace
..
fifo
..
file
..
fs
tmpfs
..
..
geom
class
concat
..
eli
..
gate
..
gpt
..
mirror
..
nop
..
part
..
raid3
..
shsec
..
stripe
..
uzip
etalon
..
..
..
..
kern
acct
..
execve
..
pipe
..
..
kqueue
libkqueue
..
..
mac
bsdextended
..
portacl
..
..
mqueue
..
net
..
netinet
..
netipsec
tunnel
..
..
netpfil
pf
ioctl
..
..
..
opencrypto
..
pjdfstest
chflags
..
chmod
..
chown
..
ftruncate
..
granular
..
link
..
mkdir
..
mkfifo
..
mknod
..
open
..
rename
..
rmdir
..
symlink
..
truncate
..
unlink
..
utimensat
..
..
posixshm
..
sys
..
vfs
..
vm
..
..
usr.bin
apply
..
awk
..
basename
..
bmake
archives
fmt_44bsd
..
fmt_44bsd_mod
..
fmt_oldbsd
..
..
basic
t0
..
t1
..
t2
..
t3
..
..
execution
ellipsis
..
empty
..
joberr
..
plus
..
..
shell
builtin
..
meta
..
path
..
path_select
..
replace
..
select
..
..
suffixes
basic
..
src_wild1
..
src_wild2
..
..
syntax
directive-t0
..
enl
..
funny-targets
..
semi
..
..
sysmk
t0
2
1
..
..
mk
..
..
t1
2
1
..
..
mk
..
..
t2
2
1
..
..
mk
..
..
..
variables
modifier_M
..
modifier_t
..
opt_V
..
t0
..
..
..
bsdcat
..
calendar
..
cmp
..
compress
..
cpio
..
col
..
comm
..
csplit
..
cut
..
dc
..
diff
..
dirname
..
du
..
file2c
..
find
..
fold
..
getconf
..
grep
..
gzip
..
head
..
hexdump
..
ident
..
indent
..
join
..
jot
..
lastcomm
..
limits
..
m4
..
mkimg
..
ncal
..
opensm
..
pr
..
printf
..
procstat
..
rs
..
sdiff
..
sed
regress.multitest.out
..
..
seq
..
soelim
..
stat
..
tail
..
tar
..
timeout
..
tr
..
truncate
..
units
..
uudecode
..
uuencode
..
uniq
..
vmstat
..
xargs
..
xinstall
..
xo
..
yacc
yacc
..
..
..
usr.sbin
chown
..
etcupdate
..
extattr
..
fstyp
..
makefs
..
newsyslog
..
nmtree
..
pw
..
rpcbind
..
sa
..
..
..
# vim: set expandtab ts=4 sw=4: