bitcoinfuzz icon indicating copy to clipboard operation
bitcoinfuzz copied to clipboard

Add new target for `VerifyScript`

Open brunoerg opened this issue 2 months ago • 4 comments

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);

brunoerg avatar Oct 16 '25 19:10 brunoerg

We can get rid of script target btw.

brunoerg avatar Oct 16 '25 21:10 brunoerg

Hi @brunoerg can I work on this?

Sukuna0007Abhi avatar Oct 18 '25 06:10 Sukuna0007Abhi

Hi @brunoerg can I work on this?

I started doing this, sorry. That's why I assigned this issue to me.

brunoerg avatar Oct 20 '25 11:10 brunoerg

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

Sukuna0007Abhi avatar Oct 20 '25 11:10 Sukuna0007Abhi