From cb6289b622766f488d422b102e7bd9c410154372 Mon Sep 17 00:00:00 2001 From: Andrew Zaborowski Date: Mon, 6 Jun 2022 11:01:54 +0200 Subject: [PATCH] auto-t: Work around wpa_s config issue in testP2P Since commit 922fa099721903b106a7bc1ccd1ffe8c4a7bce69 in hostap, our setting of config_methods on P2P-client interface was ignored. Work around that commit, in addition to the previous workaround we have in this test, to again ensure the correct config_methods value is used. --- autotests/testP2P/connection_test.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/autotests/testP2P/connection_test.py b/autotests/testP2P/connection_test.py index 383e0cef..479076e8 100644 --- a/autotests/testP2P/connection_test.py +++ b/autotests/testP2P/connection_test.py @@ -87,7 +87,12 @@ class Test(unittest.TestCase): # For some reason wpa_supplicant's newly created P2P-client interface doesn't inherit # the settings from the main interface which were loaded from the config file # (P2P-device and P2P-GO interfaces do), we need to set config_methods again. + # + # Additionally since hostap commit 922fa099721903b106a7bc1ccd1ffe8c4a7bce69, + # just setting config_methods to the same value we used last time is detected as + # a NOOP so we need to set it to a different value first for it to take effect. peer_ifname = 'p2p-' + wpas.interface.name + '-0' + wpas.set('config_methods', '', ifname=peer_ifname) wpas.set('config_methods', wpas.config['config_methods'], ifname=peer_ifname) wpas.set('device_name', wpas.config['device_name'], ifname=peer_ifname) wpas.set('device_type', wpas.config['device_type'], ifname=peer_ifname)