]> git.ipfire.org Git - thirdparty/hostap.git/blobdiff - tests/hwsim/test_dbus.py
tests: Avoid confusing "DETACH failed" exception prints in D-Bus tests
[thirdparty/hostap.git] / tests / hwsim / test_dbus.py
index 1356fd00a8a953d1cc7a58044189f859f9da5381..33a6b76098ea13347ddd7fad2ae468bc6b85d22a 100644 (file)
@@ -9,6 +9,7 @@ import logging
 logger = logging.getLogger()
 import subprocess
 import time
+import struct
 import sys
 
 try:
@@ -54,7 +55,7 @@ def prepare_dbus(dev):
         wpas = dbus.Interface(wpas_obj, WPAS_DBUS_SERVICE)
         path = wpas.GetInterface(dev.ifname)
         if_obj = bus.get_object(WPAS_DBUS_SERVICE, path)
-        return (bus,wpas_obj,path,if_obj)
+        return (bus, wpas_obj, path, if_obj)
     except Exception as e:
         raise HwsimSkip("Could not connect to D-Bus: %s" % e)
 
@@ -102,15 +103,15 @@ class alloc_fail_dbus(object):
 
 def start_ap(ap, ssid="test-wps",
              ap_uuid="27ea801a-9e5c-4e73-bd82-f89cbcd10d7e"):
-    params = { "ssid": ssid, "eap_server": "1", "wps_state": "2",
-               "wpa_passphrase": "12345678", "wpa": "2",
-               "wpa_key_mgmt": "WPA-PSK", "rsn_pairwise": "CCMP",
-               "ap_pin": "12345670", "uuid": ap_uuid}
+    params = {"ssid": ssid, "eap_server": "1", "wps_state": "2",
+              "wpa_passphrase": "12345678", "wpa": "2",
+              "wpa_key_mgmt": "WPA-PSK", "rsn_pairwise": "CCMP",
+              "ap_pin": "12345670", "uuid": ap_uuid}
     return hostapd.add_ap(ap, params)
 
 def test_dbus_getall(dev, apdev):
     """D-Bus GetAll"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     props = wpas_obj.GetAll(WPAS_DBUS_SERVICE,
                             dbus_interface=dbus.PROPERTIES_IFACE)
@@ -134,7 +135,7 @@ def test_dbus_getall(dev, apdev):
     if len(res) != 0:
         raise Exception("Unexpected Networks entry: " + str(res))
 
-    hapd = hostapd.add_ap(apdev[0], { "ssid": "open" })
+    hapd = hostapd.add_ap(apdev[0], {"ssid": "open"})
     bssid = apdev[0]['bssid']
     dev[0].scan_for_bss(bssid, freq=2412)
     id = dev[0].add_network()
@@ -170,7 +171,7 @@ def test_dbus_getall(dev, apdev):
 
 def test_dbus_getall_oom(dev, apdev):
     """D-Bus GetAll wpa_config_get_all() OOM"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     id = dev[0].add_network()
     dev[0].set_network(id, "disabled", "0")
@@ -204,13 +205,13 @@ def dbus_set(dbus, wpas_obj, prop, val):
 
 def test_dbus_properties(dev, apdev):
     """D-Bus Get/Set fi.w1.wpa_supplicant1 properties"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     dbus_get(dbus, wpas_obj, "DebugLevel", expect="msgdump")
     dbus_set(dbus, wpas_obj, "DebugLevel", "debug")
     dbus_get(dbus, wpas_obj, "DebugLevel", expect="debug")
-    for (val,err) in [ (3, "Error.Failed: wrong property type"),
-                       ("foo", "Error.Failed: wrong debug level value") ]:
+    for (val, err) in [(3, "Error.Failed: wrong property type"),
+                       ("foo", "Error.Failed: wrong debug level value")]:
         try:
             dbus_set(dbus, wpas_obj, "DebugLevel", val)
             raise Exception("Invalid DebugLevel value accepted: " + str(val))
@@ -327,10 +328,10 @@ def test_dbus_properties(dev, apdev):
 
 def test_dbus_set_global_properties(dev, apdev):
     """D-Bus Get/Set fi.w1.wpa_supplicant1 interface global properties"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     dev[0].set("model_name", "")
