diff options
| author | csjp <csjp@FreeBSD.org> | 2006-01-15 01:55:45 +0000 |
|---|---|---|
| committer | csjp <csjp@FreeBSD.org> | 2006-01-15 01:55:45 +0000 |
| commit | ef078951d6c5cdaeb8063a72dc0d02834bd46cc0 (patch) | |
| tree | 79902c724680df272cbb2333879c135635787714 /sys/kern/vfs_aio.c | |
| parent | 68b7569e603e2300c9423cb1ebaa4038d2d611bb (diff) | |
| download | FreeBSD-src-ef078951d6c5cdaeb8063a72dc0d02834bd46cc0.zip FreeBSD-src-ef078951d6c5cdaeb8063a72dc0d02834bd46cc0.tar.gz | |
Initialize ki to p->p_aioinfo after we know it's going to be referencing
a valid kaioinfo structure. This avoids a potential NULL pointer dereference.
Found with: Coverity Prevent(tm)
MFC after: 2 weeks
Diffstat (limited to 'sys/kern/vfs_aio.c')
| -rw-r--r-- | sys/kern/vfs_aio.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/kern/vfs_aio.c b/sys/kern/vfs_aio.c index 5466959..22a3bb7 100644 --- a/sys/kern/vfs_aio.c +++ b/sys/kern/vfs_aio.c @@ -2318,9 +2318,9 @@ aio_waitcomplete(struct thread *td, struct aio_waitcomplete_args *uap) timo = tvtohz(&atv); } - ki = p->p_aioinfo; - if (ki == NULL) + if (p->p_aioinfo == NULL) aio_init_aioinfo(p); + ki = p->p_aioinfo; for (;;) { PROC_LOCK(p); |
