DGNum/hostapd
6
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

tests: Explicitly set address to bind to

Browse source 
pyrad internally uses socket.getaddrinfo. While it is documented that
one can pass an empty string to bind to all interfaces. The underlying
function expects None instead though, breaking the call.

Either way, it does not hurt to set the bind address to 127.0.0.1
explicitly, so do so to avoid issues.

Signed-off-by: Benjamin Berg <benjamin.berg@intel.com>
This commit is contained in:
Benjamin Berg 2023-12-25 12:20:59 +02:00 committed by Jouni Malinen
parent 77f0b40872
commit 7a47756373
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
tests/hwsim/test_eap_proto.py
View file

@ -160,7 +160,7 @@ def start_radius_server(eap_handler):
srv.hosts["127.0.0.1"] = pyrad.server.RemoteHost("127.0.0.1",
b"radius",
"localhost")
srv.BindToAddress("")
srv.BindToAddress("127.0.0.1")
t_stop = threading.Event()
t = threading.Thread(target=run_pyrad_server, args=(srv, t_stop, eap_handler))
t.start()

6
tests/hwsim/test_radius.py
View file

@ -1132,7 +1132,7 @@ def test_radius_protocol(dev, apdev):
srv.hosts["127.0.0.1"] = pyrad.server.RemoteHost("127.0.0.1",
b"radius",
"localhost")
srv.BindToAddress("")
srv.BindToAddress("127.0.0.1")
t_events = {}
t_events['stop'] = threading.Event()
t_events['msg_auth'] = threading.Event()
@ -1249,7 +1249,7 @@ def start_radius_psk_server(psk, invalid_code=False, acct_interim_interval=0,
srv.hosts["127.0.0.1"] = pyrad.server.RemoteHost("127.0.0.1",
b"radius",
"localhost")
srv.BindToAddress("")
srv.BindToAddress("127.0.0.1")
t_events = {}
t_events['stop'] = threading.Event()
t_events['psk'] = psk
@ -1595,7 +1595,7 @@ def test_ap_vlan_wpa2_psk_radius_required(dev, apdev):
srv.hosts["127.0.0.1"] = pyrad.server.RemoteHost("127.0.0.1",
b"radius",
"localhost")
srv.BindToAddress("")
srv.BindToAddress("127.0.0.1")
t_events = {}
t_events['stop'] = threading.Event()
t_events['long'] = threading.Event()