mirrors
/
tailscale
mirror of https://github.com/tailscale/tailscale.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

net/interfaces: don't try to fork on iOS in likelyHomeRouterIPDarwin 

No subprocesses allowed on iOS. Will need to do this differently later.
This commit is contained in:
Brad Fitzpatrick 2020-07-31 10:35:15 -07:00
parent 8c850947db
commit c253d4f948
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
net/interfaces/interfaces_darwin.go
View File

@ -10,6 +10,7 @@ import (
"go4.org/mem" "go4.org/mem"
"inet.af/netaddr" "inet.af/netaddr"
"tailscale.com/util/lineread" "tailscale.com/util/lineread"
"tailscale.com/version"
) )
func init() { func init() {
@ -32,6 +33,13 @@ default link#14 UCSI utun2
*/ */
func likelyHomeRouterIPDarwin() (ret netaddr.IP, ok bool) { func likelyHomeRouterIPDarwin() (ret netaddr.IP, ok bool) {
if version.IsMobile() {
// Don't try to do subprocesses on iOS. Ends up with log spam like:
// kernel: "Sandbox: IPNExtension(86580) deny(1) process-fork"
// TODO(bradfitz): let our iOS app register a func with this package
// and have it call into C/Swift to get the routing table.
return ret, false
}
cmd := exec.Command("/usr/sbin/netstat", "-r", "-n", "-f", "inet") cmd := exec.Command("/usr/sbin/netstat", "-r", "-n", "-f", "inet")
stdout, err := cmd.StdoutPipe() stdout, err := cmd.StdoutPipe()
if err != nil { if err != nil {