sherlock-audit / 2023-01-sentiment-judging

2 stars 0 forks source link

GalloDaSballo - Unapproved Accounts will revert until approved #18

Closed github-actions[bot] closed 1 year ago

github-actions[bot] commented 1 year ago

GalloDaSballo

medium

Unapproved Accounts will revert until approved

Summary

Each new user account will get reverts until plutusDAO approves them.

Vulnerability Detail

The call from AccountManager -> Account.exec will revert until PlutusDAO approves each account contract individually

This will prevent them from using the token, while it may be active as an asset

Impact

The integration will require an extra manual step, for it to be usable by specific accounts

Code Snippet

https://arbiscan.io/address/0x13f0d29b5b83654a200e4540066713d50547606e#code#F1#L170

  function _isEligibleSender() private view {
    if (
      msg.sender != tx.origin && whitelist.isWhitelisted(msg.sender) == false && partners[msg.sender].isActive == false
    ) revert UNAUTHORIZED();
  }

Tool used

Manual Review

Recommendation

Discuss with PlutusDAO for ways to automate the approval

Duplicate of #24