Sparkn

CodeFox Inc.
DeFiFoundryProxy
15,000 USDC
View results
Submission Details
Severity: low
Valid

STORAGE COLLISION BETWEEN PROXY AND IMPLEMENTATION (LACK EIP 1967)

Summary

STORAGE COLLISION BETWEEN PROXY AND IMPLEMENTATION (LACK EIP 1967)

Vulnerability Details

Storage collision because of lack of EIP1967 could cause conflicts and override sensible variables.

Impact

When you implement proxies, logic and implementation share the same storage layout. In order to avoid storage conflicts EIP1967 was proposed.(https://eips.ethereum.org/EIPS/eip-1967) The idea is to set proxy variables at fixed positions .

There is a table at OZ site that explains this scenario more in detail:

https://docs.openzeppelin.com/upgrades-plugins/1.x/proxies

section “Unstructured Storaged Proxies”

Tools Used

Recommendations

Consider using EIP1967

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.