Page MenuHomePhabricator

Use 72 byte dummy signatures when watching only inputs may be used
ClosedPublic

Authored by deadalnix on Nov 19 2019, 15:53.

Details

Summary

With watching only inputs, we do not know how large the signatures
for those inputs will be as their signers may not have implemented
71 byte signatures. Thus we estimate their fees using the 72 byte
dummy signature to ensure that we pay enough fees.

This only effects fundrawtransaction when includeWatching is true.

This is a partial backport of Core PR13666 : https://github.com/bitcoin/bitcoin/pull/13666/commits/e306be742932d4ea5aca0ea4768e54b2fc3dc6a0

Depends on D4468

Test Plan
make check
./test/functional/test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
pr13666-3
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 8158
Build 14348: Default Diff Build & Tests
Build 14347: arc lint + arc unit