Define x/TokenFactory BeforeSend hook sentinel error

[dalmirel]

The BeforeSend hook implementation returns an error gained from the smart contract execution and depends on the info provided with the SC implementation.

Artifacts:

• osmosis/cosmos sdk fork at commit hash: 6fcd25f
• Full Token Factory branch holding new features

Analysis summary:

osmosis/x/tokenfactory/keeper/beforesend.go

Lines 105 to 110 in f09305e

	_, err = h.wasmkeeper.Sudo(ctx.WithEventManager(em), cwAddr, msgBz)
	fmt.Println(err)
	if err != nil {
	return err
	}
	}

Suggestion:
Token factory module should ensure that the error returned from the module holds a unique error code, that will explain what happened.

• Define a sentinel error that will hold information about the denomination triggering the hook
• and wrap the error returned from the smart contract as well, in order to get the information about the reason - if provided in the smart contract (blocked from/to account address; the contract is frozen,...).

[dalmirel]

Tagging audit collaboration team, to review issues as agreed. @ValarDragon @sunnya97

[ValarDragon]

Agreed, we should wrap these error types, and make more meaningful wrapping info to indicate this is the source

Nice point!

[mattverse]

hey @alexanderbez are you stabbing this rn? If not, I'd like to PR this change with the other audit items!

[alexanderbez]

hey @alexanderbez are you stabbing this rn? If not, I'd like to PR this change with the other audit items!

@mattverse go for it homie!

I'll be happy to review