mirror of
https://github.com/tailscale/tailscale.git
synced 2024-11-29 04:55:31 +00:00
1c238cdce6
The code was using a C "int", which is a signed 32-bit integer.
That means some valid IP addresses were negative numbers.
(In particular, the default router address handed out by AT&T
fiber: 192.168.1.254. No I don't know why they do that.)
A negative number is < 255, and so was treated by the Go code
as an error.
This fixes the unit test failure:
$ go test -v -run=TestLikelyHomeRouterIPSyscallExec ./net/interfaces
=== RUN TestLikelyHomeRouterIPSyscallExec
interfaces_darwin_cgo_test.go:15: syscall() = invalid IP, false, netstat = 192.168.1.254, true
--- FAIL: TestLikelyHomeRouterIPSyscallExec (0.00s)
Signed-off-by: David Crawshaw <crawshaw@tailscale.com>
(cherry picked from commit
|
||
---|---|---|
.. | ||
interfaces_darwin_cgo_test.go | ||
interfaces_darwin_cgo.go | ||
interfaces_darwin_nocgo.go | ||
interfaces_darwin.go | ||
interfaces_defaultrouteif_todo.go | ||
interfaces_linux_test.go | ||
interfaces_linux.go | ||
interfaces_test.go | ||
interfaces_windows_test.go | ||
interfaces_windows.go | ||
interfaces.go | ||
route.h |