Skip to content

Latest commit

 

History

History
10 lines (7 loc) · 955 Bytes

File metadata and controls

10 lines (7 loc) · 955 Bytes

Message call with hardcoded gas amount

Description:

The transfer() and send() functions forward a fixed amount of 2300 gas. Historically, it has often been recommended to use these functions for value transfers to guard against reentrancy attacks. However, the gas cost of EVM instructions may change significantly during hard forks which may break already deployed contract systems that make fixed assumptions about gas costs. For example. EIP 1884 broke several existing smart contracts due to a cost increase in the SLOAD instruction.

Remediation:

Avoid the use of transfer() and send() and do not otherwise specify a fixed amount of gas when performing calls. Use .call.value(...)("") instead. Use the checks-effects-interactions pattern and/or reentrancy locks to prevent reentrancy attacks.

References:

https://swcregistry.io/docs/SWC-134