bitcoinfuzz
bitcoinfuzz copied to clipboard
Add new target for `VerifyScript`
Bitcoin Core:
bool VerifyScript(const CScript& scriptSig, const CScript& scriptPubKey, const CScriptWitness* witness, unsigned int flags, const BaseSignatureChecker& checker, ScriptError* serror)
{
static const CScriptWitness emptyWitness;
if (witness == nullptr) {
witness = &emptyWitness;
}
bool hadWitness = false;
set_error(serror, SCRIPT_ERR_UNKNOWN_ERROR);
NBitcoin:
public bool VerifyScript(Script scriptSig, WitScript witness, Script scriptPubKey, TransactionChecker checker)
{
scriptSig = scriptSig ?? Script.Empty;
witness = witness ?? WitScript.Empty;
ExecutionData = new ExecutionData();
SetError(ScriptError.UnknownError);
We can get rid of script target btw.
Hi @brunoerg can I work on this?
Hi @brunoerg can I work on this?
I started doing this, sorry. That's why I assigned this issue to me.
Ok sir @brunoerg, understood , pls I am a first time contributor could you suggest me some issues that I can work with..I am preparing for some bitcoin related internships