[src/trunk]: src/sys/netipsec CID 1220169: Reverse NULL

[christos <christos%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/378eb11709b5
branches:  trunk
changeset: 329717:378eb11709b5
user:      christos <christos%NetBSD.org@localhost>
date:      Thu Jun 05 17:18:19 2014 +0000

description:
CID 1220169: Reverse NULL

diffstat:

 sys/netipsec/key.c |  7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diffs (32 lines):

diff -r a436370a414d -r 378eb11709b5 sys/netipsec/key.c
--- a/sys/netipsec/key.c        Thu Jun 05 17:17:26 2014 +0000
+++ b/sys/netipsec/key.c        Thu Jun 05 17:18:19 2014 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: key.c,v 1.89 2014/06/05 17:11:54 christos Exp $        */
+/*     $NetBSD: key.c,v 1.90 2014/06/05 17:18:19 christos Exp $        */
 /*     $FreeBSD: src/sys/netipsec/key.c,v 1.3.2.3 2004/02/14 22:23:23 bms Exp $        */
 /*     $KAME: key.c,v 1.191 2001/06/27 10:46:49 sakane Exp $   */
 
@@ -32,7 +32,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: key.c,v 1.89 2014/06/05 17:11:54 christos Exp $");
+__KERNEL_RCSID(0, "$NetBSD: key.c,v 1.90 2014/06/05 17:18:19 christos Exp $");
 
 /*
  * This code is referd to RFC 2367
@@ -1220,11 +1220,12 @@
        case PF_INET:
            {
                struct inpcb *pcb = sotoinpcb(so);
-               struct inpcbpolicy *sp = pcb->inp_sp;
 
                /* Does it have a PCB ? */
                if (pcb == NULL)
                        return;
+
+               struct inpcbpolicy *sp = pcb->inp_sp;
                key_freesp_so(&sp->sp_in);
                key_freesp_so(&sp->sp_out);
            }