code-423n4 / 2023-04-caviar-findings

9 stars 4 forks source link

Upgraded Q -> 2 from #215 [1683053585771] #993

Closed c4-judge closed 1 year ago

c4-judge commented 1 year ago

Judge has assessed an item in Issue #215 as 2 risk. The relevant finding follows:

QA10 Both EthRouter#buy() and EthRouter#sell() do not check whether recipient == address(0), as a result, they might send royalty fees to the zero address - loss of funds.

c4-judge commented 1 year ago

GalloDaSballo marked the issue as duplicate of #596

c4-judge commented 1 year ago

GalloDaSballo marked the issue as satisfactory