Abracadabra Mimswap - 0xJaeger's results

General Information

Platform: Code4rena

Start Date: 07/03/2024

Pot Size: $63,000 USDC

Total HM: 20

Participants: 36

Period: 5 days

Judge: cccz

Total Solo HM: 11

Id: 349

League: BLAST

Abracadabra Money

Findings Distribution

Researcher Performance

Rank: 35/36

Findings: 1

Award: $15.33

🌟 Selected for report: 0

🚀 Solo Findings: 0

Awards

15.328 USDC - $15.33

Labels

bug
downgraded by judge
grade-b
QA (Quality Assurance)
:robot:_18_group
duplicate-18
Q-20

External Links

Lines of code

https://github.com/code-423n4/2024-03-abracadabra-money/blob/1f4693fdbf33e9ad28132643e2d6f7635834c6c6/src/staking/LockingMultiRewards.sol#L349

Vulnerability details

Impact

The stake() and lock() functions have a modifier whenNotPaused restricting use when the contract is paused. However, the privileged stakeFor() function allows operators to deposit and lock funds (for themselves or others) regardless of the pause state.

Proof of Concept

stakeFor() is missing the whenNotPaused modifier.

Tools Used

Manual Analysis

Add the missing modifier to the function so that deposit are blocked when contract is paused.

Assessed type

Other

#0 - c4-pre-sort

2024-03-15T12:46:44Z

141345 marked the issue as duplicate of #18

#1 - c4-judge

2024-03-29T16:07:17Z

thereksfour changed the severity to QA (Quality Assurance)

#2 - c4-judge

2024-03-29T16:43:39Z

thereksfour marked the issue as grade-b

AuditHub

A portfolio for auditors, a security profile for protocols, a hub for web3 security.

Built bymalatrax © 2024

Auditors

Browse

Contests

Browse

Get in touch

ContactTwitter