mirrors/lnd
1
0
Fork 0
mirror of https://github.com/lightningnetwork/lnd.git synced 2024-11-19 09:53:54 +01:00
Code Issues Projects Releases Wiki Activity

pilot: avoid cancelling the tx subscription twice, use proper wg value

Browse Source 
In this commit we fix a newly introduce bug wherein we would close the
transaction subscription twice on shutdown. This would lead to a
shutdown, but an unclean one as it would panic due to closing a channel
twice.

We fix this my removing a defer statement such that, we’ll only cancel
the subscription once.
This commit is contained in:
Olaoluwa Osuntokun 2018-01-09 12:18:57 -08:00
parent 9a76b3ee58
commit e5b74174b5
No known key found for this signature in database
GPG Key ID: 964EA263DD637C21
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
pilot.go
View File

@ -183,7 +183,7 @@ func initAutoPilot(svr *server, cfg *autoPilotConfig) (*autopilot.Agent, error)
// We'll launch a goroutine to provide the agent with notifications // We'll launch a goroutine to provide the agent with notifications
// whenever the balance of the wallet changes. // whenever the balance of the wallet changes.
svr.wg.Add(1) svr.wg.Add(2)
go func() { go func() {
defer txnSubscription.Cancel() defer txnSubscription.Cancel()
defer svr.wg.Done() defer svr.wg.Done()
@ -199,7 +199,6 @@ func initAutoPilot(svr *server, cfg *autoPilotConfig) (*autopilot.Agent, error)
}() }()
go func() { go func() {
defer txnSubscription.Cancel()
defer svr.wg.Done() defer svr.wg.Done()
for { for {