Summary

If you have a view function, you can just copy a storage into a memory variable

Vulnerability Details

By changing the unnecessary storage specifier into memory, you decrease the gas cost of a transaction, since reading from storage is a costly operation

Impact

excessively increased gas cost

Tools Used

manual review

Recommendations

in all view functions, change the storage specifier to memory