code-423n4 / 2021-10-slingshot-findings

0 stars 0 forks source link

ModuleRegistry: Rename modulesIndex → isModule #41

Open code423n4 opened 2 years ago

code423n4 commented 2 years ago

Handle

hickuphh3

Vulnerability details

Impact

Since the modulesIndex mapping is already declared as a public variable, the getter method isModule() is unnecessary as it provides no extra functionality.

Recommended Mitigation Steps

Rename modulesIndex to isModule, then remove the isModule() function.