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

autopilot/manager: fix slice modifications

Browse Source 
When appending to a slice, there is no guarantee the slice won't be
modified. So instead of appending to the global slice
availableHeuristics, we create a temporary local one.
This commit is contained in:
Johan T. Halseth 2019-09-18 13:25:12 +02:00
parent df5baa4275
commit f0ba4be758
No known key found for this signature in database
GPG Key ID: 15BAADA29DA20D26
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
autopilot/manager.go
View File

@ -319,8 +319,12 @@ func (m *Manager) queryHeuristics(nodes map[NodeID]struct{}, localState bool) (
// We'll start by getting the scores from each available sub-heuristic, // We'll start by getting the scores from each available sub-heuristic,
// in addition the current agent heuristic. // in addition the current agent heuristic.
var heuristics []AttachmentHeuristic
heuristics = append(heuristics, availableHeuristics...)
heuristics = append(heuristics, m.cfg.PilotCfg.Heuristic)
report := make(HeuristicScores) report := make(HeuristicScores)
for _, h := range append(availableHeuristics, m.cfg.PilotCfg.Heuristic) { for _, h := range heuristics {
name := h.Name() name := h.Name()
// If the agent heuristic is among the simple heuristics it // If the agent heuristic is among the simple heuristics it