cwinfo/yggdrasil-go
5
0
Fork 0
mirror of https://github.com/cwinfo/yggdrasil-go.git synced 2024-12-23 04:05:39 +00:00
Code Releases Activity

Don't panic when connect returns nil (fixes #1086) (#1089)

Browse Source 
* Don't panic when connect returns `nil` (fixes #1086)

It isn't clear to me why this would happen but let's guard the condition anyway.

* Log inconsistent error state

---------

Co-authored-by: Neil Alexander <neilalexander@users.noreply.github.com>
This commit is contained in:
Neil 2023-11-04 18:42:42 +00:00 committed by GitHub
parent 93a5adfd18
commit 0346af46da
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/core/link.go
View File

@ -273,7 +273,10 @@ func (l *links) add(u *url.URL, sintf string, linkType linkType) error {
} }
conn, err := l.connect(state.ctx, u, info, options) conn, err := l.connect(state.ctx, u, info, options)
if err != nil { if err != nil || conn == nil {
if err == nil && conn == nil {
l.core.log.Warnf("Link %q reached inconsistent error state", u.String())
}
if linkType == linkTypePersistent { if linkType == linkTypePersistent {
// If the link is a persistent configured peering, // If the link is a persistent configured peering,
// store information about the connection error so // store information about the connection error so