akovalenko/lnd.xprv
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

server: avoid nil pointer deference within OutboundPeerConnected

Browse Source
This commit is contained in:
Olaoluwa Osuntokun 2017-08-15 17:49:10 -07:00
parent 06ff0a0192
commit c4e43dbaa2
No known key found for this signature in database
GPG Key ID: 9CC5B105D03521A2
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
server.go
View File

@ -960,7 +960,9 @@ func (s *server) OutboundPeerConnected(connReq *connmgr.ConnReq, conn net.Conn)
// entry for this peer from the map. // entry for this peer from the map.
if connReqs, ok := s.persistentConnReqs[pubStr]; ok { if connReqs, ok := s.persistentConnReqs[pubStr]; ok {
for _, pConnReq := range connReqs { for _, pConnReq := range connReqs {
if pConnReq.ID() != connReq.ID() { if connReq != nil &&
pConnReq.ID() != connReq.ID() {
s.connMgr.Remove(pConnReq.ID()) s.connMgr.Remove(pConnReq.ID())
} }
} }
@ -977,7 +979,11 @@ func (s *server) OutboundPeerConnected(connReq *connmgr.ConnReq, conn net.Conn)
srvrLog.Warnf("Established outbound connection to "+ srvrLog.Warnf("Established outbound connection to "+
"peer %x, but already connected, dropping conn", "peer %x, but already connected, dropping conn",
nodePub.SerializeCompressed()) nodePub.SerializeCompressed())
s.connMgr.Remove(connReq.ID())
if connReq != nil {
s.connMgr.Remove(connReq.ID())
}
conn.Close() conn.Close()
return return
} }