Skip to content

Comments

Fix panic when interface not set in Solaris netstat -rn output#42

Merged
jackpal merged 1 commit intojackpal:masterfrom
sot-tech:master
Mar 6, 2024
Merged

Fix panic when interface not set in Solaris netstat -rn output#42
jackpal merged 1 commit intojackpal:masterfrom
sot-tech:master

Conversation

@sot-tech
Copy link

@sot-tech sot-tech commented Mar 6, 2024

Interface name for default route may be omitted (i.e. if set in /etc/defaultrouter)

github.com/jackpal/gateway.parseNetstatToRouteStruct({0xc001226000, 0x3e9, 0xc0008f2000?})
	github.com/jackpal/[email protected]/gateway_parsers.go:394 +0x605
github.com/jackpal/gateway.parseUnixGatewayIP({0xc001226000?, 0x6c2e313234373532?, 0x6264?})
	github.com/jackpal/[email protected]/gateway_parsers.go:355 +0x1c
github.com/jackpal/gateway.discoverGatewayOSSpecific()
	github.com/jackpal/[email protected]/gateway_solaris.go:22 +0x2a
github.com/jackpal/gateway.DiscoverGateway(...)
	github.com/jackpal/[email protected]/gateway.go:46

See: syncthing/syncthing#9453

Interface name for default route may be omitted (i.e. if set in /etc/defaultrouter)
@jackpal jackpal merged commit 99e9148 into jackpal:master Mar 6, 2024
@jackpal
Copy link
Owner

jackpal commented Mar 6, 2024

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants