code-423n4 / 2021-11-nested-findings

1 stars 1 forks source link

NestedAsset.setFactory should be named addFactory #204

Open code423n4 opened 2 years ago

code423n4 commented 2 years ago

Handle

hyh

Vulnerability details

setFactory should be named addFactory as it doesn't set the only factory, but adds to the list of factories

https://github.com/code-423n4/2021-11-nested/blob/main/contracts/NestedAsset.sol#L133