-    props = [ ('Okc', '0', '1'), ('ModelName', '', 'blahblahblah') ]
+    props = [('Okc', '0', '1'), ('ModelName', '', 'blahblahblah')]
 
     for p in props:
         res = if_obj.Get(WPAS_DBUS_IFACE, p[0],
@@ -349,7 +350,7 @@ def test_dbus_set_global_properties(dev, apdev):
 
 def test_dbus_invalid_method(dev, apdev):
     """D-Bus invalid method"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wps = dbus.Interface(if_obj, WPAS_DBUS_IFACE_WPS)
 
     try:
@@ -383,7 +384,7 @@ def test_dbus_get_set_wps(dev, apdev):
         dev[0].set("device_type", "0-00000000-0")
 
 def _test_dbus_get_set_wps(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     if_obj.Get(WPAS_DBUS_IFACE_WPS, "ConfigMethods",
                dbus_interface=dbus.PROPERTIES_IFACE)
@@ -414,13 +415,13 @@ def _test_dbus_get_set_wps(dev, apdev):
         if val != expected_val:
             raise Exception("Unexpected Get(ProcessCredentials) result({}): {}".format(i, val))
 
-    tests = [ ("device_name", "DeviceName"),
-              ("manufacturer", "Manufacturer"),
-              ("model_name", "ModelName"),
-              ("model_number", "ModelNumber"),
-              ("serial_number", "SerialNumber") ]
+    tests = [("device_name", "DeviceName"),
+             ("manufacturer", "Manufacturer"),
+             ("model_name", "ModelName"),
+             ("model_number", "ModelNumber"),
+             ("serial_number", "SerialNumber")]
 
-    for f1,f2 in tests:
+    for f1, f2 in tests:
         val2 = "test-value-test"
         dev[0].set(f1, val2)
         val = if_obj.Get(WPAS_DBUS_IFACE_WPS, f2,
@@ -517,26 +518,26 @@ def _test_dbus_get_set_wps(dev, apdev):
 
 def test_dbus_wps_invalid(dev, apdev):
     """D-Bus invaldi WPS operation"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wps = dbus.Interface(if_obj, WPAS_DBUS_IFACE_WPS)
 
-    failures = [ {'Role': 'foo', 'Type': 'pbc'},
-                 {'Role': 123, 'Type': 'pbc'},
-                 {'Type': 'pbc'},
-                 {'Role': 'enrollee'},
-                 {'Role': 'registrar'},
-                 {'Role': 'enrollee', 'Type': 123},
-                 {'Role': 'enrollee', 'Type': 'foo'},
-                 {'Role': 'enrollee', 'Type': 'pbc',
-                  'Bssid': '02:33:44:55:66:77'},
-                 {'Role': 'enrollee', 'Type': 'pin', 'Pin': 123},
-                 {'Role': 'enrollee', 'Type': 'pbc',
-                  'Bssid': dbus.ByteArray(b'12345')},
-                 {'Role': 'enrollee', 'Type': 'pbc',
-                  'P2PDeviceAddress': 12345},
-                 {'Role': 'enrollee', 'Type': 'pbc',
-                  'P2PDeviceAddress': dbus.ByteArray(b'12345')},
-                 {'Role': 'enrollee', 'Type': 'pbc', 'Foo': 'bar'} ]
+    failures = [{'Role': 'foo', 'Type': 'pbc'},
+                {'Role': 123, 'Type': 'pbc'},
+                {'Type': 'pbc'},
+                {'Role': 'enrollee'},
+                {'Role': 'registrar'},
+                {'Role': 'enrollee', 'Type': 123},
+                {'Role': 'enrollee', 'Type': 'foo'},
+                {'Role': 'enrollee', 'Type': 'pbc',
+                 'Bssid': '02:33:44:55:66:77'},
+                {'Role': 'enrollee', 'Type': 'pin', 'Pin': 123},
+                {'Role': 'enrollee', 'Type': 'pbc',
+                 'Bssid': dbus.ByteArray(b'12345')},
+                {'Role': 'enrollee', 'Type': 'pbc',
+                 'P2PDeviceAddress': 12345},
+                {'Role': 'enrollee', 'Type': 'pbc',
+                 'P2PDeviceAddress': dbus.ByteArray(b'12345')},
+                {'Role': 'enrollee', 'Type': 'pbc', 'Foo': 'bar'}]
     for args in failures:
         try:
             wps.Start(args)
@@ -547,14 +548,14 @@ def test_dbus_wps_invalid(dev, apdev):
 
 def test_dbus_wps_oom(dev, apdev):
     """D-Bus WPS operation (OOM)"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wps = dbus.Interface(if_obj, WPAS_DBUS_IFACE_WPS)
 
     with alloc_fail_dbus(dev[0], 1, "=wpas_dbus_getter_state", "Get"):
         if_obj.Get(WPAS_DBUS_IFACE, "State",
                    dbus_interface=dbus.PROPERTIES_IFACE)
 
-    hapd = hostapd.add_ap(apdev[0], { "ssid": "open" })
+    hapd = hostapd.add_ap(apdev[0], {"ssid": "open"})
     bssid = apdev[0]['bssid']
     dev[0].scan_for_bss(bssid, freq=2412)
 
@@ -613,7 +614,7 @@ def test_dbus_wps_pbc(dev, apdev):
         dev[0].request("SET wps_cred_processing 0")
 
 def _test_dbus_wps_pbc(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wps = dbus.Interface(if_obj, WPAS_DBUS_IFACE_WPS)
 
     hapd = start_ap(apdev[0])
@@ -684,7 +685,7 @@ def _test_dbus_wps_pbc(dev, apdev):
 
 def test_dbus_wps_pbc_overlap(dev, apdev):
     """D-Bus WPS/PBC operation and signal for PBC overlap"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wps = dbus.Interface(if_obj, WPAS_DBUS_IFACE_WPS)
 
     hapd = start_ap(apdev[0])
@@ -741,7 +742,7 @@ def test_dbus_wps_pin(dev, apdev):
         dev[0].request("SET wps_cred_processing 0")
 
 def _test_dbus_wps_pin(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wps = dbus.Interface(if_obj, WPAS_DBUS_IFACE_WPS)
 
     hapd = start_ap(apdev[0])
@@ -780,7 +781,7 @@ def _test_dbus_wps_pin(dev, apdev):
 
         def start_pin(self, *args):
             logger.debug("start_pin")
-            bssid_ay = dbus.ByteArray(binascii.unhexlify(bssid.replace(':','').encode()))
+            bssid_ay = dbus.ByteArray(binascii.unhexlify(bssid.replace(':', '').encode()))
             wps.Start({'Role': 'enrollee', 'Type': 'pin', 'Pin': '12345670',
                        'Bssid': bssid_ay})
             return False
@@ -802,7 +803,7 @@ def test_dbus_wps_pin2(dev, apdev):
         dev[0].request("SET wps_cred_processing 0")
 
 def _test_dbus_wps_pin2(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wps = dbus.Interface(if_obj, WPAS_DBUS_IFACE_WPS)
 
     hapd = start_ap(apdev[0])
@@ -840,7 +841,7 @@ def _test_dbus_wps_pin2(dev, apdev):
 
         def start_pin(self, *args):
             logger.debug("start_pin")
-            bssid_ay = dbus.ByteArray(binascii.unhexlify(bssid.replace(':','').encode()))
+            bssid_ay = dbus.ByteArray(binascii.unhexlify(bssid.replace(':', '').encode()))
             res = wps.Start({'Role': 'enrollee', 'Type': 'pin',
                              'Bssid': bssid_ay})
             pin = res['Pin']
@@ -865,7 +866,7 @@ def test_dbus_wps_pin_m2d(dev, apdev):
         dev[0].request("SET wps_cred_processing 0")
 
 def _test_dbus_wps_pin_m2d(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wps = dbus.Interface(if_obj, WPAS_DBUS_IFACE_WPS)
 
     hapd = start_ap(apdev[0])
@@ -906,7 +907,7 @@ def _test_dbus_wps_pin_m2d(dev, apdev):
 
         def start_pin(self, *args):
             logger.debug("start_pin")
-            bssid_ay = dbus.ByteArray(binascii.unhexlify(bssid.replace(':','').encode()))
+            bssid_ay = dbus.ByteArray(binascii.unhexlify(bssid.replace(':', '').encode()))
             wps.Start({'Role': 'enrollee', 'Type': 'pin', 'Pin': '12345670',
                        'Bssid': bssid_ay})
             return False
@@ -928,7 +929,7 @@ def test_dbus_wps_reg(dev, apdev):
         dev[0].request("SET wps_cred_processing 0")
 
 def _test_dbus_wps_reg(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wps = dbus.Interface(if_obj, WPAS_DBUS_IFACE_WPS)
 
     hapd = start_ap(apdev[0])
@@ -961,7 +962,7 @@ def _test_dbus_wps_reg(dev, apdev):
 
         def start_reg(self, *args):
             logger.debug("start_reg")
-            bssid_ay = dbus.ByteArray(binascii.unhexlify(bssid.replace(':','').encode()))
+            bssid_ay = dbus.ByteArray(binascii.unhexlify(bssid.replace(':', '').encode()))
             wps.Start({'Role': 'registrar', 'Type': 'pin',
                        'Pin': '12345670', 'Bssid': bssid_ay})
             return False
@@ -977,7 +978,7 @@ def _test_dbus_wps_reg(dev, apdev):
 
 def test_dbus_wps_cancel(dev, apdev):
     """D-Bus WPS Cancel operation"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wps = dbus.Interface(if_obj, WPAS_DBUS_IFACE_WPS)
 
     hapd = start_ap(apdev[0])
@@ -985,7 +986,7 @@ def test_dbus_wps_cancel(dev, apdev):
 
     wps.Cancel()
     dev[0].scan_for_bss(bssid, freq="2412")
-    bssid_ay = dbus.ByteArray(binascii.unhexlify(bssid.replace(':','').encode()))
+    bssid_ay = dbus.ByteArray(binascii.unhexlify(bssid.replace(':', '').encode()))
     wps.Start({'Role': 'enrollee', 'Type': 'pin', 'Pin': '12345670',
                'Bssid': bssid_ay})
     wps.Cancel()
@@ -993,45 +994,45 @@ def test_dbus_wps_cancel(dev, apdev):
 
 def test_dbus_scan_invalid(dev, apdev):
     """D-Bus invalid scan method"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
-    tests = [ ({}, "InvalidArgs"),
-              ({'Type': 123}, "InvalidArgs"),
-              ({'Type': 'foo'}, "InvalidArgs"),
-              ({'Type': 'active', 'Foo': 'bar'}, "InvalidArgs"),
-              ({'Type': 'active', 'SSIDs': 'foo'}, "InvalidArgs"),
-              ({'Type': 'active', 'SSIDs': ['foo']}, "InvalidArgs"),
-              ({'Type': 'active',
-                'SSIDs': [ dbus.ByteArray(b"1"), dbus.ByteArray(b"2"),
-                           dbus.ByteArray(b"3"), dbus.ByteArray(b"4"),
-                           dbus.ByteArray(b"5"), dbus.ByteArray(b"6"),
-                           dbus.ByteArray(b"7"), dbus.ByteArray(b"8"),
-                           dbus.ByteArray(b"9"), dbus.ByteArray(b"10"),
-                           dbus.ByteArray(b"11"), dbus.ByteArray(b"12"),
-                           dbus.ByteArray(b"13"), dbus.ByteArray(b"14"),
-                           dbus.ByteArray(b"15"), dbus.ByteArray(b"16"),
-                           dbus.ByteArray(b"17") ]},
-               "InvalidArgs"),
-              ({'Type': 'active',
-                'SSIDs': [ dbus.ByteArray(b"1234567890abcdef1234567890abcdef1") ]},
-               "InvalidArgs"),
-              ({'Type': 'active', 'IEs': 'foo'}, "InvalidArgs"),
-              ({'Type': 'active', 'IEs': ['foo']}, "InvalidArgs"),
-              ({'Type': 'active', 'Channels': 2412 }, "InvalidArgs"),
-              ({'Type': 'active', 'Channels': [ 2412 ] }, "InvalidArgs"),
-              ({'Type': 'active',
-                'Channels': [ (dbus.Int32(2412), dbus.UInt32(20)) ] },
-               "InvalidArgs"),
-              ({'Type': 'active',
-                'Channels': [ (dbus.UInt32(2412), dbus.Int32(20)) ] },
-               "InvalidArgs"),
-              ({'Type': 'active', 'AllowRoam': "yes" }, "InvalidArgs"),
-              ({'Type': 'passive', 'IEs': [ dbus.ByteArray(b"\xdd\x00") ]},
-               "InvalidArgs"),
-              ({'Type': 'passive', 'SSIDs': [ dbus.ByteArray(b"foo") ]},
-               "InvalidArgs")]
-    for (t,err) in tests:
+    tests = [({}, "InvalidArgs"),
+             ({'Type': 123}, "InvalidArgs"),
+             ({'Type': 'foo'}, "InvalidArgs"),
+             ({'Type': 'active', 'Foo': 'bar'}, "InvalidArgs"),
+             ({'Type': 'active', 'SSIDs': 'foo'}, "InvalidArgs"),
+             ({'Type': 'active', 'SSIDs': ['foo']}, "InvalidArgs"),
+             ({'Type': 'active',
+               'SSIDs': [dbus.ByteArray(b"1"), dbus.ByteArray(b"2"),
+                         dbus.ByteArray(b"3"), dbus.ByteArray(b"4"),
+                         dbus.ByteArray(b"5"), dbus.ByteArray(b"6"),
+                         dbus.ByteArray(b"7"), dbus.ByteArray(b"8"),
+                         dbus.ByteArray(b"9"), dbus.ByteArray(b"10"),
+                         dbus.ByteArray(b"11"), dbus.ByteArray(b"12"),
+                         dbus.ByteArray(b"13"), dbus.ByteArray(b"14"),
+                         dbus.ByteArray(b"15"), dbus.ByteArray(b"16"),
+                         dbus.ByteArray(b"17")]},
+              "InvalidArgs"),
+             ({'Type': 'active',
+               'SSIDs': [dbus.ByteArray(b"1234567890abcdef1234567890abcdef1")]},
+              "InvalidArgs"),
+             ({'Type': 'active', 'IEs': 'foo'}, "InvalidArgs"),
+             ({'Type': 'active', 'IEs': ['foo']}, "InvalidArgs"),
+             ({'Type': 'active', 'Channels': 2412}, "InvalidArgs"),
+             ({'Type': 'active', 'Channels': [2412]}, "InvalidArgs"),
+             ({'Type': 'active',
+               'Channels': [(dbus.Int32(2412), dbus.UInt32(20))]},
+              "InvalidArgs"),
+             ({'Type': 'active',
+               'Channels': [(dbus.UInt32(2412), dbus.Int32(20))]},
+              "InvalidArgs"),
+             ({'Type': 'active', 'AllowRoam': "yes"}, "InvalidArgs"),
+             ({'Type': 'passive', 'IEs': [dbus.ByteArray(b"\xdd\x00")]},
+              "InvalidArgs"),
+             ({'Type': 'passive', 'SSIDs': [dbus.ByteArray(b"foo")]},
+              "InvalidArgs")]
+    for (t, err) in tests:
         try:
             iface.Scan(t)
             raise Exception("Invalid Scan() arguments accepted: " + str(t))
@@ -1041,42 +1042,42 @@ def test_dbus_scan_invalid(dev, apdev):
 
 def test_dbus_scan_oom(dev, apdev):
     """D-Bus scan method and OOM"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     with alloc_fail_dbus(dev[0], 1,
                          "wpa_scan_clone_params;wpas_dbus_handler_scan",
                          "Scan", expected="ScanError: Scan request rejected"):
-        iface.Scan({ 'Type': 'passive',
-                     'Channels': [ (dbus.UInt32(2412), dbus.UInt32(20)) ] })
+        iface.Scan({'Type': 'passive',
+                    'Channels': [(dbus.UInt32(2412), dbus.UInt32(20))]})
 
     with alloc_fail_dbus(dev[0], 1,
                          "=wpas_dbus_get_scan_channels;wpas_dbus_handler_scan",
                          "Scan"):
-        iface.Scan({ 'Type': 'passive',
-                     'Channels': [ (dbus.UInt32(2412), dbus.UInt32(20)) ] })
+        iface.Scan({'Type': 'passive',
+                    'Channels': [(dbus.UInt32(2412), dbus.UInt32(20))]})
 
     with alloc_fail_dbus(dev[0], 1,
                          "=wpas_dbus_get_scan_ies;wpas_dbus_handler_scan",
                          "Scan"):
-        iface.Scan({ 'Type': 'active',
-                     'IEs': [ dbus.ByteArray(b"\xdd\x00") ],
-                     'Channels': [ (dbus.UInt32(2412), dbus.UInt32(20)) ] })
+        iface.Scan({'Type': 'active',
+                    'IEs': [dbus.ByteArray(b"\xdd\x00")],
+                    'Channels': [(dbus.UInt32(2412), dbus.UInt32(20))]})
 
     with alloc_fail_dbus(dev[0], 1,
                          "=wpas_dbus_get_scan_ssids;wpas_dbus_handler_scan",
                          "Scan"):
-        iface.Scan({ 'Type': 'active',
-                     'SSIDs': [ dbus.ByteArray(b"open"),
-                                dbus.ByteArray() ],
-                     'Channels': [ (dbus.UInt32(2412), dbus.UInt32(20)) ] })
+        iface.Scan({'Type': 'active',
+                    'SSIDs': [dbus.ByteArray(b"open"),
+                              dbus.ByteArray()],
+                    'Channels': [(dbus.UInt32(2412), dbus.UInt32(20))]})
 
 def test_dbus_scan(dev, apdev):
     """D-Bus scan and related signals"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
-    hapd = hostapd.add_ap(apdev[0], { "ssid": "open" })
+    hapd = hostapd.add_ap(apdev[0], {"ssid": "open"})
 
     class TestDbusScan(TestDbus):
         def __init__(self, bus):
@@ -1124,10 +1125,10 @@ def test_dbus_scan(dev, apdev):
         def run_scan(self, *args):
             logger.debug("run_scan")
             iface.Scan({'Type': 'active',
-                        'SSIDs': [ dbus.ByteArray(b"open"),
-                                   dbus.ByteArray() ],
-                        'IEs': [ dbus.ByteArray(b"\xdd\x00"),
-                                 dbus.ByteArray() ],
+                        'SSIDs': [dbus.ByteArray(b"open"),
+                                  dbus.ByteArray()],
+                        'IEs': [dbus.ByteArray(b"\xdd\x00"),
+                                dbus.ByteArray()],
                         'AllowRoam': False,
                         'Channels': [(dbus.UInt32(2412), dbus.UInt32(20))]})
             return False
@@ -1154,7 +1155,7 @@ def test_dbus_scan(dev, apdev):
 
 def test_dbus_scan_busy(dev, apdev):
     """D-Bus scan trigger rejection when busy with previous scan"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     if "OK" not in dev[0].request("SCAN freq=2412-2462"):
@@ -1176,7 +1177,7 @@ def test_dbus_scan_busy(dev, apdev):
 
 def test_dbus_scan_abort(dev, apdev):
     """D-Bus scan trigger and abort"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     iface.Scan({'Type': 'active', 'AllowRoam': False})
@@ -1198,7 +1199,7 @@ def test_dbus_scan_abort(dev, apdev):
 
 def test_dbus_connect(dev, apdev):
     """D-Bus AddNetwork and connect"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     ssid = "test-wpa2-psk"
@@ -1279,10 +1280,10 @@ def test_dbus_connect(dev, apdev):
 
         def run_connect(self, *args):
             logger.debug("run_connect")
-            args = dbus.Dictionary({ 'ssid': ssid,
-                                     'key_mgmt': 'WPA-PSK',
-                                     'psk': passphrase,
-                                     'scan_freq': 2412 },
+            args = dbus.Dictionary({'ssid': ssid,
+                                    'key_mgmt': 'WPA-PSK',
+                                    'psk': passphrase,
+                                    'scan_freq': 2412},
                                    signature='sv')
             self.netw = iface.AddNetwork(args)
             iface.SelectNetwork(self.netw)
@@ -1301,7 +1302,7 @@ def test_dbus_connect(dev, apdev):
 
 def test_dbus_connect_psk_mem(dev, apdev):
     """D-Bus AddNetwork and connect with memory-only PSK"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     ssid = "test-wpa2-psk"
@@ -1337,10 +1338,10 @@ def test_dbus_connect_psk_mem(dev, apdev):
 
         def run_connect(self, *args):
             logger.debug("run_connect")
-            args = dbus.Dictionary({ 'ssid': ssid,
-                                     'key_mgmt': 'WPA-PSK',
-                                     'mem_only_psk': 1,
-                                     'scan_freq': 2412 },
+            args = dbus.Dictionary({'ssid': ssid,
+                                    'key_mgmt': 'WPA-PSK',
+                                    'mem_only_psk': 1,
+                                    'scan_freq': 2412},
                                    signature='sv')
             self.netw = iface.AddNetwork(args)
             iface.SelectNetwork(self.netw)
@@ -1355,7 +1356,7 @@ def test_dbus_connect_psk_mem(dev, apdev):
 
 def test_dbus_connect_oom(dev, apdev):
     """D-Bus AddNetwork and connect when out-of-memory"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     if "OK" not in dev[0].request("TEST_ALLOC_FAIL 0:"):
@@ -1433,10 +1434,10 @@ def test_dbus_connect_oom(dev, apdev):
 
         def run_connect(self, *args):
             logger.debug("run_connect")
-            args = dbus.Dictionary({ 'ssid': ssid,
-                                     'key_mgmt': 'WPA-PSK',
-                                     'psk': passphrase,
-                                     'scan_freq': 2412 },
+            args = dbus.Dictionary({'ssid': ssid,
+                                    'key_mgmt': 'WPA-PSK',
+                                    'psk': passphrase,
+                                    'scan_freq': 2412},
                                    signature='sv')
             try:
                 self.netw = iface.AddNetwork(args)
@@ -1497,7 +1498,7 @@ def test_dbus_connect_oom(dev, apdev):
 
 def test_dbus_while_not_connected(dev, apdev):
     """D-Bus invalid operations while not connected"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     try:
@@ -1517,7 +1518,7 @@ def test_dbus_while_not_connected(dev, apdev):
 def test_dbus_connect_eap(dev, apdev):
     """D-Bus AddNetwork and connect to EAP network"""
     check_altsubject_match_support(dev[0])
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     ssid = "ieee8021x-open"
@@ -1554,8 +1555,8 @@ def test_dbus_connect_eap(dev, apdev):
                     iface.EAPLogoff()
                     logger.info("Set dNSName constraint")
                     net_obj = bus.get_object(WPAS_DBUS_SERVICE, self.netw)
-                    args = dbus.Dictionary({ 'altsubject_match':
-                                             self.server_dnsname },
+                    args = dbus.Dictionary({'altsubject_match':
+                                            self.server_dnsname},
                                            signature='sv')
                     net_obj.Set(WPAS_DBUS_NETWORK, "Properties", args,
                                 dbus_interface=dbus.PROPERTIES_IFACE)
@@ -1564,8 +1565,8 @@ def test_dbus_connect_eap(dev, apdev):
                     iface.Disconnect()
                     logger.info("Set non-matching dNSName constraint")
                     net_obj = bus.get_object(WPAS_DBUS_SERVICE, self.netw)
-                    args = dbus.Dictionary({ 'altsubject_match':
-                                             self.server_dnsname + "FOO" },
+                    args = dbus.Dictionary({'altsubject_match':
+                                            self.server_dnsname + "FOO"},
                                            signature='sv')
                     net_obj.Set(WPAS_DBUS_NETWORK, "Properties", args,
                                 dbus_interface=dbus.PROPERTIES_IFACE)
@@ -1606,15 +1607,15 @@ def test_dbus_connect_eap(dev, apdev):
 
         def run_connect(self, *args):
             logger.debug("run_connect")
-            args = dbus.Dictionary({ 'ssid': ssid,
-                                     'key_mgmt': 'IEEE8021X',
-                                     'eapol_flags': 0,
-                                     'eap': 'TTLS',
-                                     'anonymous_identity': 'ttls',
-                                     'identity': 'pap user',
-                                     'ca_cert': 'auth_serv/ca.pem',
-                                     'phase2': 'auth=PAP',
-                                     'scan_freq': 2412 },
+            args = dbus.Dictionary({'ssid': ssid,
+                                    'key_mgmt': 'IEEE8021X',
+                                    'eapol_flags': 0,
+                                    'eap': 'TTLS',
+                                    'anonymous_identity': 'ttls',
+                                    'identity': 'pap user',
+                                    'ca_cert': 'auth_serv/ca.pem',
+                                    'phase2': 'auth=PAP',
+                                    'scan_freq': 2412},
                                    signature='sv')
             self.netw = iface.AddNetwork(args)
             iface.SelectNetwork(self.netw)
@@ -1631,15 +1632,15 @@ def test_dbus_connect_eap(dev, apdev):
 
 def test_dbus_network(dev, apdev):
     """D-Bus AddNetwork/RemoveNetwork parameters and error cases"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
-    args = dbus.Dictionary({ 'ssid': "foo",
-                             'key_mgmt': 'WPA-PSK',
-                             'psk': "12345678",
-                             'identity': dbus.ByteArray([ 1, 2 ]),
-                             'priority': dbus.Int32(0),
-                             'scan_freq': dbus.UInt32(2412) },
+    args = dbus.Dictionary({'ssid': "foo",
+                            'key_mgmt': 'WPA-PSK',
+                            'psk': "12345678",
+                            'identity': dbus.ByteArray([1, 2]),
+                            'priority': dbus.Int32(0),
+                            'scan_freq': dbus.UInt32(2412)},
                            signature='sv')
     netw = iface.AddNetwork(args)
     id = int(dev[0].list_networks()[0]['id'])
@@ -1648,10 +1649,10 @@ def test_dbus_network(dev, apdev):
         raise Exception("Invalid scan_freq value: " + str(val))
     iface.RemoveNetwork(netw)
 
-    args = dbus.Dictionary({ 'ssid': "foo",
-                             'key_mgmt': 'NONE',
-                             'scan_freq': "2412 2432",
-                             'freq_list': "2412 2417 2432" },
+    args = dbus.Dictionary({'ssid': "foo",
+                            'key_mgmt': 'NONE',
+                            'scan_freq': "2412 2432",
+                            'freq_list': "2412 2417 2432"},
                            signature='sv')
     netw = iface.AddNetwork(args)
     id = int(dev[0].list_networks()[0]['id'])
@@ -1675,11 +1676,11 @@ def test_dbus_network(dev, apdev):
         if "NetworkUnknown" not in str(e):
             raise Exception("Unexpected error message for invalid RemoveNetwork: " + str(e))
 
-    args = dbus.Dictionary({ 'ssid': "foo1", 'key_mgmt': 'NONE',
-                             'identity': "testuser", 'scan_freq': '2412' },
+    args = dbus.Dictionary({'ssid': "foo1", 'key_mgmt': 'NONE',
+                            'identity': "testuser", 'scan_freq': '2412'},
                            signature='sv')
     netw1 = iface.AddNetwork(args)
-    args = dbus.Dictionary({ 'ssid': "foo2", 'key_mgmt': 'NONE' },
+    args = dbus.Dictionary({'ssid': "foo2", 'key_mgmt': 'NONE'},
                            signature='sv')
     netw2 = iface.AddNetwork(args)
     res = if_obj.Get(WPAS_DBUS_IFACE, "Networks",
@@ -1712,7 +1713,7 @@ def test_dbus_network(dev, apdev):
         if "Error.Failed: wrong property type" not in str(e):
             raise Exception("Unexpected error message for invalid Set(Enabled,1): " + str(e))
 
-    args = dbus.Dictionary({ 'ssid': "foo1new" }, signature='sv')
+    args = dbus.Dictionary({'ssid': "foo1new"}, signature='sv')
     net_obj.Set(WPAS_DBUS_NETWORK, "Properties", args,
                 dbus_interface=dbus.PROPERTIES_IFACE)
     res = net_obj.Get(WPAS_DBUS_NETWORK, "Properties",
@@ -1729,11 +1730,11 @@ def test_dbus_network(dev, apdev):
         raise Exception("Unexpected number of networks")
     iface.RemoveAllNetworks()
 
-    tests = [ dbus.Dictionary({ 'psk': "1234567" }, signature='sv'),
-              dbus.Dictionary({ 'identity': dbus.ByteArray() },
-                              signature='sv'),
-              dbus.Dictionary({ 'identity': dbus.Byte(1) }, signature='sv'),
-              dbus.Dictionary({ 'identity': "" }, signature='sv') ]
+    tests = [dbus.Dictionary({'psk': "1234567"}, signature='sv'),
+             dbus.Dictionary({'identity': dbus.ByteArray()},
+                             signature='sv'),
+             dbus.Dictionary({'identity': dbus.Byte(1)}, signature='sv'),
+             dbus.Dictionary({'identity': ""}, signature='sv')]
     for args in tests:
         try:
             iface.AddNetwork(args)
@@ -1744,11 +1745,11 @@ def test_dbus_network(dev, apdev):
 
 def test_dbus_network_oom(dev, apdev):
     """D-Bus AddNetwork/RemoveNetwork parameters and OOM error cases"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
-    args = dbus.Dictionary({ 'ssid': "foo1", 'key_mgmt': 'NONE',
-                             'identity': "testuser", 'scan_freq': '2412' },
+    args = dbus.Dictionary({'ssid': "foo1", 'key_mgmt': 'NONE',
+                            'identity': "testuser", 'scan_freq': '2412'},
                            signature='sv')
     netw1 = iface.AddNetwork(args)
     net_obj = bus.get_object(WPAS_DBUS_SERVICE, netw1)
@@ -1767,7 +1768,7 @@ def test_dbus_network_oom(dev, apdev):
         iface.RemoveNetwork(dbus.ObjectPath("/fi/w1/wpa_supplicant1/Interfaces/1234/Networks/1234"))
 
     with alloc_fail(dev[0], 1, "wpa_dbus_register_object_per_iface;wpas_dbus_register_network"):
-        args = dbus.Dictionary({ 'ssid': "foo2", 'key_mgmt': 'NONE' },
+        args = dbus.Dictionary({'ssid': "foo2", 'key_mgmt': 'NONE'},
                                signature='sv')
         try:
             netw = iface.AddNetwork(args)
@@ -1791,28 +1792,28 @@ def test_dbus_network_oom(dev, apdev):
                          "=wpa_config_add_network;wpas_dbus_handler_add_network",
                          "AddNetwork",
                          "UnknownError: wpa_supplicant could not add a network"):
-        args = dbus.Dictionary({ 'ssid': "foo2", 'key_mgmt': 'NONE' },
+        args = dbus.Dictionary({'ssid': "foo2", 'key_mgmt': 'NONE'},
                                signature='sv')
         netw = iface.AddNetwork(args)
 
-    tests = [ (1,
-               'wpa_dbus_dict_get_entry;set_network_properties;wpas_dbus_handler_add_network',
-               dbus.Dictionary({ 'ssid': dbus.ByteArray(b' ') },
-                               signature='sv')),
-              (1, '=set_network_properties;wpas_dbus_handler_add_network',
-               dbus.Dictionary({ 'ssid': 'foo' }, signature='sv')),
-              (1, '=set_network_properties;wpas_dbus_handler_add_network',
-               dbus.Dictionary({ 'eap': 'foo' }, signature='sv')),
-              (1, '=set_network_properties;wpas_dbus_handler_add_network',
-               dbus.Dictionary({ 'priority': dbus.UInt32(1) },
-                               signature='sv')),
-              (1, '=set_network_properties;wpas_dbus_handler_add_network',
-               dbus.Dictionary({ 'priority': dbus.Int32(1) },
-                               signature='sv')),
-              (1, '=set_network_properties;wpas_dbus_handler_add_network',
-               dbus.Dictionary({ 'ssid': dbus.ByteArray(b' ') },
-                               signature='sv')) ]
-    for (count,funcs,args) in tests:
+    tests = [(1,
+              'wpa_dbus_dict_get_entry;set_network_properties;wpas_dbus_handler_add_network',
+              dbus.Dictionary({'ssid': dbus.ByteArray(b' ')},
+                              signature='sv')),
+             (1, '=set_network_properties;wpas_dbus_handler_add_network',
+              dbus.Dictionary({'ssid': 'foo'}, signature='sv')),
+             (1, '=set_network_properties;wpas_dbus_handler_add_network',
+              dbus.Dictionary({'eap': 'foo'}, signature='sv')),
+             (1, '=set_network_properties;wpas_dbus_handler_add_network',
+              dbus.Dictionary({'priority': dbus.UInt32(1)},
+                              signature='sv')),
+             (1, '=set_network_properties;wpas_dbus_handler_add_network',
+              dbus.Dictionary({'priority': dbus.Int32(1)},
+                              signature='sv')),
+             (1, '=set_network_properties;wpas_dbus_handler_add_network',
+              dbus.Dictionary({'ssid': dbus.ByteArray(b' ')},
+                              signature='sv'))]
+    for (count, funcs, args) in tests:
         with alloc_fail_dbus(dev[0], count, funcs, "AddNetwork", "InvalidArgs"):
             netw = iface.AddNetwork(args)
 
@@ -1842,10 +1843,10 @@ def test_dbus_interface(dev, apdev):
         subprocess.call(['iw', 'reg', 'set', '00'])
 
 def _test_dbus_interface(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wpas = dbus.Interface(wpas_obj, WPAS_DBUS_SERVICE)
 
-    params = dbus.Dictionary({ 'Ifname': 'lo', 'Driver': 'none' },
+    params = dbus.Dictionary({'Ifname': 'lo', 'Driver': 'none'},
                              signature='sv')
     path = wpas.CreateInterface(params)
     logger.debug("New interface path: " + str(path))
@@ -1853,10 +1854,10 @@ def _test_dbus_interface(dev, apdev):
     if path != path2:
         raise Exception("Interface object mismatch")
 
-    params = dbus.Dictionary({ 'Ifname': 'lo',
-                               'Driver': 'none',
-                               'ConfigFile': 'foo',
-                               'BridgeIfname': 'foo', },
+    params = dbus.Dictionary({'Ifname': 'lo',
+                              'Driver': 'none',
+                              'ConfigFile': 'foo',
+                              'BridgeIfname': 'foo',},
                              signature='sv')
     try:
         wpas.CreateInterface(params)
@@ -1873,8 +1874,8 @@ def _test_dbus_interface(dev, apdev):
         if "InterfaceUnknown" not in str(e):
             raise Exception("Unexpected error message for invalid RemoveInterface: " + str(e))
 
-    params = dbus.Dictionary({ 'Ifname': 'lo', 'Driver': 'none',
-                               'Foo': 123 },
+    params = dbus.Dictionary({'Ifname': 'lo', 'Driver': 'none',
+                              'Foo': 123},
                              signature='sv')
     try:
         wpas.CreateInterface(params)
@@ -1883,7 +1884,7 @@ def _test_dbus_interface(dev, apdev):
         if "InvalidArgs" not in str(e):
             raise Exception("Unexpected error message for invalid CreateInterface: " + str(e))
 
-    params = dbus.Dictionary({ 'Driver': 'none' }, signature='sv')
+    params = dbus.Dictionary({'Driver': 'none'}, signature='sv')
     try:
         wpas.CreateInterface(params)
         raise Exception("Invalid CreateInterface() accepted")
@@ -1900,17 +1901,17 @@ def _test_dbus_interface(dev, apdev):
 
 def test_dbus_interface_oom(dev, apdev):
     """D-Bus CreateInterface/GetInterface/RemoveInterface OOM error cases"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wpas = dbus.Interface(wpas_obj, WPAS_DBUS_SERVICE)
 
     with alloc_fail_dbus(dev[0], 1, "wpa_dbus_dict_get_entry;wpas_dbus_handler_create_interface", "CreateInterface", "InvalidArgs"):
-        params = dbus.Dictionary({ 'Ifname': 'lo', 'Driver': 'none' },
+        params = dbus.Dictionary({'Ifname': 'lo', 'Driver': 'none'},
                                  signature='sv')
         wpas.CreateInterface(params)
 
     for i in range(1, 1000):
         dev[0].request("TEST_ALLOC_FAIL %d:wpa_supplicant_add_iface;wpas_dbus_handler_create_interface" % i)
-        params = dbus.Dictionary({ 'Ifname': 'lo', 'Driver': 'none' },
+        params = dbus.Dictionary({'Ifname': 'lo', 'Driver': 'none'},
                                  signature='sv')
         try:
             npath = wpas.CreateInterface(params)
@@ -1927,15 +1928,15 @@ def test_dbus_interface_oom(dev, apdev):
         except dbus.exceptions.DBusException as e:
             pass
 
-    for arg in [ 'Driver', 'Ifname', 'ConfigFile', 'BridgeIfname' ]:
+    for arg in ['Driver', 'Ifname', 'ConfigFile', 'BridgeIfname']:
         with alloc_fail_dbus(dev[0], 1, "=wpas_dbus_handler_create_interface",
                              "CreateInterface"):
-            params = dbus.Dictionary({ arg: 'foo' }, signature='sv')
+            params = dbus.Dictionary({arg: 'foo'}, signature='sv')
             wpas.CreateInterface(params)
 
 def test_dbus_blob(dev, apdev):
     """D-Bus AddNetwork/RemoveNetwork parameters and error cases"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     blob = dbus.ByteArray(b"\x01\x02\x03")
@@ -2010,7 +2011,7 @@ def test_dbus_blob(dev, apdev):
 
 def test_dbus_blob_oom(dev, apdev):
     """D-Bus AddNetwork/RemoveNetwork OOM error cases"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     for i in range(1, 4):
@@ -2020,7 +2021,7 @@ def test_dbus_blob_oom(dev, apdev):
 
 def test_dbus_autoscan(dev, apdev):
     """D-Bus Autoscan()"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     iface.AutoScan("foo")
@@ -2030,7 +2031,7 @@ def test_dbus_autoscan(dev, apdev):
 
 def test_dbus_autoscan_oom(dev, apdev):
     """D-Bus Autoscan() OOM"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     with alloc_fail_dbus(dev[0], 1, "wpas_dbus_handler_autoscan", "AutoScan"):
@@ -2039,10 +2040,10 @@ def test_dbus_autoscan_oom(dev, apdev):
 
 def test_dbus_tdls_invalid(dev, apdev):
     """D-Bus invalid TDLS operations"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
-    hapd = hostapd.add_ap(apdev[0], { "ssid": "test-open" })
+    hapd = hostapd.add_ap(apdev[0], {"ssid": "test-open"})
     connect_2sta_open(dev, hapd)
     addr1 = dev[1].p2p_interface_addr()
 
@@ -2101,7 +2102,7 @@ def test_dbus_tdls_invalid(dev, apdev):
 
 def test_dbus_tdls_oom(dev, apdev):
     """D-Bus TDLS operations during OOM"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     with alloc_fail_dbus(dev[0], 1, "wpa_tdls_add_peer", "TDLSSetup",
@@ -2110,10 +2111,10 @@ def test_dbus_tdls_oom(dev, apdev):
 
 def test_dbus_tdls(dev, apdev):
     """D-Bus TDLS"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
-    hapd = hostapd.add_ap(apdev[0], { "ssid": "test-open" })
+    hapd = hostapd.add_ap(apdev[0], {"ssid": "test-open"})
     connect_2sta_open(dev, hapd)
 
     addr1 = dev[1].p2p_interface_addr()
@@ -2181,10 +2182,10 @@ def test_dbus_tdls_channel_switch(dev, apdev):
     if flags & 0x800000000 == 0:
         raise HwsimSkip("Driver does not support TDLS channel switching")
 
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
-    hapd = hostapd.add_ap(apdev[0], { "ssid": "test-open" })
+    hapd = hostapd.add_ap(apdev[0], {"ssid": "test-open"})
     connect_2sta_open(dev, hapd)
 
     addr1 = dev[1].p2p_interface_addr()
@@ -2227,7 +2228,7 @@ def test_dbus_tdls_channel_switch(dev, apdev):
                 logger.info("Unexpected TDLSStatus: " + res)
 
             # Unknown dict entry
-            args = dbus.Dictionary({ 'Foobar': dbus.Byte(1) },
+            args = dbus.Dictionary({'Foobar': dbus.Byte(1)},
                                    signature='sv')
             try:
                 iface.TDLSChannelSwitch(args)
@@ -2244,7 +2245,7 @@ def test_dbus_tdls_channel_switch(dev, apdev):
                     raise Exception("Unexpected exception")
 
             # Missing Frequency
-            args = dbus.Dictionary({ 'OperClass': dbus.Byte(1) },
+            args = dbus.Dictionary({'OperClass': dbus.Byte(1)},
                                    signature='sv')
             try:
                 iface.TDLSChannelSwitch(args)
@@ -2253,8 +2254,8 @@ def test_dbus_tdls_channel_switch(dev, apdev):
                     raise Exception("Unexpected exception")
 
             # Missing PeerAddress
-            args = dbus.Dictionary({ 'OperClass': dbus.Byte(1),
-                                     'Frequency': dbus.UInt32(2417) },
+            args = dbus.Dictionary({'OperClass': dbus.Byte(1),
+                                     'Frequency': dbus.UInt32(2417)},
                                    signature='sv')
             try:
                 iface.TDLSChannelSwitch(args)
@@ -2263,15 +2264,15 @@ def test_dbus_tdls_channel_switch(dev, apdev):
                     raise Exception("Unexpected exception")
 
             # Valid parameters
-            args = dbus.Dictionary({ 'OperClass': dbus.Byte(1),
-                                     'Frequency': dbus.UInt32(2417),
-                                     'PeerAddress': addr1,
-                                     'SecChannelOffset': dbus.UInt32(0),
-                                     'CenterFrequency1': dbus.UInt32(0),
-                                     'CenterFrequency2': dbus.UInt32(0),
-                                     'Bandwidth': dbus.UInt32(20),
-                                     'HT': dbus.Boolean(False),
-                                     'VHT': dbus.Boolean(False) },
+            args = dbus.Dictionary({'OperClass': dbus.Byte(1),
+                                    'Frequency': dbus.UInt32(2417),
+                                    'PeerAddress': addr1,
+                                    'SecChannelOffset': dbus.UInt32(0),
+                                    'CenterFrequency1': dbus.UInt32(0),
+                                    'CenterFrequency2': dbus.UInt32(0),
+                                    'Bandwidth': dbus.UInt32(20),
+                                    'HT': dbus.Boolean(False),
+                                    'VHT': dbus.Boolean(False)},
                                    signature='sv')
             iface.TDLSChannelSwitch(args)
 
@@ -2294,7 +2295,7 @@ def test_dbus_tdls_channel_switch(dev, apdev):
 
 def test_dbus_pkcs11(dev, apdev):
     """D-Bus SetPKCS11EngineAndModulePath()"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     try:
@@ -2337,7 +2338,7 @@ def test_dbus_apscan(dev, apdev):
         dev[0].request("AP_SCAN 1")
 
 def _test_dbus_apscan(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     res = if_obj.Get(WPAS_DBUS_IFACE, "ApScan",
                      dbus_interface=dbus.PROPERTIES_IFACE)
@@ -2379,7 +2380,7 @@ def test_dbus_pmf(dev, apdev):
         dev[0].request("SET pmf 0")
 
 def _test_dbus_pmf(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     dev[0].set("pmf", "0")
     res = if_obj.Get(WPAS_DBUS_IFACE, "Pmf",
@@ -2400,14 +2401,14 @@ def _test_dbus_pmf(dev, apdev):
 
 def test_dbus_fastreauth(dev, apdev):
     """D-Bus Get/Set FastReauth"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     res = if_obj.Get(WPAS_DBUS_IFACE, "FastReauth",
                      dbus_interface=dbus.PROPERTIES_IFACE)
     if res != True:
         raise Exception("Unexpected initial FastReauth value: " + str(res))
 
-    for i in [ False, True ]:
+    for i in [False, True]:
         if_obj.Set(WPAS_DBUS_IFACE, "FastReauth", dbus.Boolean(i),
                      dbus_interface=dbus.PROPERTIES_IFACE)
         res = if_obj.Get(WPAS_DBUS_IFACE, "FastReauth",
@@ -2428,7 +2429,7 @@ def test_dbus_fastreauth(dev, apdev):
 
 def test_dbus_bss_expire(dev, apdev):
     """D-Bus Get/Set BSSExpireAge and BSSExpireCount"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     if_obj.Set(WPAS_DBUS_IFACE, "BSSExpireAge", dbus.UInt32(179),
                dbus_interface=dbus.PROPERTIES_IFACE)
@@ -2490,7 +2491,7 @@ def test_dbus_country(dev, apdev):
         subprocess.call(['iw', 'reg', 'set', '00'])
 
 def _test_dbus_country(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     # work around issues with possible pending regdom event from the end of
     # the previous test case
@@ -2551,7 +2552,7 @@ def test_dbus_scan_interval(dev, apdev):
         dev[0].request("SCAN_INTERVAL 5")
 
 def _test_dbus_scan_interval(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     if_obj.Set(WPAS_DBUS_IFACE, "ScanInterval", dbus.Int32(3),
                dbus_interface=dbus.PROPERTIES_IFACE)
@@ -2581,7 +2582,7 @@ def _test_dbus_scan_interval(dev, apdev):
 
 def test_dbus_probe_req_reporting(dev, apdev):
     """D-Bus Probe Request reporting"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     dev[1].p2p_find(social=True)
 
@@ -2616,7 +2617,7 @@ def test_dbus_probe_req_reporting(dev, apdev):
         def run_test(self, *args):
             logger.debug("run_test")
             p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
-            params = dbus.Dictionary({ 'frequency': 2412 })
+            params = dbus.Dictionary({'frequency': 2412})
             p2p.GroupAdd(params)
             return False
 
@@ -2645,7 +2646,7 @@ def test_dbus_probe_req_reporting(dev, apdev):
 
 def test_dbus_probe_req_reporting_oom(dev, apdev):
     """D-Bus Probe Request reporting (OOM)"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     # Need to make sure this process has not already subscribed to avoid false
@@ -2669,7 +2670,7 @@ def test_dbus_probe_req_reporting_oom(dev, apdev):
 
 def test_dbus_p2p_invalid(dev, apdev):
     """D-Bus invalid P2P operations"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     try:
@@ -2686,11 +2687,11 @@ def test_dbus_p2p_invalid(dev, apdev):
         if "InvalidArgs" not in str(e):
             raise Exception("Unexpected error message for invalid RejectPeer(): " + str(e))
 
-    tests = [ { },
-              { 'peer': 'foo' },
-              { 'foo': "bar" },
-              { 'iface': "abc" },
-              { 'iface': 123 } ]
+    tests = [{},
+             {'peer': 'foo'},
+             {'foo': "bar"},
+             {'iface': "abc"},
+             {'iface': 123}]
     for t in tests:
         try:
             p2p.RemoveClient(t)
@@ -2699,24 +2700,24 @@ def test_dbus_p2p_invalid(dev, apdev):
             if "InvalidArgs" not in str(e):
                 raise Exception("Unexpected error message for invalid RemoveClient(): " + str(e))
 
-    tests = [ {'DiscoveryType': 'foo'},
-              {'RequestedDeviceTypes': 'foo'},
-              {'RequestedDeviceTypes': ['foo']},
-              {'RequestedDeviceTypes': ['1','2','3','4','5','6','7','8','9',
-                                        '10','11','12','13','14','15','16',
-                                        '17']},
-              {'RequestedDeviceTypes': dbus.Array([], signature="s")},
-              {'RequestedDeviceTypes': dbus.Array([['foo']], signature="as")},
-              {'RequestedDeviceTypes': dbus.Array([], signature="i")},
-              {'RequestedDeviceTypes': [dbus.ByteArray(b'12345678'),
-                                        dbus.ByteArray(b'1234567')]},
-              {'Foo': dbus.Int16(1)},
-              {'Foo': dbus.UInt16(1)},
-              {'Foo': dbus.Int64(1)},
-              {'Foo': dbus.UInt64(1)},
-              {'Foo': dbus.Double(1.23)},
-              {'Foo': dbus.Signature('s')},
-              {'Foo': 'bar'}]
+    tests = [{'DiscoveryType': 'foo'},
+             {'RequestedDeviceTypes': 'foo'},
+             {'RequestedDeviceTypes': ['foo']},
+             {'RequestedDeviceTypes': ['1', '2', '3', '4', '5', '6', '7', '8',
+                                       '9', '10', '11', '12', '13', '14', '15',
+                                       '16', '17']},
+             {'RequestedDeviceTypes': dbus.Array([], signature="s")},
+             {'RequestedDeviceTypes': dbus.Array([['foo']], signature="as")},
+             {'RequestedDeviceTypes': dbus.Array([], signature="i")},
+             {'RequestedDeviceTypes': [dbus.ByteArray(b'12345678'),
+                                       dbus.ByteArray(b'1234567')]},
+             {'Foo': dbus.Int16(1)},
+             {'Foo': dbus.UInt16(1)},
+             {'Foo': dbus.Int64(1)},
+             {'Foo': dbus.UInt64(1)},
+             {'Foo': dbus.Double(1.23)},
+             {'Foo': dbus.Signature('s')},
+             {'Foo': 'bar'}]
     for t in tests:
         try:
             p2p.Find(dbus.Dictionary(t))
@@ -2725,9 +2726,9 @@ def test_dbus_p2p_invalid(dev, apdev):
             if "InvalidArgs" not in str(e):
                 raise Exception("Unexpected error message for invalid Find(): " + str(e))
 
-    for p in [ "/foo",
-               "/fi/w1/wpa_supplicant1/Interfaces/1234",
-               "/fi/w1/wpa_supplicant1/Interfaces/1234/Networks/1234" ]:
+    for p in ["/foo",
+              "/fi/w1/wpa_supplicant1/Interfaces/1234",
+              "/fi/w1/wpa_supplicant1/Interfaces/1234/Networks/1234"]:
         try:
             p2p.RemovePersistentGroup(dbus.ObjectPath(p))
             raise Exception("Invalid RemovePersistentGroup accepted")
@@ -2766,8 +2767,8 @@ def test_dbus_p2p_invalid(dev, apdev):
 
     try:
         dev[0].request("P2P_SET disabled 1")
-        args = { 'duration1': 30000, 'interval1': 102400,
-                 'duration2': 20000, 'interval2': 102400 }
+        args = {'duration1': 30000, 'interval1': 102400,
+                'duration2': 20000, 'interval2': 102400}
         p2p.PresenceRequest(args)
         raise Exception("Invalid PresenceRequest accepted")
     except dbus.exceptions.DBusException as e:
@@ -2813,10 +2814,10 @@ def test_dbus_p2p_invalid(dev, apdev):
 
     try:
         dev[0].request("P2P_SET disabled 1")
-        args = { 'peer': path,
-                 'join': True,
-                 'wps_method': 'pbc',
-                 'frequency': 2412 }
+        args = {'peer': path,
+                'join': True,
+                'wps_method': 'pbc',
+                'frequency': 2412}
         pin = p2p.Connect(args)
         raise Exception("Invalid Connect accepted")
     except dbus.exceptions.DBusException as e:
@@ -2825,9 +2826,9 @@ def test_dbus_p2p_invalid(dev, apdev):
     finally:
         dev[0].request("P2P_SET disabled 0")
 
-    tests = [ { 'frequency': dbus.Int32(-1) },
-              { 'wps_method': 'pbc' },
-              { 'wps_method': 'foo' } ]
+    tests = [{'frequency': dbus.Int32(-1)},
+             {'wps_method': 'pbc'},
+             {'wps_method': 'foo'}]
     for args in tests:
         try:
             pin = p2p.Connect(args)
@@ -2838,7 +2839,7 @@ def test_dbus_p2p_invalid(dev, apdev):
 
     try:
         dev[0].request("P2P_SET disabled 1")
-        args = { 'peer': path }
+        args = {'peer': path}
         pin = p2p.Invite(args)
         raise Exception("Invalid Invite accepted")
     except dbus.exceptions.DBusException as e:
@@ -2848,29 +2849,29 @@ def test_dbus_p2p_invalid(dev, apdev):
         dev[0].request("P2P_SET disabled 0")
 
     try:
-        args = { 'foo': 'bar' }
+        args = {'foo': 'bar'}
         pin = p2p.Invite(args)
         raise Exception("Invalid Invite accepted")
     except dbus.exceptions.DBusException as e:
         if "InvalidArgs" not in str(e):
             raise Exception("Unexpected error message for invalid Connect: " + str(e))
 
-    tests = [ (path, 'display', "InvalidArgs"),
-              (dbus.ObjectPath(path + "/Peers/00112233445566"),
-               'display',
-               "UnknownError: Failed to send provision discovery request"),
-              (dbus.ObjectPath(path + "/Peers/00112233445566"),
-               'keypad',
-               "UnknownError: Failed to send provision discovery request"),
-              (dbus.ObjectPath(path + "/Peers/00112233445566"),
-               'pbc',
-               "UnknownError: Failed to send provision discovery request"),
-              (dbus.ObjectPath(path + "/Peers/00112233445566"),
-               'pushbutton',
-               "UnknownError: Failed to send provision discovery request"),
-              (dbus.ObjectPath(path + "/Peers/00112233445566"),
-               'foo', "InvalidArgs") ]
-    for (p,method,err) in tests:
+    tests = [(path, 'display', "InvalidArgs"),
+             (dbus.ObjectPath(path + "/Peers/00112233445566"),
+              'display',
+              "UnknownError: Failed to send provision discovery request"),
+             (dbus.ObjectPath(path + "/Peers/00112233445566"),
+              'keypad',
+              "UnknownError: Failed to send provision discovery request"),
+             (dbus.ObjectPath(path + "/Peers/00112233445566"),
+              'pbc',
+              "UnknownError: Failed to send provision discovery request"),
+             (dbus.ObjectPath(path + "/Peers/00112233445566"),
+              'pushbutton',
+              "UnknownError: Failed to send provision discovery request"),
+             (dbus.ObjectPath(path + "/Peers/00112233445566"),
+              'foo', "InvalidArgs")]
+    for (p, method, err) in tests:
         try:
             p2p.ProvisionDiscoveryRequest(p, method)
             raise Exception("Invalid ProvisionDiscoveryRequest accepted")
@@ -2891,55 +2892,56 @@ def test_dbus_p2p_invalid(dev, apdev):
 
 def test_dbus_p2p_oom(dev, apdev):
     """D-Bus P2P operations and OOM"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     with alloc_fail_dbus(dev[0], 1, "_wpa_dbus_dict_entry_get_string_array",
                          "Find", "InvalidArgs"):
-        p2p.Find(dbus.Dictionary({ 'Foo': [ 'bar' ] }))
+        p2p.Find(dbus.Dictionary({'Foo': ['bar']}))
 
     with alloc_fail_dbus(dev[0], 2, "_wpa_dbus_dict_entry_get_string_array",
                          "Find", "InvalidArgs"):
-        p2p.Find(dbus.Dictionary({ 'Foo': [ 'bar' ] }))
+        p2p.Find(dbus.Dictionary({'Foo': ['bar']}))
 
     with alloc_fail_dbus(dev[0], 10, "_wpa_dbus_dict_entry_get_string_array",
                          "Find", "InvalidArgs"):
-        p2p.Find(dbus.Dictionary({ 'Foo': [ '1','2','3','4','5','6','7','8','9' ] }))
+        p2p.Find(dbus.Dictionary({'Foo': ['1', '2', '3', '4', '5', '6', '7',
+                                          '8', '9']}))
 
     with alloc_fail_dbus(dev[0], 1, ":=_wpa_dbus_dict_entry_get_binarray",
                          "Find", "InvalidArgs"):
-        p2p.Find(dbus.Dictionary({ 'Foo': [ dbus.ByteArray(b'123') ] }))
+        p2p.Find(dbus.Dictionary({'Foo': [dbus.ByteArray(b'123')]}))
 
     with alloc_fail_dbus(dev[0], 1, "_wpa_dbus_dict_entry_get_byte_array;_wpa_dbus_dict_entry_get_binarray",
                          "Find", "InvalidArgs"):
-        p2p.Find(dbus.Dictionary({ 'Foo': [ dbus.ByteArray(b'123') ] }))
+        p2p.Find(dbus.Dictionary({'Foo': [dbus.ByteArray(b'123')]}))
 
     with alloc_fail_dbus(dev[0], 2, "=_wpa_dbus_dict_entry_get_binarray",
                          "Find", "InvalidArgs"):
-        p2p.Find(dbus.Dictionary({ 'Foo': [ dbus.ByteArray(b'123'),
-                                            dbus.ByteArray(b'123'),
-                                            dbus.ByteArray(b'123'),
-                                            dbus.ByteArray(b'123'),
-                                            dbus.ByteArray(b'123'),
-                                            dbus.ByteArray(b'123'),
-                                            dbus.ByteArray(b'123'),
-                                            dbus.ByteArray(b'123'),
-                                            dbus.ByteArray(b'123'),
-                                            dbus.ByteArray(b'123'),
-                                            dbus.ByteArray(b'123') ] }))
+        p2p.Find(dbus.Dictionary({'Foo': [dbus.ByteArray(b'123'),
+                                          dbus.ByteArray(b'123'),
+                                          dbus.ByteArray(b'123'),
+                                          dbus.ByteArray(b'123'),
+                                          dbus.ByteArray(b'123'),
+                                          dbus.ByteArray(b'123'),
+                                          dbus.ByteArray(b'123'),
+                                          dbus.ByteArray(b'123'),
+                                          dbus.ByteArray(b'123'),
+                                          dbus.ByteArray(b'123'),
+                                          dbus.ByteArray(b'123')]}))
 
     with alloc_fail_dbus(dev[0], 1, "wpabuf_alloc_ext_data;_wpa_dbus_dict_entry_get_binarray",
                          "Find", "InvalidArgs"):
-        p2p.Find(dbus.Dictionary({ 'Foo': [ dbus.ByteArray(b'123') ] }))
+        p2p.Find(dbus.Dictionary({'Foo': [dbus.ByteArray(b'123')]}))
 
     with alloc_fail_dbus(dev[0], 1, "_wpa_dbus_dict_fill_value_from_variant;wpas_dbus_handler_p2p_find",
                          "Find", "InvalidArgs"):
-        p2p.Find(dbus.Dictionary({ 'Foo': path }))
+        p2p.Find(dbus.Dictionary({'Foo': path}))
 
     with alloc_fail_dbus(dev[0], 1, "_wpa_dbus_dict_entry_get_byte_array",
                          "AddService", "InvalidArgs"):
-        args = { 'service_type': 'bonjour',
-                 'response': dbus.ByteArray(500*b'b') }
+        args = {'service_type': 'bonjour',
+                'response': dbus.ByteArray(500*b'b')}
         p2p.AddService(args)
 
     with alloc_fail_dbus(dev[0], 2, "_wpa_dbus_dict_entry_get_byte_array",
@@ -2954,7 +2956,7 @@ def test_dbus_p2p_discovery(dev, apdev):
         dev[1].request("VENDOR_ELEM_REMOVE 1 *")
 
 def run_dbus_p2p_discovery(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     addr0 = dev[0].p2p_dev_addr()
@@ -2964,7 +2966,7 @@ def run_dbus_p2p_discovery(dev, apdev):
     dev[1].request("VENDOR_ELEM_ADD 1 dd06001122335566")
     dev[1].p2p_listen()
     addr1 = dev[1].p2p_dev_addr()
-    a1 = binascii.unhexlify(addr1.replace(':',''))
+    a1 = binascii.unhexlify(addr1.replace(':', ''))
 
     wfd_devinfo = "00001c440028"
     dev[2].request("SET wifi_display 1")
@@ -2972,7 +2974,7 @@ def run_dbus_p2p_discovery(dev, apdev):
     wfd = binascii.unhexlify('000006' + wfd_devinfo)
     dev[2].p2p_listen()
     addr2 = dev[2].p2p_dev_addr()
-    a2 = binascii.unhexlify(addr2.replace(':',''))
+    a2 = binascii.unhexlify(addr2.replace(':', ''))
 
     res = if_obj.GetAll(WPAS_DBUS_IFACE_P2PDEVICE,
                         dbus_interface=dbus.PROPERTIES_IFACE)
@@ -2983,7 +2985,7 @@ def run_dbus_p2p_discovery(dev, apdev):
 
     args = {'DiscoveryType': 'social',
             'RequestedDeviceTypes': [dbus.ByteArray(b'12345678')],
-            'Timeout': dbus.Int32(1) }
+            'Timeout': dbus.Int32(1)}
     p2p.Find(dbus.Dictionary(args))
     p2p.StopFind()
 
@@ -3135,7 +3137,7 @@ def run_dbus_p2p_discovery(dev, apdev):
 
 def test_dbus_p2p_discovery_freq(dev, apdev):
     """D-Bus P2P discovery on a specific non-social channel"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     addr1 = dev[1].p2p_dev_addr()
@@ -3176,7 +3178,7 @@ def test_dbus_p2p_discovery_freq(dev, apdev):
 
 def test_dbus_p2p_service_discovery(dev, apdev):
     """D-Bus P2P service discovery"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     addr0 = dev[0].p2p_dev_addr()
@@ -3185,9 +3187,9 @@ def test_dbus_p2p_service_discovery(dev, apdev):
     bonjour_query = dbus.ByteArray(binascii.unhexlify('0b5f6166706f766572746370c00c000c01'))
     bonjour_response = dbus.ByteArray(binascii.unhexlify('074578616d706c65c027'))
 
-    args = { 'service_type': 'bonjour',
-             'query': bonjour_query,
-             'response': bonjour_response }
+    args = {'service_type': 'bonjour',
+            'query': bonjour_query,
+            'response': bonjour_response}
     p2p.AddService(args)
     p2p.FlushService()
     p2p.AddService(args)
@@ -3199,8 +3201,8 @@ def test_dbus_p2p_service_discovery(dev, apdev):
         if "InvalidArgs" not in str(e):
             raise Exception("Unexpected error message for invalid DeleteService(): " + str(e))
 
-    args = { 'service_type': 'bonjour',
-             'query': bonjour_query }
+    args = {'service_type': 'bonjour',
+            'query': bonjour_query}
     p2p.DeleteService(args)
     try:
         p2p.DeleteService(args)
@@ -3209,9 +3211,9 @@ def test_dbus_p2p_service_discovery(dev, apdev):
         if "InvalidArgs" not in str(e):
             raise Exception("Unexpected error message for invalid DeleteService(): " + str(e))
 
-    args = { 'service_type': 'upnp',
-             'version': 0x10,
-             'service': 'uuid:6859dede-8574-59ab-9332-123456789012::upnp:rootdevice' }
+    args = {'service_type': 'upnp',
+            'version': 0x10,
+            'service': 'uuid:6859dede-8574-59ab-9332-123456789012::upnp:rootdevice'}
     p2p.AddService(args)
     p2p.DeleteService(args)
     try:
@@ -3221,18 +3223,18 @@ def test_dbus_p2p_service_discovery(dev, apdev):
         if "InvalidArgs" not in str(e):
             raise Exception("Unexpected error message for invalid DeleteService(): " + str(e))
 
-    tests = [ { 'service_type': 'foo' },
-              { 'service_type': 'foo', 'query': bonjour_query },
-              { 'service_type': 'upnp' },
-              { 'service_type': 'upnp', 'version': 0x10 },
-              { 'service_type': 'upnp',
-                'service': 'uuid:6859dede-8574-59ab-9332-123456789012::upnp:rootdevice' },
-              { 'version': 0x10,
-                'service': 'uuid:6859dede-8574-59ab-9332-123456789012::upnp:rootdevice' },
-              { 'service_type': 'upnp', 'foo': 'bar' },
-              { 'service_type': 'bonjour' },
-              { 'service_type': 'bonjour', 'query': 'foo' },
-              { 'service_type': 'bonjour', 'foo': 'bar' } ]
+    tests = [{'service_type': 'foo'},
+             {'service_type': 'foo', 'query': bonjour_query},
+             {'service_type': 'upnp'},
+             {'service_type': 'upnp', 'version': 0x10},
+             {'service_type': 'upnp',
+              'service': 'uuid:6859dede-8574-59ab-9332-123456789012::upnp:rootdevice'},
+             {'version': 0x10,
+              'service': 'uuid:6859dede-8574-59ab-9332-123456789012::upnp:rootdevice'},
+             {'service_type': 'upnp', 'foo': 'bar'},
+             {'service_type': 'bonjour'},
+             {'service_type': 'bonjour', 'query': 'foo'},
+             {'service_type': 'bonjour', 'foo': 'bar'}]
     for args in tests:
         try:
             p2p.DeleteService(args)
@@ -3241,21 +3243,21 @@ def test_dbus_p2p_service_discovery(dev, apdev):
             if "InvalidArgs" not in str(e):
                 raise Exception("Unexpected error message for invalid DeleteService(): " + str(e))
 
-    tests = [ { 'service_type': 'foo' },
-              { 'service_type': 'upnp' },
-              { 'service_type': 'upnp', 'version': 0x10 },
-              { 'service_type': 'upnp',
-                'service': 'uuid:6859dede-8574-59ab-9332-123456789012::upnp:rootdevice' },
-              { 'version': 0x10,
-                'service': 'uuid:6859dede-8574-59ab-9332-123456789012::upnp:rootdevice' },
-              { 'service_type': 'upnp', 'foo': 'bar' },
-              { 'service_type': 'bonjour' },
-              { 'service_type': 'bonjour', 'query': 'foo' },
-              { 'service_type': 'bonjour', 'response': 'foo' },
-              { 'service_type': 'bonjour', 'query': bonjour_query },
-              { 'service_type': 'bonjour', 'response': bonjour_response },
-              { 'service_type': 'bonjour', 'query': dbus.ByteArray(500*b'a') },
-              { 'service_type': 'bonjour', 'foo': 'bar' } ]
+    tests = [{'service_type': 'foo'},
+             {'service_type': 'upnp'},
+             {'service_type': 'upnp', 'version': 0x10},
+             {'service_type': 'upnp',
+              'service': 'uuid:6859dede-8574-59ab-9332-123456789012::upnp:rootdevice'},
+             {'version': 0x10,
+              'service': 'uuid:6859dede-8574-59ab-9332-123456789012::upnp:rootdevice'},
+             {'service_type': 'upnp', 'foo': 'bar'},
+             {'service_type': 'bonjour'},
+             {'service_type': 'bonjour', 'query': 'foo'},
+             {'service_type': 'bonjour', 'response': 'foo'},
+             {'service_type': 'bonjour', 'query': bonjour_query},
+             {'service_type': 'bonjour', 'response': bonjour_response},
+             {'service_type': 'bonjour', 'query': dbus.ByteArray(500*b'a')},
+             {'service_type': 'bonjour', 'foo': 'bar'}]
     for args in tests:
         try:
             p2p.AddService(args)
@@ -3264,7 +3266,7 @@ def test_dbus_p2p_service_discovery(dev, apdev):
             if "InvalidArgs" not in str(e):
                 raise Exception("Unexpected error message for invalid AddService(): " + str(e))
 
-    args = { 'tlv': dbus.ByteArray(b"\x02\x00\x00\x01") }
+    args = {'tlv': dbus.ByteArray(b"\x02\x00\x00\x01")}
     ref = p2p.ServiceDiscoveryRequest(args)
     p2p.ServiceDiscoveryCancelRequest(ref)
     try:
@@ -3280,33 +3282,33 @@ def test_dbus_p2p_service_discovery(dev, apdev):
         if "InvalidArgs" not in str(e):
             raise Exception("Unexpected error message for invalid AddService(): " + str(e))
 
-    args = { 'service_type': 'upnp',
-             'version': 0x10,
-             'service': 'ssdp:foo' }
+    args = {'service_type': 'upnp',
+            'version': 0x10,
+            'service': 'ssdp:foo'}
     ref = p2p.ServiceDiscoveryRequest(args)
     p2p.ServiceDiscoveryCancelRequest(ref)
 
-    tests =  [ { 'service_type': 'foo' },
-               { 'foo': 'bar' },
-               { 'tlv': 'foo' },
-               { },
-               { 'version': 0 },
-               { 'service_type': 'upnp',
-                 'service': 'ssdp:foo' },
-               { 'service_type': 'upnp',
-                 'version': 0x10 },
-               { 'service_type': 'upnp',
-                 'version': 0x10,
-                 'service': 'ssdp:foo',
-                 'peer_object': dbus.ObjectPath(path + "/Peers") },
-               { 'service_type': 'upnp',
-                 'version': 0x10,
-                 'service': 'ssdp:foo',
-                 'peer_object': path + "/Peers" },
-               { 'service_type': 'upnp',
-                 'version': 0x10,
-                 'service': 'ssdp:foo',
-                 'peer_object': dbus.ObjectPath(path + "/Peers/00112233445566") } ]
+    tests = [{'service_type': 'foo'},
+             {'foo': 'bar'},
+             {'tlv': 'foo'},
+             {},
+             {'version': 0},
+             {'service_type': 'upnp',
+              'service': 'ssdp:foo'},
+             {'service_type': 'upnp',
+              'version': 0x10},
+             {'service_type': 'upnp',
+              'version': 0x10,
+              'service': 'ssdp:foo',
+              'peer_object': dbus.ObjectPath(path + "/Peers")},
+             {'service_type': 'upnp',
+              'version': 0x10,
+              'service': 'ssdp:foo',
+              'peer_object': path + "/Peers"},
+             {'service_type': 'upnp',
+              'version': 0x10,
+              'service': 'ssdp:foo',
+              'peer_object': dbus.ObjectPath(path + "/Peers/00112233445566")}]
     for args in tests:
         try:
             p2p.ServiceDiscoveryRequest(args)
@@ -3315,7 +3317,7 @@ def test_dbus_p2p_service_discovery(dev, apdev):
             if "InvalidArgs" not in str(e):
                 raise Exception("Unexpected error message for invalid ServiceDiscoveryRequest(): " + str(e))
 
-    args = { 'foo': 'bar' }
+    args = {'foo': 'bar'}
     try:
         p2p.ServiceDiscoveryResponse(dbus.Dictionary(args, signature='sv'))
         raise Exception("Invalid ServiceDiscoveryResponse accepted")
@@ -3325,7 +3327,7 @@ def test_dbus_p2p_service_discovery(dev, apdev):
 
 def test_dbus_p2p_service_discovery_query(dev, apdev):
     """D-Bus P2P service discovery query"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     addr0 = dev[0].p2p_dev_addr()
@@ -3351,8 +3353,8 @@ def test_dbus_p2p_service_discovery_query(dev, apdev):
 
         def deviceFound(self, path):
             logger.debug("deviceFound: path=%s" % path)
-            args = { 'peer_object': path,
-                     'tlv': dbus.ByteArray(b"\x02\x00\x00\x01") }
+            args = {'peer_object': path,
+                    'tlv': dbus.ByteArray(b"\x02\x00\x00\x01")}
             p2p.ServiceDiscoveryRequest(args)
 
         def serviceDiscoveryResponse(self, sd_request):
@@ -3383,7 +3385,7 @@ def test_dbus_p2p_service_discovery_external(dev, apdev):
         dev[0].request("P2P_SERV_DISC_EXTERNAL 0")
 
 def _test_dbus_p2p_service_discovery_external(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     addr0 = dev[0].p2p_dev_addr()
@@ -3416,10 +3418,10 @@ def _test_dbus_p2p_service_discovery_external(dev, apdev):
         def serviceDiscoveryRequest(self, sd_request):
             logger.debug("serviceDiscoveryRequest: sd_request=%s" % str(sd_request))
             self.sd = True
-            args = { 'peer_object': sd_request['peer_object'],
-                     'frequency': sd_request['frequency'],
-                     'dialog_token': sd_request['dialog_token'],
-                     'tlvs': dbus.ByteArray(binascii.unhexlify(resp)) }
+            args = {'peer_object': sd_request['peer_object'],
+                    'frequency': sd_request['frequency'],
+                    'dialog_token': sd_request['dialog_token'],
+                    'tlvs': dbus.ByteArray(binascii.unhexlify(resp))}
             p2p.ServiceDiscoveryResponse(dbus.Dictionary(args, signature='sv'))
             self.loop.quit()
 
@@ -3451,7 +3453,7 @@ def _test_dbus_p2p_service_discovery_external(dev, apdev):
 
 def test_dbus_p2p_autogo(dev, apdev):
     """D-Bus P2P autonomous GO"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     addr0 = dev[0].p2p_dev_addr()
@@ -3553,16 +3555,12 @@ def test_dbus_p2p_autogo(dev, apdev):
             logger.debug("provisionDiscoveryRequestDisplayPin - peer=%s pin=%s" % (peer_object, pin))
             self.peer_path = peer_object
             peer = binascii.unhexlify(peer_object.split('/')[-1])
-            addr = ""
-            for p in peer:
-                if len(addr) > 0:
-                    addr += ':'
-                addr += '%02x' % ord(p)
-
-            params = { 'Role': 'registrar',
-                       'P2PDeviceAddress': self.peer['DeviceAddress'],
-                       'Bssid': self.peer['DeviceAddress'],
-                       'Type': 'pin' }
+            addr = ':'.join(["%02x" % i for i in struct.unpack('6B', peer)])
+
+            params = {'Role': 'registrar',
+                      'P2PDeviceAddress': self.peer['DeviceAddress'],
+                      'Bssid': self.peer['DeviceAddress'],
+                      'Type': 'pin'}
             wps = dbus.Interface(self.g_if_obj, WPAS_DBUS_IFACE_WPS)
             try:
                 wps.Start(params)
@@ -3572,10 +3570,10 @@ def test_dbus_p2p_autogo(dev, apdev):
                 if "InvalidArgs" not in str(e):
                     self.exceptions = True
                     raise Exception("Unexpected error message: " + str(e))
-            params = { 'Role': 'registrar',
-                       'P2PDeviceAddress': self.peer['DeviceAddress'],
-                       'Type': 'pin',
-                       'Pin': '12345670' }
+            params = {'Role': 'registrar',
+                      'P2PDeviceAddress': self.peer['DeviceAddress'],
+                      'Type': 'pin',
+                      'Pin': '12345670'}
             logger.info("Authorize peer to connect to the group")
             wps.Start(params)
 
@@ -3609,7 +3607,7 @@ def test_dbus_p2p_autogo(dev, apdev):
             # dictionary with 'WPSVendorExtensions' as the key surrounding these
             # values.. The current implementations maintains support for that
             # for backwards compability reasons. Verify that encoding first.
-            vals = dbus.Dictionary({ 'WPSVendorExtensions': [ ext ]},
+            vals = dbus.Dictionary({'WPSVendorExtensions': [ext]},
                                    signature='sv')
             g_obj.Set(WPAS_DBUS_GROUP, 'WPSVendorExtensions', vals,
                       dbus_interface=dbus.PROPERTIES_IFACE)
@@ -3650,7 +3648,7 @@ def test_dbus_p2p_autogo(dev, apdev):
                     self.exceptions = True
                     raise Exception("Unexpected error message for invalid Set(WPSVendorExtensions): " + str(e))
 
-            vals = dbus.Dictionary({ 'Foo': [ ext ]}, signature='sv')
+            vals = dbus.Dictionary({'Foo': [ext]}, signature='sv')
             try:
                 g_obj.Set(WPAS_DBUS_GROUP, 'WPSVendorExtensions', vals,
                           dbus_interface=dbus.PROPERTIES_IFACE)
@@ -3661,7 +3659,7 @@ def test_dbus_p2p_autogo(dev, apdev):
                     self.exceptions = True
                     raise Exception("Unexpected error message for invalid Set(WPSVendorExtensions): " + str(e))
 
-            vals = [ "foo" ]
+            vals = ["foo"]
             try:
                 g_obj.Set(WPAS_DBUS_GROUP, 'WPSVendorExtensions', vals,
                           dbus_interface=dbus.PROPERTIES_IFACE)
@@ -3672,7 +3670,7 @@ def test_dbus_p2p_autogo(dev, apdev):
                     self.exceptions = True
                     raise Exception("Unexpected error message for invalid Set(WPSVendorExtensions): " + str(e))
 
-            vals = [ [ "foo" ] ]
+            vals = [["foo"]]
             try:
                 g_obj.Set(WPAS_DBUS_GROUP, 'WPSVendorExtensions', vals,
                           dbus_interface=dbus.PROPERTIES_IFACE)
@@ -3683,7 +3681,7 @@ def test_dbus_p2p_autogo(dev, apdev):
                     self.exceptions = True
                     raise Exception("Unexpected error message for invalid Set(WPSVendorExtensions): " + str(e))
 
-            p2p.RemoveClient({ 'peer': self.peer_path })
+            p2p.RemoveClient({'peer': self.peer_path})
 
             self.waiting_end = True
             group_p2p = dbus.Interface(self.g_if_obj,
@@ -3713,7 +3711,7 @@ def test_dbus_p2p_autogo(dev, apdev):
 
 def test_dbus_p2p_autogo_pbc(dev, apdev):
     """D-Bus P2P autonomous GO and PBC"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     addr0 = dev[0].p2p_dev_addr()
@@ -3767,14 +3765,10 @@ def test_dbus_p2p_autogo_pbc(dev, apdev):
             logger.debug("provisionDiscoveryPBCRequest - peer=%s" % peer_object)
             self.peer_path = peer_object
             peer = binascii.unhexlify(peer_object.split('/')[-1])
-            addr = ""
-            for p in peer:
-                if len(addr) > 0:
-                    addr += ':'
-                addr += '%02x' % ord(p)
-            params = { 'Role': 'registrar',
-                       'P2PDeviceAddress': self.peer['DeviceAddress'],
-                       'Type': 'pbc' }
+            addr = ':'.join(["%02x" % i for i in struct.unpack('6B', peer)])
+            params = {'Role': 'registrar',
+                      'P2PDeviceAddress': self.peer['DeviceAddress'],
+                      'Type': 'pbc'}
             logger.info("Authorize peer to connect to the group")
             wps = dbus.Interface(self.g_if_obj, WPAS_DBUS_IFACE_WPS)
             wps.Start(params)
@@ -3803,7 +3797,7 @@ def test_dbus_p2p_autogo_pbc(dev, apdev):
 
 def test_dbus_p2p_autogo_legacy(dev, apdev):
     """D-Bus P2P autonomous GO and legacy STA"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     addr0 = dev[0].p2p_dev_addr()
@@ -3832,12 +3826,12 @@ def test_dbus_p2p_autogo_legacy(dev, apdev):
             res = g_obj.GetAll(WPAS_DBUS_GROUP,
                                dbus_interface=dbus.PROPERTIES_IFACE,
                                byte_arrays=True)
-            bssid = ':'.join([binascii.hexlify(l) for l in res['BSSID']])
+            bssid = ':'.join(["%02x" % i for i in struct.unpack('6B', res['BSSID'])])
 
             pin = '12345670'
-            params = { 'Role': 'enrollee',
-                       'Type': 'pin',
-                       'Pin': pin }
+            params = {'Role': 'enrollee',
+                      'Type': 'pin',
+                      'Pin': pin}
             g_if_obj = bus.get_object(WPAS_DBUS_SERVICE,
                                       properties['interface_object'])
             wps = dbus.Interface(g_if_obj, WPAS_DBUS_IFACE_WPS)
@@ -3873,7 +3867,7 @@ def test_dbus_p2p_autogo_legacy(dev, apdev):
 
 def test_dbus_p2p_join(dev, apdev):
     """D-Bus P2P join an autonomous GO"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     addr1 = dev[1].p2p_dev_addr()
@@ -3910,17 +3904,17 @@ def test_dbus_p2p_join(dev, apdev):
                                   dbus_interface=dbus.PROPERTIES_IFACE,
                                   byte_arrays=True)
             logger.debug('peer properties: ' + str(res))
-            if addr2.replace(':','') in path:
+            if addr2.replace(':', '') in path:
                 self.peer = path
-            elif addr1.replace(':','') in path:
+            elif addr1.replace(':', '') in path:
                 self.go = path
             if self.peer and self.go:
                 logger.info("Join the group")
                 p2p.StopFind()
-                args = { 'peer': self.go,
-                         'join': True,
-                         'wps_method': 'pin',
-                         'frequency': 2412 }
+                args = {'peer': self.go,
+                        'join': True,
+                        'wps_method': 'pin',
+                        'frequency': 2412}
                 pin = p2p.Connect(args)
 
                 dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1')
@@ -3962,11 +3956,11 @@ def test_dbus_p2p_join(dev, apdev):
                     raise Exception("Unexpected error message for invalid Set(WPSVendorExtensions): " + str(e))
 
             group_p2p = dbus.Interface(g_if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
-            args = { 'duration1': 30000, 'interval1': 102400,
-                     'duration2': 20000, 'interval2': 102400 }
+            args = {'duration1': 30000, 'interval1': 102400,
+                    'duration2': 20000, 'interval2': 102400}
             group_p2p.PresenceRequest(args)
 
-            args = { 'peer': self.peer }
+            args = {'peer': self.peer}
             group_p2p.Invite(args)
 
         def groupFinished(self, properties):
@@ -3998,7 +3992,7 @@ def test_dbus_p2p_join(dev, apdev):
 
 def test_dbus_p2p_invitation_received(dev, apdev):
     """D-Bus P2P and InvitationReceived"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     form(dev[0], dev[1])
@@ -4053,7 +4047,7 @@ def test_dbus_p2p_config(dev, apdev):
         dev[0].request("P2P_SET ssid_postfix ")
 
 def _test_dbus_p2p_config(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     res = if_obj.Get(WPAS_DBUS_IFACE_P2PDEVICE, "P2PDeviceConfig",
@@ -4071,9 +4065,9 @@ def _test_dbus_p2p_config(dev, apdev):
         if res[k] != res2[k]:
             raise Exception("Parameter %s value changes" % k)
 
-    changes = { 'SsidPostfix': 'foo',
-                'VendorExtension': [ dbus.ByteArray(b'\x11\x22\x33\x44') ],
-                'SecondaryDeviceTypes': [ dbus.ByteArray(b'\x11\x22\x33\x44\x55\x66\x77\x88') ]}
+    changes = {'SsidPostfix': 'foo',
+               'VendorExtension': [dbus.ByteArray(b'\x11\x22\x33\x44')],
+               'SecondaryDeviceTypes': [dbus.ByteArray(b'\x11\x22\x33\x44\x55\x66\x77\x88')]}
     if_obj.Set(WPAS_DBUS_IFACE_P2PDEVICE, "P2PDeviceConfig",
                dbus.Dictionary(changes, signature='sv'),
                dbus_interface=dbus.PROPERTIES_IFACE)
@@ -4087,9 +4081,9 @@ def _test_dbus_p2p_config(dev, apdev):
     if 'SecondaryDeviceTypes' not in res2 or len(res2['SecondaryDeviceTypes']) != 1:
         raise Exception("SecondaryDeviceType does not match")
 
-    changes = { 'SsidPostfix': '',
-                'VendorExtension': dbus.Array([], signature="ay"),
-                'SecondaryDeviceTypes': dbus.Array([], signature="ay") }
+    changes = {'SsidPostfix': '',
+               'VendorExtension': dbus.Array([], signature="ay"),
+               'SecondaryDeviceTypes': dbus.Array([], signature="ay")}
     if_obj.Set(WPAS_DBUS_IFACE_P2PDEVICE, "P2PDeviceConfig",
                dbus.Dictionary(changes, signature='sv'),
                dbus_interface=dbus.PROPERTIES_IFACE)
@@ -4117,7 +4111,7 @@ def _test_dbus_p2p_config(dev, apdev):
 
     try:
         dev[0].request("P2P_SET disabled 1")
-        changes = { 'SsidPostfix': 'foo' }
+        changes = {'SsidPostfix': 'foo'}
         if_obj.Set(WPAS_DBUS_IFACE_P2PDEVICE, "P2PDeviceConfig",
                    dbus.Dictionary(changes, signature='sv'),
                    dbus_interface=dbus.PROPERTIES_IFACE)
@@ -4128,9 +4122,9 @@ def _test_dbus_p2p_config(dev, apdev):
     finally:
         dev[0].request("P2P_SET disabled 0")
 
-    tests = [ { 'DeviceName': 123 },
-              { 'SsidPostfix': 123 },
-              { 'Foo': 'Bar' } ]
+    tests = [{'DeviceName': 123},
+             {'SsidPostfix': 123},
+             {'Foo': 'Bar'}]
     for changes in tests:
         try:
             if_obj.Set(WPAS_DBUS_IFACE_P2PDEVICE, "P2PDeviceConfig",
@@ -4143,7 +4137,7 @@ def _test_dbus_p2p_config(dev, apdev):
 
 def test_dbus_p2p_persistent(dev, apdev):
     """D-Bus P2P persistent group"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     class TestDbusP2p(TestDbus):
@@ -4198,7 +4192,7 @@ def test_dbus_p2p_persistent(dev, apdev):
     res = p_obj.Get(WPAS_DBUS_PERSISTENT_GROUP, "Properties",
                     dbus_interface=dbus.PROPERTIES_IFACE, byte_arrays=True)
     logger.info("Persistent group Properties: " + str(res))
-    vals = dbus.Dictionary({ 'ssid': 'DIRECT-foo' }, signature='sv')
+    vals = dbus.Dictionary({'ssid': 'DIRECT-foo'}, signature='sv')
     p_obj.Set(WPAS_DBUS_PERSISTENT_GROUP, "Properties", vals,
               dbus_interface=dbus.PROPERTIES_IFACE)
     res2 = p_obj.Get(WPAS_DBUS_PERSISTENT_GROUP, "Properties",
@@ -4211,8 +4205,8 @@ def test_dbus_p2p_persistent(dev, apdev):
     if res2['ssid'] != '"DIRECT-foo"':
         raise Exception("Unexpected ssid")
 
-    args = dbus.Dictionary({ 'ssid': 'DIRECT-testing',
-                             'psk': '1234567890' }, signature='sv')
+    args = dbus.Dictionary({'ssid': 'DIRECT-testing',
+                            'psk': '1234567890'}, signature='sv')
     group = p2p.AddPersistentGroup(args)
 
     groups = if_obj.Get(WPAS_DBUS_IFACE_P2PDEVICE, "PersistentGroups",
@@ -4236,7 +4230,7 @@ def test_dbus_p2p_persistent(dev, apdev):
 
 def test_dbus_p2p_reinvoke_persistent(dev, apdev):
     """D-Bus P2P reinvoke persistent group"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     addr0 = dev[0].p2p_dev_addr()
@@ -4279,7 +4273,7 @@ def test_dbus_p2p_reinvoke_persistent(dev, apdev):
                 res = g_obj.GetAll(WPAS_DBUS_GROUP,
                                    dbus_interface=dbus.PROPERTIES_IFACE,
                                    byte_arrays=True)
-                bssid = ':'.join([binascii.hexlify(l) for l in res['BSSID']])
+                bssid = ':'.join(["%02x" % i for i in struct.unpack('6B', res['BSSID'])])
                 dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1')
                 dev1.scan_for_bss(bssid, freq=2412)
                 dev1.global_request("P2P_CONNECT " + addr0 + " 12345670 join")
@@ -4294,8 +4288,8 @@ def test_dbus_p2p_reinvoke_persistent(dev, apdev):
                 dev1.global_request("SET persistent_reconnect 1")
                 dev1.p2p_listen()
 
-                args = { 'persistent_group_object': dbus.ObjectPath(path),
-                         'peer': self.peer_path }
+                args = {'persistent_group_object': dbus.ObjectPath(path),
+                        'peer': self.peer_path}
                 try:
                     pin = p2p.Invite(args)
                     raise Exception("Invalid Invite accepted")
@@ -4303,8 +4297,8 @@ def test_dbus_p2p_reinvoke_persistent(dev, apdev):
                     if "InvalidArgs" not in str(e):
                         raise Exception("Unexpected error message for invalid Invite: " + str(e))
 
-                args = { 'persistent_group_object': self.persistent,
-                         'peer': self.peer_path }
+                args = {'persistent_group_object': self.persistent,
+                        'peer': self.peer_path}
                 pin = p2p.Invite(args)
                 self.invited = True
 
@@ -4328,16 +4322,12 @@ def test_dbus_p2p_reinvoke_persistent(dev, apdev):
             logger.debug("provisionDiscoveryRequestDisplayPin - peer=%s pin=%s" % (peer_object, pin))
             self.peer_path = peer_object
             peer = binascii.unhexlify(peer_object.split('/')[-1])
-            addr = ""
-            for p in peer:
-                if len(addr) > 0:
-                    addr += ':'
-                addr += '%02x' % ord(p)
-            params = { 'Role': 'registrar',
-                       'P2PDeviceAddress': self.peer['DeviceAddress'],
-                       'Bssid': self.peer['DeviceAddress'],
-                       'Type': 'pin',
-                       'Pin': '12345670' }
+            addr = ':'.join(["%02x" % i for i in struct.unpack('6B', peer)])
+            params = {'Role': 'registrar',
+                      'P2PDeviceAddress': self.peer['DeviceAddress'],
+                      'Bssid': self.peer['DeviceAddress'],
+                      'Type': 'pin',
+                      'Pin': '12345670'}
             logger.info("Authorize peer to connect to the group")
             dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1')
             wps = dbus.Interface(self.g_if_obj, WPAS_DBUS_IFACE_WPS)
@@ -4375,7 +4365,7 @@ def test_dbus_p2p_reinvoke_persistent(dev, apdev):
 
 def test_dbus_p2p_go_neg_rx(dev, apdev):
     """D-Bus P2P GO Negotiation receive"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
     addr0 = dev[0].p2p_dev_addr()
 
@@ -4411,8 +4401,8 @@ def test_dbus_p2p_go_neg_rx(dev, apdev):
             logger.debug("goNegotiationRequest: path=%s dev_passwd_id=%d go_intent=%d" % (path, dev_passwd_id, go_intent))
             if dev_passwd_id != 1:
                 raise Exception("Unexpected dev_passwd_id=%d" % dev_passwd_id)
-            args = { 'peer': path, 'wps_method': 'display', 'pin': '12345670',
-                     'go_intent': 15, 'persistent': False, 'frequency': 5175 }
+            args = {'peer': path, 'wps_method': 'display', 'pin': '12345670',
+                    'go_intent': 15, 'persistent': False, 'frequency': 5175}
             try:
                 p2p.Connect(args)
                 raise Exception("Invalid Connect accepted")
@@ -4420,8 +4410,8 @@ def test_dbus_p2p_go_neg_rx(dev, apdev):
                 if "ConnectChannelUnsupported" not in str(e):
                     raise Exception("Unexpected error message for invalid Connect: " + str(e))
 
-            args = { 'peer': path, 'wps_method': 'display', 'pin': '12345670',
-                     'go_intent': 15, 'persistent': False }
+            args = {'peer': path, 'wps_method': 'display', 'pin': '12345670',
+                    'go_intent': 15, 'persistent': False}
             p2p.Connect(args)
 
         def goNegotiationSuccess(self, properties):
@@ -4457,7 +4447,7 @@ def test_dbus_p2p_go_neg_rx(dev, apdev):
 
 def test_dbus_p2p_go_neg_auth(dev, apdev):
     """D-Bus P2P GO Negotiation authorized"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
     addr0 = dev[0].p2p_dev_addr()
     dev[1].p2p_listen()
@@ -4493,8 +4483,8 @@ def test_dbus_p2p_go_neg_auth(dev, apdev):
 
         def deviceFound(self, path):
             logger.debug("deviceFound: path=%s" % path)
-            args = { 'peer': path, 'wps_method': 'keypad',
-                     'go_intent': 15, 'authorize_only': True }
+            args = {'peer': path, 'wps_method': 'keypad',
+                    'go_intent': 15, 'authorize_only': True}
             try:
                 p2p.Connect(args)
                 raise Exception("Invalid Connect accepted")
@@ -4502,8 +4492,8 @@ def test_dbus_p2p_go_neg_auth(dev, apdev):
                 if "InvalidArgs" not in str(e):
                     raise Exception("Unexpected error message for invalid Connect: " + str(e))
 
-            args = { 'peer': path, 'wps_method': 'keypad', 'pin': '12345670',
-                     'go_intent': 15, 'authorize_only': True }
+            args = {'peer': path, 'wps_method': 'keypad', 'pin': '12345670',
+                    'go_intent': 15, 'authorize_only': True}
             p2p.Connect(args)
             p2p.Listen(10)
             dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1')
@@ -4558,7 +4548,7 @@ def test_dbus_p2p_go_neg_auth(dev, apdev):
 
 def test_dbus_p2p_go_neg_init(dev, apdev):
     """D-Bus P2P GO Negotiation initiation"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
     addr0 = dev[0].p2p_dev_addr()
     dev[1].p2p_listen()
@@ -4591,8 +4581,8 @@ def test_dbus_p2p_go_neg_init(dev, apdev):
         def deviceFound(self, path):
             logger.debug("deviceFound: path=%s" % path)
             dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1')
-            args = { 'peer': path, 'wps_method': 'keypad', 'pin': '12345670',
-                     'go_intent': 0 }
+            args = {'peer': path, 'wps_method': 'keypad', 'pin': '12345670',
+                    'go_intent': 0}
             p2p.Connect(args)
 
             ev = dev1.wait_global_event(["P2P-GO-NEG-REQUEST"], timeout=15)
@@ -4603,6 +4593,9 @@ def test_dbus_p2p_go_neg_init(dev, apdev):
             if ev is None:
                 raise Exception("Group formation timed out")
             self.sta_group_ev = ev
+            dev1.close_monitor_global()
+            dev1.close_monitor_mon()
+            dev1 = None
 
         def goNegotiationSuccess(self, properties):
             logger.debug("goNegotiationSuccess: properties=%s" % str(properties))
@@ -4613,9 +4606,10 @@ def test_dbus_p2p_go_neg_init(dev, apdev):
                                       properties['interface_object'])
             group_p2p = dbus.Interface(g_if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
             group_p2p.Disconnect()
-            dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1')
+            dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1', monitor=False)
             dev1.group_form_result(self.sta_group_ev)
             dev1.remove_group()
+            dev1 = None
 
         def groupFinished(self, properties):
             logger.debug("groupFinished: " + str(properties))
@@ -4653,7 +4647,7 @@ def test_dbus_p2p_go_neg_init(dev, apdev):
 
 def test_dbus_p2p_group_termination_by_go(dev, apdev):
     """D-Bus P2P group removal on GO terminating the group"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
     addr0 = dev[0].p2p_dev_addr()
     dev[1].p2p_listen()
@@ -4686,8 +4680,8 @@ def test_dbus_p2p_group_termination_by_go(dev, apdev):
         def deviceFound(self, path):
             logger.debug("deviceFound: path=%s" % path)
             dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1')
-            args = { 'peer': path, 'wps_method': 'keypad', 'pin': '12345670',
-                     'go_intent': 0 }
+            args = {'peer': path, 'wps_method': 'keypad', 'pin': '12345670',
+                    'go_intent': 0}
             p2p.Connect(args)
 
             ev = dev1.wait_global_event(["P2P-GO-NEG-REQUEST"], timeout=15)
@@ -4698,6 +4692,9 @@ def test_dbus_p2p_group_termination_by_go(dev, apdev):
             if ev is None:
                 raise Exception("Group formation timed out")
             self.sta_group_ev = ev
+            dev1.close_monitor_global()
+            dev1.close_monitor_mon()
+            dev1 = None
 
         def goNegotiationSuccess(self, properties):
             logger.debug("goNegotiationSuccess: properties=%s" % str(properties))
@@ -4706,7 +4703,7 @@ def test_dbus_p2p_group_termination_by_go(dev, apdev):
             logger.debug("groupStarted: " + str(properties))
             g_if_obj = bus.get_object(WPAS_DBUS_SERVICE,
                                       properties['interface_object'])
-            dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1')
+            dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1', monitor=False)
             dev1.group_form_result(self.sta_group_ev)
             dev1.remove_group()
 
@@ -4748,7 +4745,7 @@ def test_dbus_p2p_group_idle_timeout(dev, apdev):
         dev[0].global_request("SET p2p_group_idle 0")
 
 def _test_dbus_p2p_group_idle_timeout(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
     addr0 = dev[0].p2p_dev_addr()
     dev[1].p2p_listen()
@@ -4782,8 +4779,8 @@ def _test_dbus_p2p_group_idle_timeout(dev, apdev):
         def deviceFound(self, path):
             logger.debug("deviceFound: path=%s" % path)
             dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1')
-            args = { 'peer': path, 'wps_method': 'keypad', 'pin': '12345670',
-                     'go_intent': 0 }
+            args = {'peer': path, 'wps_method': 'keypad', 'pin': '12345670',
+                    'go_intent': 0}
             p2p.Connect(args)
 
             ev = dev1.wait_global_event(["P2P-GO-NEG-REQUEST"], timeout=15)
@@ -4794,6 +4791,9 @@ def _test_dbus_p2p_group_idle_timeout(dev, apdev):
             if ev is None:
                 raise Exception("Group formation timed out")
             self.sta_group_ev = ev
+            dev1.close_monitor_global()
+            dev1.close_monitor_mon()
+            dev1 = None
 
         def goNegotiationSuccess(self, properties):
             logger.debug("goNegotiationSuccess: properties=%s" % str(properties))
@@ -4803,7 +4803,7 @@ def _test_dbus_p2p_group_idle_timeout(dev, apdev):
             self.group_started = True
             g_if_obj = bus.get_object(WPAS_DBUS_SERVICE,
                                       properties['interface_object'])
-            dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1')
+            dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1', monitor=False)
             dev1.group_form_result(self.sta_group_ev)
             ifaddr = dev1.group_request("STA-FIRST").splitlines()[0]
             # Force disassociation with different reason code so that the
@@ -4845,7 +4845,7 @@ def _test_dbus_p2p_group_idle_timeout(dev, apdev):
 
 def test_dbus_p2p_wps_failure(dev, apdev):
     """D-Bus P2P WPS failure"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
     addr0 = dev[0].p2p_dev_addr()
 
@@ -4880,8 +4880,8 @@ def test_dbus_p2p_wps_failure(dev, apdev):
             logger.debug("goNegotiationRequest: path=%s dev_passwd_id=%d go_intent=%d" % (path, dev_passwd_id, go_intent))
             if dev_passwd_id != 1:
                 raise Exception("Unexpected dev_passwd_id=%d" % dev_passwd_id)
-            args = { 'peer': path, 'wps_method': 'display', 'pin': '12345670',
-                     'go_intent': 15 }
+            args = {'peer': path, 'wps_method': 'display', 'pin': '12345670',
+                    'go_intent': 15}
             p2p.Connect(args)
 
         def goNegotiationSuccess(self, properties):
@@ -4921,7 +4921,7 @@ def test_dbus_p2p_wps_failure(dev, apdev):
 
 def test_dbus_p2p_two_groups(dev, apdev):
     """D-Bus P2P with two concurrent groups"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
     dev[0].request("SET p2p_no_group_iface 0")
@@ -4963,9 +4963,9 @@ def test_dbus_p2p_two_groups(dev, apdev):
 
         def deviceFound(self, path):
             logger.debug("deviceFound: path=%s" % path)
-            if addr2.replace(':','') in path:
+            if addr2.replace(':', '') in path:
                 self.peer = path
-            elif addr1.replace(':','') in path:
+            elif addr1.replace(':', '') in path:
                 self.go = path
             if self.go and not self.group1:
                 logger.info("Join the group")
@@ -4974,11 +4974,11 @@ def test_dbus_p2p_two_groups(dev, apdev):
                 dev1 = WpaSupplicant('wlan1', '/tmp/wpas-wlan1')
                 dev1.group_ifname = dev1_group_ifname
                 dev1.group_request("WPS_PIN any " + pin)
-                args = { 'peer': self.go,
-                         'join': True,
-                         'wps_method': 'pin',
-                         'pin': pin,
-                         'frequency': 2412 }
+                args = {'peer': self.go,
+                        'join': True,
+                        'wps_method': 'pin',
+                        'pin': pin,
+                        'frequency': 2412}
                 p2p.Connect(args)
 
         def groupStarted(self, properties):
@@ -5001,7 +5001,7 @@ def test_dbus_p2p_two_groups(dev, apdev):
                                                 self.group1iface)
 
                 logger.info("Start autonomous GO")
-                params = dbus.Dictionary({ 'frequency': 2412 })
+                params = dbus.Dictionary({'frequency': 2412})
                 p2p.GroupAdd(params)
             elif not self.group2:
                 self.group2 = properties['group_object']
@@ -5012,16 +5012,16 @@ def test_dbus_p2p_two_groups(dev, apdev):
 
             if self.group1 and self.group2:
                 logger.info("Authorize peer to join the group")
-                a2 = binascii.unhexlify(addr2.replace(':',''))
-                params = { 'Role': 'enrollee',
-                           'P2PDeviceAddress': dbus.ByteArray(a2),
-                           'Bssid': dbus.ByteArray(a2),
-                           'Type': 'pin',
-                           'Pin': '12345670' }
+                a2 = binascii.unhexlify(addr2.replace(':', ''))
+                params = {'Role': 'enrollee',
+                          'P2PDeviceAddress': dbus.ByteArray(a2),
+                          'Bssid': dbus.ByteArray(a2),
+                          'Type': 'pin',
+                          'Pin': '12345670'}
                 g_wps = dbus.Interface(self.g2_if_obj, WPAS_DBUS_IFACE_WPS)
                 g_wps.Start(params)
 
-                bssid = ':'.join([binascii.hexlify(l) for l in self.g2_bssid])
+                bssid = ':'.join(["%02x" % i for i in struct.unpack('6B', self.g2_bssid)])
                 dev2 = WpaSupplicant('wlan2', '/tmp/wpas-wlan2')
                 dev2.scan_for_bss(bssid, freq=2412)
                 dev2.global_request("P2P_CONNECT " + bssid + " 12345670 join freq=2412")
@@ -5112,7 +5112,7 @@ def test_dbus_p2p_two_groups(dev, apdev):
 
 def test_dbus_p2p_cancel(dev, apdev):
     """D-Bus P2P Cancel"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
     try:
         p2p.Cancel()
@@ -5138,8 +5138,8 @@ def test_dbus_p2p_cancel(dev, apdev):
 
         def deviceFound(self, path):
             logger.debug("deviceFound: path=%s" % path)
-            args = { 'peer': path, 'wps_method': 'keypad', 'pin': '12345670',
-                     'go_intent': 0 }
+            args = {'peer': path, 'wps_method': 'keypad', 'pin': '12345670',
+                    'go_intent': 0}
             p2p.Connect(args)
             p2p.Cancel()
             self.done = True
@@ -5159,13 +5159,13 @@ def test_dbus_p2p_cancel(dev, apdev):
 
 def test_dbus_p2p_ip_addr(dev, apdev):
     """D-Bus P2P and IP address parameters"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     p2p = dbus.Interface(if_obj, WPAS_DBUS_IFACE_P2PDEVICE)
 
-    vals = [ ("IpAddrGo", "192.168.43.1"),
-             ("IpAddrMask", "255.255.255.0"),
-             ("IpAddrStart", "192.168.43.100"),
-             ("IpAddrEnd", "192.168.43.199") ]
+    vals = [("IpAddrGo", "192.168.43.1"),
+            ("IpAddrMask", "255.255.255.0"),
+            ("IpAddrStart", "192.168.43.100"),
+            ("IpAddrEnd", "192.168.43.199")]
     for field, value in vals:
         if_obj.Set(WPAS_DBUS_IFACE, field, value,
                    dbus_interface=dbus.PROPERTIES_IFACE)
@@ -5231,7 +5231,7 @@ def test_dbus_p2p_ip_addr(dev, apdev):
 
 def test_dbus_introspect(dev, apdev):
     """D-Bus introspection"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
 
     res = if_obj.Introspect(WPAS_DBUS_IFACE,
                             dbus_interface=dbus.INTROSPECTABLE_IFACE)
@@ -5277,7 +5277,7 @@ def test_dbus_introspect(dev, apdev):
 
 def run_busctl(service, obj):
     logger.info("busctl introspect %s %s" % (service, obj))
-    cmd = subprocess.Popen([ 'busctl', 'introspect', service, obj ],
+    cmd = subprocess.Popen(['busctl', 'introspect', service, obj],
                            stdout=subprocess.PIPE,
                            stderr=subprocess.PIPE)
     out = cmd.communicate()
@@ -5290,13 +5290,13 @@ def run_busctl(service, obj):
 
 def test_dbus_introspect_busctl(dev, apdev):
     """D-Bus introspection with busctl"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     ifaces = dbus_get(dbus, wpas_obj, "Interfaces")
     run_busctl(WPAS_DBUS_SERVICE, WPAS_DBUS_PATH)
     run_busctl(WPAS_DBUS_SERVICE, WPAS_DBUS_PATH + "/Interfaces")
     run_busctl(WPAS_DBUS_SERVICE, ifaces[0])
 
-    hapd = hostapd.add_ap(apdev[0], { "ssid": "open" })
+    hapd = hostapd.add_ap(apdev[0], {"ssid": "open"})
     bssid = apdev[0]['bssid']
     dev[0].scan_for_bss(bssid, freq=2412)
     id = dev[0].add_network()
@@ -5308,7 +5308,7 @@ def test_dbus_introspect_busctl(dev, apdev):
 
 def test_dbus_ap(dev, apdev):
     """D-Bus AddNetwork for AP mode"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     ssid = "test-wpa2-psk"
@@ -5392,12 +5392,12 @@ def test_dbus_ap(dev, apdev):
 
         def run_connect(self, *args):
             logger.debug("run_connect")
-            args = dbus.Dictionary({ 'ssid': ssid,
-                                     'key_mgmt': 'WPA-PSK',
-                                     'psk': passphrase,
-                                     'mode': 2,
-                                     'frequency': 2412,
-                                     'scan_freq': 2412 },
+            args = dbus.Dictionary({'ssid': ssid,
+                                    'key_mgmt': 'WPA-PSK',
+                                    'psk': passphrase,
+                                    'mode': 2,
+                                    'frequency': 2412,
+                                    'scan_freq': 2412},
                                    signature='sv')
             self.netw = iface.AddNetwork(args)
             iface.SelectNetwork(self.netw)
@@ -5413,7 +5413,7 @@ def test_dbus_ap(dev, apdev):
 
 def test_dbus_connect_wpa_eap(dev, apdev):
     """D-Bus AddNetwork and connection with WPA+WPA2-Enterprise AP"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     ssid = "test-wpa-eap"
@@ -5447,14 +5447,14 @@ def test_dbus_connect_wpa_eap(dev, apdev):
 
         def run_connect(self, *args):
             logger.debug("run_connect")
-            args = dbus.Dictionary({ 'ssid': ssid,
-                                     'key_mgmt': 'WPA-EAP',
-                                     'eap': 'PEAP',
-                                     'identity': 'user',
-                                     'password': 'password',
-                                     'ca_cert': 'auth_serv/ca.pem',
-                                     'phase2': 'auth=MSCHAPV2',
-                                     'scan_freq': 2412 },
+            args = dbus.Dictionary({'ssid': ssid,
+                                    'key_mgmt': 'WPA-EAP',
+                                    'eap': 'PEAP',
+                                    'identity': 'user',
+                                    'password': 'password',
+                                    'ca_cert': 'auth_serv/ca.pem',
+                                    'phase2': 'auth=MSCHAPV2',
+                                    'scan_freq': 2412},
                                    signature='sv')
             self.netw = iface.AddNetwork(args)
             iface.SelectNetwork(self.netw)
@@ -5475,7 +5475,7 @@ def test_dbus_ap_scan_2_ap_mode_scan(dev, apdev):
         dev[0].request("AP_SCAN 1")
 
 def _test_dbus_ap_scan_2_ap_mode_scan(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     if "OK" not in dev[0].request("AP_SCAN 2"):
@@ -5520,10 +5520,10 @@ def _test_dbus_ap_scan_2_ap_mode_scan(dev, apdev):
 
 def test_dbus_expectdisconnect(dev, apdev):
     """D-Bus ExpectDisconnect"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     wpas = dbus.Interface(wpas_obj, WPAS_DBUS_SERVICE)
 
-    params = { "ssid": "test-open" }
+    params = {"ssid": "test-open"}
     hapd = hostapd.add_ap(apdev[0], params)
     dev[0].connect("test-open", key_mgmt="NONE", scan_freq="2412")
 
@@ -5535,7 +5535,7 @@ def test_dbus_expectdisconnect(dev, apdev):
 
 def test_dbus_save_config(dev, apdev):
     """D-Bus SaveConfig"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
     try:
         iface.SaveConfig()
@@ -5552,7 +5552,7 @@ def test_dbus_vendor_elem(dev, apdev):
         dev[0].request("VENDOR_ELEM_REMOVE 1 *")
 
 def _test_dbus_vendor_elem(dev, apdev):
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     dev[0].request("VENDOR_ELEM_REMOVE 1 *")
@@ -5657,12 +5657,12 @@ def _test_dbus_vendor_elem(dev, apdev):
 
 def test_dbus_assoc_reject(dev, apdev):
     """D-Bus AssocStatusCode"""
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     iface = dbus.Interface(if_obj, WPAS_DBUS_IFACE)
 
     ssid = "test-open"
-    params = { "ssid": ssid,
-               "max_listen_interval": "1" }
+    params = {"ssid": ssid,
+              "max_listen_interval": "1"}
     hapd = hostapd.add_ap(apdev[0], params)
 
     class TestDbusConnect(TestDbus):
@@ -5691,9 +5691,9 @@ def test_dbus_assoc_reject(dev, apdev):
                 self.loop.quit()
 
         def run_connect(self, *args):
-            args = dbus.Dictionary({ 'ssid': ssid,
-                                     'key_mgmt': 'NONE',
-                                     'scan_freq': 2412 },
+            args = dbus.Dictionary({'ssid': ssid,
+                                    'key_mgmt': 'NONE',
+                                    'scan_freq': 2412},
                                    signature='sv')
             self.netw = iface.AddNetwork(args)
             iface.SelectNetwork(self.netw)
@@ -5709,7 +5709,7 @@ def test_dbus_assoc_reject(dev, apdev):
 def test_dbus_mesh(dev, apdev):
     """D-Bus mesh"""
     check_mesh_support(dev[0])
-    (bus,wpas_obj,path,if_obj) = prepare_dbus(dev[0])
+    (bus, wpas_obj, path, if_obj) = prepare_dbus(dev[0])
     mesh = dbus.Interface(if_obj, WPAS_DBUS_IFACE_MESH)
 
     add_open_mesh_network(dev[1])