highperfocused/lnd
1
0
Fork 0
mirror of https://github.com/lightningnetwork/lnd.git synced 2025-10-10 22:42:40 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #5743 from guggero/amp-string-nil

Browse Source 
record: fix nil pointer in log string
This commit is contained in:
Oliver Gugger
2021-09-16 20:16:01 +02:00
committed by GitHub
parent 7d012ae581 dd3f3e7ce5
commit 44d73d7e3e
3 changed files with 28 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
docs/release-notes/release-notes-0.14.0.md
View File

@@ -287,19 +287,31 @@ you.
## Bug Fixes ## Bug Fixes
A bug has been fixed that would cause `lnd` to [try to bootstrap using the * A bug has been fixed that would cause `lnd` to [try to bootstrap using the
currnet DNS seeds when in SigNet currnet DNS seeds when in SigNet
mode](https://github.com/lightningnetwork/lnd/pull/5564). mode](https://github.com/lightningnetwork/lnd/pull/5564).
[A validation check for sane `CltvLimit` and `FinalCltvDelta` has been added for `REST`-initiated payments.](https://github.com/lightningnetwork/lnd/pull/5591) * [A validation check for sane `CltvLimit` and `FinalCltvDelta` has been added
for `REST`-initiated
payments](https://github.com/lightningnetwork/lnd/pull/5591).
[A bug has been fixed with Neutrino's `RegisterConfirmationsNtfn` and `RegisterSpendNtfn` calls that would cause notifications to be missed.](https://github.com/lightningnetwork/lnd/pull/5453) * [A bug has been fixed with Neutrino's `RegisterConfirmationsNtfn` and
`RegisterSpendNtfn` calls that would cause notifications to be
missed](https://github.com/lightningnetwork/lnd/pull/5453).
[A bug has been fixed when registering for spend notifications in the `txnotifier`. A re-org notification would previously not be dispatched in certain scenarios.](https://github.com/lightningnetwork/lnd/pull/5465) * [A bug has been fixed when registering for spend notifications in the
`txnotifier`. A re-org notification would previously not be dispatched in
certain scenarios](https://github.com/lightningnetwork/lnd/pull/5465).
[Catches up on blocks in the router](https://github.com/lightningnetwork/lnd/pull/5315) in order to fix an "out of order" error that crops up. * [Catches up on blocks in the
router](https://github.com/lightningnetwork/lnd/pull/5315) in order to fix an
"out of order" error that crops up.
[Fix healthcheck might be running after the max number of attempts are reached.](https://github.com/lightningnetwork/lnd/pull/5686) * [Fix healthcheck might be running after the max number of attempts are
reached](https://github.com/lightningnetwork/lnd/pull/5686).
* [Fix crash with empty AMP or MPP record in
invoice](https://github.com/lightningnetwork/lnd/pull/5743).
## Documentation ## Documentation

4
record/amp.go
View File

@@ -102,6 +102,10 @@ func (a *AMP) PayloadSize() uint64 {
// String returns a human-readble description of the amp payload fields. // String returns a human-readble description of the amp payload fields.
func (a *AMP) String() string { func (a *AMP) String() string {
if a == nil {
return "<nil>"
}
return fmt.Sprintf("root_share=%x set_id=%x child_index=%d", return fmt.Sprintf("root_share=%x set_id=%x child_index=%d",
a.rootShare, a.setID, a.childIndex) a.rootShare, a.setID, a.childIndex)
} }

4
record/mpp.go
View File

@@ -105,5 +105,9 @@ func (r *MPP) PayloadSize() uint64 {
// String returns a human-readable representation of the mpp payload field. // String returns a human-readable representation of the mpp payload field.
func (r *MPP) String() string { func (r *MPP) String() string {
if r == nil {
return "<nil>"
}
return fmt.Sprintf("total=%v, addr=%x", r.totalMsat, r.paymentAddr) return fmt.Sprintf("total=%v, addr=%x", r.totalMsat, r.paymentAddr)
} }