lightninglabs / taproot-assets / 10528329850

40%

main: 47%

Build Type

Pull #1103

github

Committed by guggero

Commit Message

multi: copied assets shouldn't inherit locktime from inputs

In this commit, we fix an existing bug related to lock times and split
commitments. Before this commit, if an input had a relative lock time,
then when we went to make the new split assets, we would _copy_ that
value into the split. This isn't correct as the input is
valid/confirmed, so we don't need to copy over the lock time
information.

The prior behavior would cause certain classes of spends to fail, as
we would be validating a new root asset that has no lock time, but the
root asset split inserted into the split commitment would be carrying
the old lock time. When verifying the split, we would set the lock times
of the split to that of the new asset:
https://github.com/lightninglabs/taproot-assets/blob/e893dee87/vm/vm.go#L305-L307.

As we copied over the lock time from the input, we would now effectively
invalid the split commitment.

Fixes https://github.com/lightninglabs/taproot-assets/issues/1099

Pull Request Pull Request #1103: commitment: splits shouldn't inherit locktime from inputs

Run Details

10 of 15 new or added lines in 3 files covered. (66.67%)

27 existing lines in 8 files now uncovered.

23972 of 59442 relevant lines covered (40.33%)

55225.01 hits per line