Platform: Code4rena
Start Date: 03/11/2022
Pot Size: $115,500 USDC
Total HM: 17
Participants: 120
Period: 7 days
Judge: LSDan
Total Solo HM: 1
Id: 174
League: ETH
Rank: 120/120
Findings: 1
Award: $2.67
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: __141345__
Also found by: 0xdeadbeef0x, 8olidity, Amithuddar, Bnke0x0, Ch_301, Deivitto, IllIllI, KingNFT, Nyx, RaymondFam, RedOneN, Satyam_Sharma, SmartSek, Tomo, adriro, bananasboys, carlitox477, cccz, cloudjunky, codexploder, corerouter, cryptonue, d3e4, datapunk, joestakey, martin, merlin, minhquanym, pashov, peanuts, rvierdiiev
2.6694 USDC - $2.67
https://github.com/code-423n4/2022-11-debtdao/blob/update-readme/contracts/utils/LineLib.sol#L48
Transfer() has typically been recommended by the security community because it helps guard against reentrancy attacks. Any smart contract that uses transfer() or send() is taking a hard dependency on gas costs by forwarding a fixed amount of gas: 2300. It works under the assumption that gas costs wouldn’t change, but that assumption turned out to be incorrect since the Istanbul hard fork. As a result, the transfer() functions may fail frequently due to limited gas.
https://github.com/code-423n4/2022-11-debtdao/blob/update-readme/contracts/utils/LineLib.sol#L48
Now it is recommended to stop using transfer() and switch to using call() instead.
#0 - c4-judge
2022-11-17T11:59:58Z
dmvt marked the issue as duplicate of #14
#1 - c4-judge
2022-11-17T19:19:07Z
dmvt marked the issue as partial-50
#2 - C4-Staff
2022-12-20T05:56:43Z
liveactionllama marked the issue as duplicate of #369