qa: Drop OpenBSD from supported platforms in `get_bind_addrs` function
What the developer wrote
Authored by Hennadii Stepanov
qa: Drop OpenBSD from supported platforms in `get_bind_addrs` function
What changed, and why it matters
This is a small test-only change. A helper function used by Bitcoin Core's automated tests is being updated to no longer claim OpenBSD as a supported platform, because OpenBSD does not include the 'lsof' tool that the function relies on. It does not change the Bitcoin Core software that users run, nor does it fix or introduce a security vulnerability.
No security action needed. Treat as normal test-maintenance commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies test/functional/test_framework/netutil.py’s get_bind_addrs(pid). That function detects which addresses a process is listening on, using different platform-specific implementations. The BSD branch uses lsof via subprocess. OpenBSD is removed from the supported-platform tuple because it does not ship lsof. This is a test-framework compatibility/documentation adjustment, not a runtime code change.
Changed components
test/functional/test_framework/netutil.pyget_bind_addrs helper functionInspect captured patch +2 / −1
diff --git a/test/functional/test_framework/netutil.py b/test/functional/test_framework/netutil.py
index 85209322..034d0ff3 100644
--- a/test/functional/test_framework/netutil.py
+++ b/test/functional/test_framework/netutil.py
@@ -95,7 +95,8 @@ def get_bind_addrs(pid):
if conn[3] == STATE_LISTEN and conn[4] in inodes:
bind_addrs.append(conn[1])
return bind_addrs
- elif sys.platform.startswith(("darwin", "freebsd", "netbsd", "openbsd")):
+ # OpenBSD is not included, as it does not ship the lsof utility.
+ elif sys.platform.startswith(("darwin", "freebsd", "netbsd")):
import re
import subprocess
output = subprocess.check_output(["lsof",
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.