btcutil: rename interfaceAddrs to InterfaceAddrs in appengine variant
What changed, and why it matters
This is a straightforward build-fix rename. A previous commit exported a helper function in one file but forgot to rename the matching no-op version used for appengine builds. This patch simply renames the function so appengine-tagged builds compile again. There is no security issue.
No security action required. Treat as normal build-fix maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
Commit 73af4d6 renames interfaceAddrs to InterfaceAddrs in btcutil/net_noop.go (build tag appengine). The prior commit exported the function in btcutil/net.go but left the appengine no-op variant unexported, causing certgen.go:79 to fail with undefined: InterfaceAddrs under -tags appengine. The diff is a pure identifier rename with no logic change.
Changed components
btcutil/net_noop.goInspect captured patch +2 / −2
diff --git a/btcutil/net_noop.go b/btcutil/net_noop.go
index ae9c1f5..9afc3a0 100644
--- a/btcutil/net_noop.go
+++ b/btcutil/net_noop.go
@@ -11,10 +11,10 @@ import (
"net"
)
-// interfaceAddrs returns a list of the system's network interface addresses.
+// InterfaceAddrs returns a list of the system's network interface addresses.
// It is wrapped here so that we can substitute it for a no-op function that
// returns an empty slice of net.Addr when building for systems that do not
// allow access to net.InterfaceAddrs().
-func interfaceAddrs() ([]net.Addr, error) {
+func InterfaceAddrs() ([]net.Addr, error) {
return []net.Addr{}, nil
}
Why this scored 17/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.