Summary

The feature of ownable contract can be achieved via modifier.

Vulnerability Details

With a modifier that ensures the msg.sender is the same as the contract owner we get contract which ownability is easier to maintain.

Impact

Readable code -> Easy to maintain -> Less/No bugs!

Tools Used

Manual review was used.

No automatic tools for code analysis were used.

Standard tools described in https://book.getfoundry.sh/ were used to deploy, test and query the contract.

Recommendations

Write a modifier. I assume the s_owner was renamed to i_owner (and declared immutable):

The functions that are supposed to be executed only by the owner of the contract should use that modifier in their signature: