Platform: Code4rena
Start Date: 04/11/2022
Pot Size: $42,500 USDC
Total HM: 9
Participants: 88
Period: 4 days
Judge: 0xean
Total Solo HM: 2
Id: 180
League: ETH
Rank: 34/88
Findings: 1
Award: $138.28
🌟 Selected for report: 0
🚀 Solo Findings: 0
138.2838 USDC - $138.28
https://github.com/code-423n4/2022-11-size/blob/79aa9c01987e57a760521acecfe81b28eab3b313/src/SizeSealed.sol#L358-L387 https://github.com/code-423n4/2022-11-size/blob/79aa9c01987e57a760521acecfe81b28eab3b313/src/SizeSealed.sol#L321-L330 https://github.com/code-423n4/2022-11-size/blob/79aa9c01987e57a760521acecfe81b28eab3b313/src/SizeSealed.sol#L336-L352 https://github.com/code-423n4/2022-11-size/blob/79aa9c01987e57a760521acecfe81b28eab3b313/src/SizeSealed.sol#L409 https://github.com/code-423n4/2022-11-size/blob/79aa9c01987e57a760521acecfe81b28eab3b313/src/SizeSealed.sol#L439
Miscalculation of transferred funds and finally, loss of funds
Solmate's safeTransfer() and safeTransferFrom() functions do not check the ext code size of the address of the recipient and so, it may lead to a miscalculation of funds as it returns true for addresses that do not have a contract inside them.
Manual review
Using SafeERC20 instead of solmate
#0 - trust1995
2022-11-08T23:05:49Z
Dup of #309 , but very low quality and vague.
#1 - c4-judge
2022-11-10T00:59:36Z
0xean marked the issue as duplicate
#2 - c4-judge
2022-12-06T00:22:05Z
0xean marked the issue as satisfactory