From 4418d041840c295df2ad6ac97459a3e460a8f3d4 Mon Sep 17 00:00:00 2001 From: Guy Harris Date: Fri, 15 Apr 2016 16:36:59 -0700 Subject: Don't check for NULL in cleanup routines, and don't pass NULL to them. pcap_close() won't pass a null pointer - it'll crash long before that when handed a null pointer, as the platforms on which we run map out page 0. Remove the null pointer checks, and, in cases where the cleanup routines are called internally, make sure we don't pass them a null pointer. --- pcap-snf.c | 3 --- 1 file changed, 3 deletions(-) (limited to 'pcap-snf.c') diff --git a/pcap-snf.c b/pcap-snf.c index 2271fd66..a453b11c 100644 --- a/pcap-snf.c +++ b/pcap-snf.c @@ -66,9 +66,6 @@ snf_platform_cleanup(pcap_t *p) { struct pcap_snf *ps = p->priv; - if (p == NULL) - return; - #ifdef SNF_HAVE_INJECT_API if (ps->snf_inj) snf_inject_close(ps->snf_inj); -- cgit v1.2.1