Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/netbsd-9]: src Pull up following revision(s) (requested by ozaki-r in ti...



details:   https://anonhg.NetBSD.org/src/rev/cddc3f989f1c
branches:  netbsd-9
changeset: 962850:cddc3f989f1c
user:      martin <martin%NetBSD.org@localhost>
date:      Mon Aug 19 16:10:35 2019 +0000

description:
Pull up following revision(s) (requested by ozaki-r in ticket #98):

        sys/net/if.c: revision 1.458
        tests/net/if/t_ifconfig.sh: revision 1.21

Restore if_ioctl on error of ifc_destroy

Otherwise subsequence ioctls won't work.

Patch from Harold Gutch on PR kern/54434 (tweaked a bit by me)
tests: check if ifconfig (ioctl) works after a failure of ifconfig destroy

This is a test for PR kern/54434.

diffstat:

 sys/net/if.c               |  18 +++++++++++++++---
 tests/net/if/t_ifconfig.sh |   7 ++++++-
 2 files changed, 21 insertions(+), 4 deletions(-)

diffs (74 lines):

diff -r 16c47194714e -r cddc3f989f1c sys/net/if.c
--- a/sys/net/if.c      Mon Aug 19 16:08:19 2019 +0000
+++ b/sys/net/if.c      Mon Aug 19 16:10:35 2019 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: if.c,v 1.457 2019/07/25 07:45:57 knakahara Exp $       */
+/*     $NetBSD: if.c,v 1.457.2.1 2019/08/19 16:10:35 martin Exp $      */
 
 /*-
  * Copyright (c) 1999, 2000, 2001, 2008 The NetBSD Foundation, Inc.
@@ -90,7 +90,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if.c,v 1.457 2019/07/25 07:45:57 knakahara Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if.c,v 1.457.2.1 2019/08/19 16:10:35 martin Exp $");
 
 #if defined(_KERNEL_OPT)
 #include "opt_inet.h"
@@ -1611,6 +1611,8 @@
        struct if_clone *ifc;
        struct ifnet *ifp;
        struct psref psref;
+       int error;
+       int (*if_ioctl)(struct ifnet *, u_long, void *);
 
        KASSERT(mutex_owned(&if_clone_mtx));
 
@@ -1627,6 +1629,7 @@
 
        /* We have to disable ioctls here */
        IFNET_LOCK(ifp);
+       if_ioctl = ifp->if_ioctl;
        ifp->if_ioctl = if_nullioctl;
        IFNET_UNLOCK(ifp);
 
@@ -1636,7 +1639,16 @@
         */
        if_put(ifp, &psref);
 
-       return (*ifc->ifc_destroy)(ifp);
+       error = (*ifc->ifc_destroy)(ifp);
+
+       if (error != 0) {
+               /* We have to restore if_ioctl on error */
+               IFNET_LOCK(ifp);
+               ifp->if_ioctl = if_ioctl;
+               IFNET_UNLOCK(ifp);
+       }
+
+       return error;
 }
 
 static bool
diff -r 16c47194714e -r cddc3f989f1c tests/net/if/t_ifconfig.sh
--- a/tests/net/if/t_ifconfig.sh        Mon Aug 19 16:08:19 2019 +0000
+++ b/tests/net/if/t_ifconfig.sh        Mon Aug 19 16:10:35 2019 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: t_ifconfig.sh,v 1.20 2019/07/04 02:46:40 ozaki-r Exp $
+# $NetBSD: t_ifconfig.sh,v 1.20.2.1 2019/08/19 16:10:35 martin Exp $
 #
 # Copyright (c) 2015 The NetBSD Foundation, Inc.
 # All rights reserved.
@@ -69,6 +69,11 @@
        atf_check -s exit:0 rump.ifconfig shmif0 up
        atf_check -s exit:0 rump.ifconfig shmif0 destroy
 
+       # Check if ifconfig (ioctl) works after a failure of ifconfig destroy
+       atf_check -s exit:0 -o ignore rump.ifconfig lo0
+       atf_check -s not-exit:0 -e ignore rump.ifconfig lo0 destroy
+       atf_check -s exit:0 -o ignore rump.ifconfig lo0
+
        unset RUMP_SERVER
 }
 



Home | Main Index | Thread Index | Old Index