diff options
author | Robert Watson <rwatson@FreeBSD.org> | 2007-05-28 17:15:05 +0000 |
---|---|---|
committer | Robert Watson <rwatson@FreeBSD.org> | 2007-05-28 17:15:05 +0000 |
commit | dede2ab3b2208b78a481a277f68e2a554c4a1816 (patch) | |
tree | 2291c80e1b38be1057933a69ee8085ea9632a769 /sys/kern | |
parent | 2ba5e1edcc4e4ca9facbf13974bc9a19c987ff33 (diff) | |
download | src-dede2ab3b2208b78a481a277f68e2a554c4a1816.tar.gz src-dede2ab3b2208b78a481a277f68e2a554c4a1816.zip |
In kern_kevent(), unconditionally fdrop() fp once fget() has succeeded,
as we never have an opportunity to set it to NULL.
Found with: Coverity Prevent(tm)
CID: 2161
Notes
Notes:
svn path=/head/; revision=170066
Diffstat (limited to 'sys/kern')
-rw-r--r-- | sys/kern/kern_event.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sys/kern/kern_event.c b/sys/kern/kern_event.c index 6272f92abe7c..aa446b6079f7 100644 --- a/sys/kern/kern_event.c +++ b/sys/kern/kern_event.c @@ -696,8 +696,7 @@ kern_kevent(struct thread *td, int fd, int nchanges, int nevents, done: kqueue_release(kq, 0); done_norel: - if (fp != NULL) - fdrop(fp, td); + fdrop(fp, td); return (error); } |