code-423n4 / 2021-04-vader-findings

1 stars 0 forks source link

mapping _isMember is never set #267

Closed code423n4 closed 3 years ago

code423n4 commented 3 years ago

Handle

paulius.eth

Vulnerability details

Impact

mapping(address => bool) _isMember; is never set nor updated so function isMember always returns false.

Recommended Mitigation Steps

Either remove this unused mapping or make use of it depending on the intentions.

0xBrian commented 3 years ago

https://github.com/vetherasset/vaderprotocol-contracts/commit/c9c066a9c8b083a751d1f4e13850253672e10b03

dmvt commented 3 years ago

duplicate of #118