diff options
author | ngie <ngie@FreeBSD.org> | 2016-04-22 18:05:34 +0000 |
---|---|---|
committer | ngie <ngie@FreeBSD.org> | 2016-04-22 18:05:34 +0000 |
commit | a9f56210b8b8a8dc6b19c46445a79600f523d9dd (patch) | |
tree | bb4bdeb74de817bae84cb2140465d0699d8249d8 /lib/libkvm/kvm_minidump_mips.c | |
parent | da15c11c3103598c772b1bbb4daf72d65af13e0c (diff) | |
download | FreeBSD-src-a9f56210b8b8a8dc6b19c46445a79600f523d9dd.zip FreeBSD-src-a9f56210b8b8a8dc6b19c46445a79600f523d9dd.tar.gz |
Fix up pointer issues with lib/libkvm
In particular,
- avoid dereferencing NULL pointers
- test pointers against NULL, not 0
- test for errout == NULL in the top-level functions (kvm_open, kvm_openfiles,
kvm_open2, etc)
- Replace a realloc and free on failure with reallocf
Found with: devel/cocchinelle
Differential Revision: https://reviews.freebsd.org/D5954
MFC after: 1 week
Reviewed by: jhb
Sponsored by: EMC / Isilon Storage Division
Diffstat (limited to 'lib/libkvm/kvm_minidump_mips.c')
-rw-r--r-- | lib/libkvm/kvm_minidump_mips.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/libkvm/kvm_minidump_mips.c b/lib/libkvm/kvm_minidump_mips.c index 997b70d..97b0824 100644 --- a/lib/libkvm/kvm_minidump_mips.c +++ b/lib/libkvm/kvm_minidump_mips.c @@ -89,7 +89,7 @@ _mips_minidump_initvtop(kvm_t *kd) off_t off; vmst = _kvm_malloc(kd, sizeof(*vmst)); - if (vmst == 0) { + if (vmst == NULL) { _kvm_err(kd, kd->program, "cannot allocate vm"); return (-1); } |