decred / dcrlnd

Decred Lightning Network Daemon ⚡️
MIT License
36 stars 24 forks source link

Fix zpay decode #59

Closed matheusd closed 4 years ago

matheusd commented 4 years ago

This fixes an issue where modifying the checksum of an LN invoice in a specific way could trigger payment to a wrong destination node.

While the payment would still fail due to the new destination node not being known, the specific error triggered would be opaque to the user, so this ensures the error will actually happen during invoice decoding vs during payment.