Skip to content

[pull] master from Melon-Tropics:master#116

Merged
pull[bot] merged 6 commits intomeonBot:masterfrom
Melon-Tropics:master
Mar 24, 2020
Merged

[pull] master from Melon-Tropics:master#116
pull[bot] merged 6 commits intomeonBot:masterfrom
Melon-Tropics:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Mar 24, 2020

See Commits and Changes for more details.


Created by pull[bot]. Want to support this open source service? Please star it : )

theStack and others added 6 commits February 28, 2020 21:20
The function IsStandardTx() returns rejection reason "scriptsig-not-pushonly"
if the transaction has at least one input for which the scriptSig consists of
any other ops than just PUSHs.
…not-pushonly" txs

5aab011 test: add unit test for non-standard "scriptsig-not-pushonly" txs (Sebastian Falbesoner)

Pull request description:

  Approaches another missing unit test of issue bitcoin#17394: Checks that the function `IsStandardTx()` returns rejection reason "scriptsig-not-pushonly" if any one of the input's scriptSig consists of any other ops than just PUSHs.

ACKs for top commit:
  MarcoFalke:
    ACK 5aab011 🍟
  practicalswift:
    ACK 5aab011 -- patch looks correct

Tree-SHA512: fbe25bcf57e5f0c8d2397eb67e61fe8d9145ba83032789adb2b67d6fcbcd87e6427e9d965e8cd7bbaaea482e39ec2f110f71ef2de079c7d1fba2712848caa9ba
…drdb.h, net_permissions.h and timedata.h

4308aa6 tests: Add fuzzing harness for functions in net_permissions.h (practicalswift)
43ff0d9 tests: Add fuzzing harness for functions in timedata.h (practicalswift)
a8695db tests: Add fuzzing harness for functions in addrdb.h (practicalswift)

Pull request description:

  Add fuzzing harnesses for functions in `addrdb.h`, `net_permissions.h` and `timedata.h`.

Top commit has no ACKs.

Tree-SHA512: ea41431e7f1944ecd0c102e6ea04e70d6763dc9b6e3a0949a4f7299897a92fa3e8e7139f9f65b9508ce8d45613ea24ec0fd6d4a8be3cfd7c23136512b17770eb
@pull pull bot added the ⤵️ pull label Mar 24, 2020
@pull pull bot merged commit 5236b2e into meonBot:master Mar 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants