diff options
| author | MarcoFalke <[email protected]> | 2020-07-26 19:01:48 +0200 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2020-07-26 19:01:51 +0200 |
| commit | 31d2b4098a9e4ee9a694ba1ad42829637cbcf3c6 (patch) | |
| tree | 04a7cc70602a5cd9b717fd988299956fd37ba7c8 /src/script/script.cpp | |
| parent | Merge #19472: [net processing] Reduce cs_main scope in MaybeDiscourageAndDisc... (diff) | |
| parent | span: Add Span::empty() and use it in script/descriptor (diff) | |
| download | discoin-31d2b4098a9e4ee9a694ba1ad42829637cbcf3c6.tar.xz discoin-31d2b4098a9e4ee9a694ba1ad42829637cbcf3c6.zip | |
Merge #19508: Work around memory-aliasing in descriptor ParsePubkey
fa2ae0ac8d43086430a29c73940ad6b1cd129e96 span: Add Span::empty() and use it in script/descriptor (MarcoFalke)
fa8a99258947a9ee3749fa472180542920cd471c Work around memory-aliasing in descriptor ParsePubkey (MarcoFalke)
Pull request description:
While this is not undefined behaviour, the memory aliasing trick is confusing when reading the code. Having `a.size()==0` and then access `a[0]` works in this particular case, but should probably be avoided to harden the code for the future.
ACKs for top commit:
theStack:
re-ACK https://github.com/bitcoin/bitcoin/pull/19508/commits/fa2ae0ac8d43086430a29c73940ad6b1cd129e96
elichai:
ACK fa2ae0ac8d43086430a29c73940ad6b1cd129e96
jonatack:
ACK fa2ae0ac8d43086430a29c73940ad6b1cd129e96
Tree-SHA512: 0ec7b09eef45504973a195923cdf1aa8522117c8e2f69b453e5ce9aa8a7e327c71138518022c32d05133dc99cb861101ed0f60fa891814ee3e9dab3a6fa61a84
Diffstat (limited to 'src/script/script.cpp')
0 files changed, 0 insertions, 0 deletions