First Flight #1: PasswordStore

Summary

The current implementation declares s_owner only as a private variable, which is considered low risk. However, there is an opportunity for optimization and increased security by declaring it as immutable.

Vulnerability Details

In the current implementation, the s_owner variable is mutable which means it can be changed within the contract. While this may not pose a significant risk on its own, there is room for improvement in terms of gas optimization and security.

Impact

Tools Used

VScode, REMIX IDE

Recommendations

Is it recommended to declare s_owner as immutable:

declaring s_owner as immutable guarantees that it's not gonna be changed in the contract even if someones tries to, this improves the security of the contract.