[src/trunk]: src/sys/net fix atf failed.

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

details:   https://anonhg.NetBSD.org/src/rev/50e12ab9841d
branches:  trunk
changeset: 352485:50e12ab9841d
user:      knakahara <knakahara%NetBSD.org@localhost>
date:      Tue Apr 04 04:34:43 2017 +0000

description:
fix atf failed.

diffstat:

 sys/net/if_l2tp.c |  19 ++++++++++++-------
 1 files changed, 12 insertions(+), 7 deletions(-)

diffs (61 lines):

diff -r f2a8cdf76886 -r 50e12ab9841d sys/net/if_l2tp.c
--- a/sys/net/if_l2tp.c Mon Apr 03 17:40:07 2017 +0000
+++ b/sys/net/if_l2tp.c Tue Apr 04 04:34:43 2017 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: if_l2tp.c,v 1.4 2017/04/03 10:17:17 knakahara Exp $    */
+/*     $NetBSD: if_l2tp.c,v 1.5 2017/04/04 04:34:43 knakahara Exp $    */
 
 /*
  * Copyright (c) 2017 Internet Initiative Japan Inc.
@@ -31,7 +31,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_l2tp.c,v 1.4 2017/04/03 10:17:17 knakahara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_l2tp.c,v 1.5 2017/04/04 04:34:43 knakahara Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_inet.h"
@@ -298,6 +298,7 @@
 static int
 l2tp_clone_destroy(struct ifnet *ifp)
 {
+       struct l2tp_variant *var;
        struct l2tp_softc *sc = container_of(ifp, struct l2tp_softc,
            l2tp_ec.ec_if);
 
@@ -307,6 +308,7 @@
         * To avoid for l2tp_transmit() to access sc->l2tp_var after free it.
         */
        mutex_enter(&sc->l2tp_lock);
+       var = sc->l2tp_var;
        l2tp_variant_update(sc, NULL);
        mutex_exit(&sc->l2tp_lock);
 
@@ -321,7 +323,7 @@
        percpu_foreach(sc->l2tp_ro_percpu, l2tp_ro_fini_pc, NULL);
        percpu_free(sc->l2tp_ro_percpu, sizeof(struct l2tp_ro));
 
-       kmem_free(sc->l2tp_var, sizeof(struct l2tp_variant));
+       kmem_free(var, sizeof(struct l2tp_variant));
        mutex_destroy(&sc->l2tp_lock);
        kmem_free(sc, sizeof(struct l2tp_softc));
 
@@ -1166,10 +1168,13 @@
         * In the manual of atomic_swap_ptr(3), there is no mention if 2nd
         * argument is rewrite or not. So, use sc->l2tp_var instead of nvar.
         */
-       if (sc->l2tp_var->lv_psrc != NULL && sc->l2tp_var->lv_pdst != NULL)
-               ifp->if_flags |= IFF_RUNNING;
-       else
-               ifp->if_flags &= ~IFF_RUNNING;
+       if (sc->l2tp_var != NULL) {
+               if (sc->l2tp_var->lv_psrc != NULL
+                   && sc->l2tp_var->lv_pdst != NULL)
+                       ifp->if_flags |= IFF_RUNNING;
+               else
+                       ifp->if_flags &= ~IFF_RUNNING;
+       }
 }
 
 static int