search

code-423n4 / 2022-01-openleverage-findings

0 stars 0 forks source link

Make `Adminable.developer` immutable since it's set only once in the constructor #255

Closed code423n4 closed 2 years ago

code423n4 commented 2 years ago

Handle

Ruhum

Vulnerability details

Impact

Immutable variables use less gas. The developer state variable seems tobe only set in the constructor of the Adminable contract

Proof of Concept

https://github.com/code-423n4/2022-01-openleverage/blob/main/openleverage-contracts/contracts/Adminable.sol#L9

Tools Used

none

Recommended Mitigation Steps

make immutable

ColaM12 commented 2 years ago

Duplicate to #11