Page MenuHomePhabricator

psbt: Assert that tx has a value in UpdatePSBTOutput
ClosedPublic

Authored by PiRK on Aug 21 2023, 11:22.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABCaa109b915fb4: psbt: Assert that tx has a value in UpdatePSBTOutput
Summary

This was previously done implicitly in boost::optional by BOOST_ASSERT.
Also, it was checked at runtime by valgrind if for some reason the
assert was disabled.

std::optional dereference won't assert, so add the Assert here
explicitly.

The explicit Assert also helps to document the code better.

This is a partial backport of core#20671 and a follow-up to D8494

Test Plan

ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable