search

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

0 stars 0 forks source link

nonReentrant modifier isn't necessary for executeTrades function #15

Open code423n4 opened 2 years ago

code423n4 commented 2 years ago

Handle

pants

Vulnerability details

Notice that executeTrades is the only function that is external and not onlyAdmin. Therefore a user can't call another function is the contract anyway and this modifier unnecessary.