Skip to content

Commit 5e9e182

Browse files
authored
Merge pull request #9965 from ellemouton/checkNilPolicy
graph/db: do nil check on ChannelEdgePolicy
2 parents a3209a5 + 5b0a950 commit 5e9e182

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

graph/db/graph.go

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -351,10 +351,16 @@ func (c *ChannelGraph) MarkEdgeLive(chanID uint64) error {
351351

352352
info := infos[0]
353353

354+
var policy1, policy2 *models.CachedEdgePolicy
355+
if info.Policy1 != nil {
356+
policy1 = models.NewCachedPolicy(info.Policy1)
357+
}
358+
if info.Policy2 != nil {
359+
policy2 = models.NewCachedPolicy(info.Policy2)
360+
}
361+
354362
c.graphCache.AddChannel(
355-
models.NewCachedEdge(info.Info),
356-
models.NewCachedPolicy(info.Policy1),
357-
models.NewCachedPolicy(info.Policy2),
363+
models.NewCachedEdge(info.Info), policy1, policy2,
358364
)
359365
}
360366

graph/db/kv_store.go

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -530,18 +530,12 @@ func (c *KVStore) ForEachChannelCacheable(cb func(*models.CachedEdgeInfo,
530530
cachedPolicy1 = models.NewCachedPolicy(
531531
policy1,
532532
)
533-
} else {
534-
log.Warnf("ChannelEdgePolicy not "+
535-
"found using %v", key1)
536533
}
537534

538535
if policy2 != nil {
539536
cachedPolicy2 = models.NewCachedPolicy(
540537
policy2,
541538
)
542-
} else {
543-
log.Warnf("ChannelEdgePolicy not "+
544-
"found using %v", key2)
545539
}
546540

547541
return cb(

0 commit comments

Comments
 (0)