From: Jouni Malinen Date: Fri, 4 Jan 2019 18:35:47 +0000 (+0200) Subject: tests: WPA2-PSK-FT AP over DS and separate hostapd process X-Git-Tag: hostap_2_8~576 X-Git-Url: http://git.ipfire.org/?p=thirdparty%2Fhostap.git;a=commitdiff_plain;h=02a4ac0feb05e2ac91b95dfb6c894d66e0056e9c tests: WPA2-PSK-FT AP over DS and separate hostapd process This is a regression test case for FT-over-DS that got broken on mac80211-based drivers when the extra key reinstallation checks were added. Signed-off-by: Jouni Malinen --- diff --git a/tests/hwsim/hostapd.py b/tests/hwsim/hostapd.py index 5c583a22a..773b0faa3 100644 --- a/tests/hwsim/hostapd.py +++ b/tests/hwsim/hostapd.py @@ -1,5 +1,5 @@ # Python class for controlling hostapd -# Copyright (c) 2013-2014, Jouni Malinen +# Copyright (c) 2013-2019, Jouni Malinen # # This software may be distributed under the terms of the BSD license. # See README for more details. @@ -22,7 +22,7 @@ def mac2tuple(mac): return struct.unpack('6B', binascii.unhexlify(mac.replace(':',''))) class HostapdGlobal: - def __init__(self, apdev=None): + def __init__(self, apdev=None, global_ctrl_override=None): try: hostname = apdev['hostname'] port = apdev['port'] @@ -33,8 +33,11 @@ class HostapdGlobal: self.hostname = hostname self.port = port if hostname is None: - self.ctrl = wpaspy.Ctrl(hapd_global) - self.mon = wpaspy.Ctrl(hapd_global) + global_ctrl = hapd_global + if global_ctrl_override: + global_ctrl = global_ctrl_override + self.ctrl = wpaspy.Ctrl(global_ctrl) + self.mon = wpaspy.Ctrl(global_ctrl) self.dbg = "" else: self.ctrl = wpaspy.Ctrl(hostname, port) @@ -377,7 +380,8 @@ class Hostapd: return vals return None -def add_ap(apdev, params, wait_enabled=True, no_enable=False, timeout=30): +def add_ap(apdev, params, wait_enabled=True, no_enable=False, timeout=30, + global_ctrl_override=None): if isinstance(apdev, dict): ifname = apdev['ifname'] try: @@ -393,7 +397,8 @@ def add_ap(apdev, params, wait_enabled=True, no_enable=False, timeout=30): logger.info("Starting AP " + ifname + " (old add_ap argument type)") hostname = None port = 8878 - hapd_global = HostapdGlobal(apdev) + hapd_global = HostapdGlobal(apdev, + global_ctrl_override=global_ctrl_override) hapd_global.remove(ifname) hapd_global.add(ifname) port = hapd_global.get_ctrl_iface_port(ifname) diff --git a/tests/hwsim/test_ap_ft.py b/tests/hwsim/test_ap_ft.py index c6a187124..6ab4f8ca2 100644 --- a/tests/hwsim/test_ap_ft.py +++ b/tests/hwsim/test_ap_ft.py @@ -1,5 +1,5 @@ # Fast BSS Transition tests -# Copyright (c) 2013-2017, Jouni Malinen +# Copyright (c) 2013-2019, Jouni Malinen # # This software may be distributed under the terms of the BSD license. # See README for more details. @@ -10,9 +10,12 @@ import os import time import logging logger = logging.getLogger() +import signal import struct +import subprocess import hwsim_utils +from hwsim import HWSimRadio import hostapd from tshark import run_tshark from utils import HwsimSkip, alloc_fail, fail_test, wait_fail_trigger, skip_with_fips, parse_ie @@ -468,6 +471,107 @@ def test_ap_ft_over_ds(dev, apdev): check_mib(dev[0], [ ("dot11RSNAAuthenticationSuiteRequested", "00-0f-ac-4"), ("dot11RSNAAuthenticationSuiteSelected", "00-0f-ac-4") ]) +def cleanup_ap_ft_separate_hostapd(): + subprocess.call(["brctl", "delif", "br0ft", "veth0"], + stderr=open('/dev/null', 'w')) + subprocess.call(["brctl", "delif", "br1ft", "veth1"], + stderr=open('/dev/null', 'w')) + subprocess.call(["ip", "link", "del", "veth0"], + stderr=open('/dev/null', 'w')) + subprocess.call(["ip", "link", "del", "veth1"], + stderr=open('/dev/null', 'w')) + for ifname in [ 'br0ft', 'br1ft', 'br-ft' ]: + subprocess.call(['ip', 'link', 'set', 'dev', ifname, 'down'], + stderr=open('/dev/null', 'w')) + subprocess.call(['brctl', 'delbr', ifname], + stderr=open('/dev/null', 'w')) + +def test_ap_ft_separate_hostapd(dev, apdev, params): + """WPA2-PSK-FT AP and separate hostapd process""" + try: + run_ap_ft_separate_hostapd(dev, apdev, params, False) + finally: + cleanup_ap_ft_separate_hostapd() + +def test_ap_ft_over_ds_separate_hostapd(dev, apdev, params): + """WPA2-PSK-FT AP over DS and separate hostapd process""" + try: + run_ap_ft_separate_hostapd(dev, apdev, params, True) + finally: + cleanup_ap_ft_separate_hostapd() + +def run_ap_ft_separate_hostapd(dev, apdev, params, over_ds): + ssid = "test-ft" + passphrase="12345678" + logdir = params['logdir'] + pidfile = os.path.join(logdir, 'ap_ft_over_ds_separate_hostapd.pid') + logfile = os.path.join(logdir, 'ap_ft_over_ds_separate_hostapd.hapd') + global_ctrl = '/var/run/hostapd-ft' + br_ifname = 'br-ft' + + try: + subprocess.check_call(['brctl', 'addbr', br_ifname]) + subprocess.check_call(['brctl', 'setfd', br_ifname, '0']) + subprocess.check_call(['ip', 'link', 'set', 'dev', br_ifname, 'up']) + + subprocess.check_call([ "ip", "link", "add", "veth0", "type", "veth", + "peer", "name", "veth0br" ]) + subprocess.check_call([ "ip", "link", "add", "veth1", "type", "veth", + "peer", "name", "veth1br" ]) + subprocess.check_call(['ip', 'link', 'set', 'dev', 'veth0br', 'up']) + subprocess.check_call(['ip', 'link', 'set', 'dev', 'veth1br', 'up']) + subprocess.check_call(['brctl', 'addif', br_ifname, 'veth0br']) + subprocess.check_call(['brctl', 'addif', br_ifname, 'veth1br']) + + subprocess.check_call(['brctl', 'addbr', 'br0ft']) + subprocess.check_call(['brctl', 'setfd', 'br0ft', '0']) + subprocess.check_call(['ip', 'link', 'set', 'dev', 'br0ft', 'up']) + subprocess.check_call(['ip', 'link', 'set', 'dev', 'veth0', 'up']) + subprocess.check_call(['brctl', 'addif', 'br0ft', 'veth0']) + subprocess.check_call(['brctl', 'addbr', 'br1ft']) + subprocess.check_call(['brctl', 'setfd', 'br1ft', '0']) + subprocess.check_call(['ip', 'link', 'set', 'dev', 'br1ft', 'up']) + subprocess.check_call(['ip', 'link', 'set', 'dev', 'veth1', 'up']) + subprocess.check_call(['brctl', 'addif', 'br1ft', 'veth1']) + except subprocess.CalledProcessError: + raise HwsimSkip("Bridge or veth not supported (kernel CONFIG_VETH)") + + with HWSimRadio() as (radio, iface): + prg = os.path.join(logdir, 'alt-hostapd/hostapd/hostapd') + if not os.path.exists(prg): + prg = '../../hostapd/hostapd' + cmd = [ prg, '-B', '-ddKt', + '-P', pidfile, '-f', logfile, '-g', global_ctrl ] + subprocess.check_call(cmd) + + hglobal = hostapd.HostapdGlobal(global_ctrl_override=global_ctrl) + apdev_ft = { 'ifname': iface } + apdev2 = [ apdev_ft, apdev[1] ] + + params = ft_params1(ssid=ssid, passphrase=passphrase) + params["r0kh"] = "ff:ff:ff:ff:ff:ff * 00112233445566778899aabbccddeeff00112233445566778899aabbccddeeff" + params["r1kh"] = "00:00:00:00:00:00 00:00:00:00:00:00 00112233445566778899aabbccddeeff00112233445566778899aabbccddeeff" + params['bridge'] = 'br0ft' + hapd0 = hostapd.add_ap(apdev2[0], params, + global_ctrl_override=global_ctrl) + apdev2[0]['bssid'] = hapd0.own_addr() + params = ft_params2(ssid=ssid, passphrase=passphrase) + params["r0kh"] = "ff:ff:ff:ff:ff:ff * 00112233445566778899aabbccddeeff00112233445566778899aabbccddeeff" + params["r1kh"] = "00:00:00:00:00:00 00:00:00:00:00:00 00112233445566778899aabbccddeeff00112233445566778899aabbccddeeff" + params['bridge'] = 'br1ft' + hapd1 = hostapd.add_ap(apdev2[1], params) + + run_roams(dev[0], apdev2, hapd0, hapd1, ssid, passphrase, + over_ds=over_ds, test_connectivity=False) + + hglobal.terminate() + + if os.path.exists(pidfile): + with open(pidfile, 'r') as f: + pid = int(f.read()) + f.close() + os.kill(pid, signal.SIGTERM) + def test_ap_ft_over_ds_ocv(dev, apdev): """WPA2-PSK-FT AP over DS""" ssid = "test-ft"