5afe / safe-core-protocol-specs

Safe{Core} Protocol is an open, modular framework to make smart accounts secure, portable, and composable.
GNU General Public License v3.0
67 stars 13 forks source link

[Signature Validator] Evaluate security of using 4 bytes selector #47

Closed akshay-ap closed 10 months ago

akshay-ap commented 11 months ago

Pull request #44 introduces signature validator flow that allows contracts validate signatures for the account. Specs define 2 possible flows validating signatures:

To differentiate between these flows, a 4 bytes selector is prepended to the signed data. The validation flow is routed based on the the value of this selector.

A malicious actor can change this selector and change the routing of the validation flow.

This issue is created to evaluate the security impact of changed routing.