Solidity合约安全考量
有个胖子他姓杨
发表于 2022-12-30 09:42:17
111
0
0
但想要确保没有人能够以出乎意料的方式使用它就困难多了。
在 Solidity 中,这一点尤为重要,因为智能合约可以用来处理通证,甚至有可能是更有价值的东西。
除此之外,智能合约的每一次执行都是公开的,而且源代码也通常是容易获得的。( e: k' o& k: \ W2 A3 e/ M( R
当然,你总是需要考虑有多大的风险:( |- Q, p' G3 g R* U; ]" M+ d$ b
你可以将智能合约与公开的(当然也对恶意用户开放)、甚至是开源的网络服务相比较。
如果你只是在某个网络服务上存储你的购物清单,则可能不必太在意,0 [. |: T) S: c5 b& t% b& V4 D% f
但如果你使用那个网络服务管理你的银行帐户,
那就需要特别当心了。
本节将列出一些陷阱和一般性的安全建议,但这绝对不全面。' T( {, B4 T9 o6 @" Y) e$ M
另外,请时刻注意的是即使你的智能合约代码没有 bug,
但编译器或者平台本身可能存在 bug。
一个已知的编译器安全相关的 bug 列表可以在 :ref:已知bug列表 找到,/ J1 D" N/ C) t# e* C6 z. ]
这个列表也可以用程序读取。
请注意其中有一个涵盖了 Solidity 编译器的代码生成器的 bug 悬赏项目。
我们的文档是开源的,请一如既往地帮助我们扩展这一节的内容(何况其中一些例子并不会造成损失)!
陷阱
私有信息和随机性8 k1 r1 j0 N+ ]
在智能合约中你所用的一切都是公开可见的,即便是局部变量和被标记成 private 的状态变量也是如此。1 b& W3 p2 m6 I: c; j4 b+ x1 j4 w5 S9 _
如果不想让矿工作弊的话,在智能合约中使用随机数会很棘手6 ]: |2 `3 \# e9 v O8 d
(译者注:在智能合约中使用随机数很难保证节点不作弊,9 x5 ~+ y+ g% P! e' F/ Q: X
这是因为智能合约中的随机数一般要依赖计算节点的本地时间得到,: O" g' C& U6 x- v
而本地时间是可以被恶意节点伪造的,因此这种方法并不安全。
通行的做法是采用 |off_chain| 的第三方服务,比如 Oraclize 来获取随机数)。' D3 ~. [. p& C) k# ~6 }
重入. z) D- u/ G* G: i* o
任何从合约 A 到合约 B 的交互以及任何从合约 A 到合约 B 的 |ether| 的转移,都会将控制权交给合约 B。3 e) F2 X* f4 c" t# r: e5 L5 J
这使得合约 B 能够在交互结束前回调 A 中的代码。
举个例子,下面的代码中有一个 bug(这只是一个代码段,不是完整的合约):2 ?! f$ ^7 u) ^% q
::; u' y2 ?8 w2 _- P
pragma solidity ^0.4.0;/ E, a9 C i% t
// 不要使用这个合约,其中包含一个 bug。: `+ q6 T: S/ t* v
contract Fund {" @, t1 y v7 w: O( v' ]% g
/// 合约中 |ether| 分成的映射。
mapping(address => uint) shares;
/// 提取你的分成。
function withdraw() public {
if (msg.sender.send(shares[msg.sender]))( t% R6 a3 K1 m- U, t
shares[msg.sender] = 0;/ s l' h8 y% R. n4 d/ C" [
}1 Z& H9 U% ^' N4 H! r2 e
}7 d1 f1 B; }! n6 i: V c
这里的问题不是很严重,因为有限的 gas 也作为 send 的一部分,但仍然暴露了一个缺陷:
|ether| 的传输过程中总是可以包含代码执行,所以接收者可以是一个回调进入 withdraw 的合约。 o( U# [( ~: \" A/ O: h# Z! P
这就会使其多次得到退款,从而将合约中的全部 |ether| 提取。0 }" Z7 z. p" t8 l5 p% A
特别地,下面的合约将允许一个攻击者多次得到退款,因为它使用了 call ,默认发送所有剩余的 gas。! |# S# a8 o& t
::8 }# T& @% C+ y' M- `, H
pragma solidity ^0.4.0;
// 不要使用这个合约,其中包含一个 bug。5 R8 w$ D. u+ ?
contract Fund {
/// 合约中 |ether| 分成的映射。
mapping(address => uint) shares;
/// 提取你的分成。; [# B$ T" I, {# N7 N
function withdraw() public {. z4 A% j' h. G. s) I* c- w
if (msg.sender.call.value(shares[msg.sender])())6 _. z! ~/ D6 @. G* W- X( K
shares[msg.sender] = 0;3 l( P& }" l# K. _' R/ o
}! \$ |6 U% z' N: J* F; M
}
为了避免重入,你可以使用下面撰写的“检查-生效-交互”(Checks-Effects-Interactions)模式:
::
pragma solidity ^0.4.11;9 t& B" h! b$ |
contract Fund {/ w3 {1 ?8 b( S
/// 合约中 |ether| 分成的映射。8 @6 D+ ^1 ]4 Z
mapping(address => uint) shares;' F( u; u& `! f$ ^ a i
/// 提取你的分成。' c4 L1 ]/ K8 H* T6 ~1 a' B3 y# C
function withdraw() public {- q# e. W& `# u! S+ ]& x5 Q3 e6 n
var share = shares[msg.sender];$ V H* I) A8 [
shares[msg.sender] = 0;! R: K+ O( x2 m: ~8 z
msg.sender.transfer(share);
}
}# C" O5 |6 l5 J. I" V3 d+ X. L
请注意重入不仅是 |ether| 传输的其中一个影响,还包括任何对另一个合约的函数调用。
更进一步说,你也不得不考虑多合约的情况。
一个被调用的合约可以修改你所依赖的另一个合约的状态。
gas 限制和循环; t+ _+ y. j: m. R$ H5 y
必须谨慎使用没有固定迭代次数的循环,例如依赖于 |storage| 值的循环:/ N8 P9 h) R l
由于区块 gas 有限,交易只能消耗一定数量的 gas。
无论是明确指出的还是正常运行过程中的,循环中的数次迭代操作所消耗的 gas 都有可能超出区块的 gas 限制,从而导致整个合约在某个时刻骤然停止。& V0 Q! ^' H8 F4 f
这可能不适用于只被用来从区块链中读取数据的 view 函数。
尽管如此,这些函数仍然可能会被其它合约当作 |on_chain| 操作的一部分来调用,并使那些操作骤然停止。
请在合约代码的说明文档中明确说明这些情况。
发送和接收 |ether|# C* A6 g* b- T
目前无论是合约还是“外部账户”都不能阻止有人给它们发送 |ether|。5 s9 c- t% K2 Q7 G' z
合约可以对一个正常的转账做出反应并拒绝它,但还有些方法可以不通过创建消息来发送 |ether|。- g. B: D" `1 a' C8 K6 ~
其中一种方法就是单纯地向合约地址“挖矿”,另一种方法就是使用 selfdestruct(x) 。$ I: `6 O N+ K# Q0 }
如果一个合约收到了 |ether| (且没有函数被调用),就会执行 fallback 函数。* o! u/ L7 P' E
如果没有 fallback 函数,那么 |ether| 会被拒收(同时会抛出异常)。
在 fallback 函数执行过程中,合约只能依靠此时可用的“gas 津贴”(2300 gas)来执行。; j& W; _; }. B v) l& K
这笔津贴并不足以用来完成任何方式的 |storage| 访问。
为了确保你的合约可以通过这种方式收到 |ether|,请你核对 fallback 函数所需的 gas 数量- A2 c4 l1 L$ H7 {+ ~; D
(在 Remix 的“详细”章节会举例说明)。
有一种方法可以通过使用 addr.call.value(x)() 向接收合约发送更多的 gas。
这本质上跟 addr.transfer(x) 是一样的,
只不过前者发送所有剩余的 gas,并且使得接收者有能力执行更加昂贵的操作
(它只会返回一个错误代码,而且也不会自动传播这个错误)。' b; C8 V% g+ [+ H m/ e; y. }
这可能包括回调发送合约或者你想不到的其它状态改变的情况。
因此这种方法无论是给诚实用户还是恶意行为者都提供了极大的灵活性。/ a! s/ \; @3 c% L' h* s0 {
如果你想要使用 address.transfer 发送 |ether| ,你需要注意以下几个细节:
如果接收者是一个合约,它会执行自己的 fallback 函数,从而可以回调发送 |ether| 的合约。如果调用的深度超过 1024,发送 |ether| 也会失败。由于调用者对调用深度有完全的控制权,他们可以强制使这次发送失败;+ w/ a5 W1 R, J! R5 K
请考虑这种可能性,或者使用 send 并且确保每次都核对它的返回值。
更好的方法是使用一种接收者可以取回 |ether| 的方式编写你的合约。发送 |ether| 也可能因为接收方合约的执行所需的 gas 多于分配的 gas 数量而失败4 s$ [' y5 y$ k( e: u9 A
(确切地说,是使用了 require , assert, revert , throw 或者因为这个操作过于昂贵) - “gas 不够用了”。8 H9 h2 _3 i5 m; c6 n0 I
如果你使用 transfer 或者 send 的同时带有返回值检查,这就为接收者提供了在发送合约中阻断进程的方法。
再次说明,最佳实践是使用 :ref:“取回”模式而不是“发送”模式。
7 c4 a! u$ T# |7 S
调用栈深度
外部函数调用随时会失败,因为它们超过了调用栈的上限 1024。
在这种情况下,Solidity 会抛出一个异常。3 P- a8 |! g8 D+ O4 D6 d
恶意行为者也许能够在与你的合约交互之前强制将调用栈设置成一个比较高的值。' k) j7 @% u* p% l
请注意,使用 .send() 时如果超出调用栈 并不会 抛出异常,而是会返回 false。5 x$ L1 I, G9 K
低级的函数比如 .call(),.callcode() 和 .delegatecall() 也都是这样的。
tx.origin- H) U3 U; c1 W* Q/ c8 ?
永远不要使用 tx.origin 做身份认证。假设你有一个如下的钱包合约:; Y% J/ J2 b' W1 A) ?0 X+ _& I1 X
::
pragma solidity ^0.4.11;
// 不要使用这个合约,其中包含一个 bug。
contract TxUserWallet {
address owner;
function TxUserWallet() public {
owner = msg.sender;2 }. L) a: a2 k# `! S
}
function transferTo(address dest, uint amount) public {6 u. \6 b6 q0 O1 i
require(tx.origin == owner);$ _4 F& d0 @7 F9 ~0 T
dest.transfer(amount);$ r9 e, I9 g; f1 l7 F1 P
}. W5 y! U( d9 {# J7 ~6 q
}
现在有人欺骗你,将 |ether| 发送到了这个恶意钱包的地址:
::
pragma solidity ^0.4.11;
interface TxUserWallet {! f7 l3 e* d3 V2 U$ v
function transferTo(address dest, uint amount) public;
}: @* J0 Z+ H. {% |# K l( J
contract TxAttackWallet {
address owner;
function TxAttackWallet() public {9 G9 f: \1 K9 j
owner = msg.sender;5 L4 @$ a9 f+ r8 y+ M9 S& E3 V' g
}* c6 t( V d4 Y! B7 Q2 T/ s- m! W
function() public {
TxUserWallet(msg.sender).transferTo(owner, msg.sender.balance);
}
}$ _# W# b! ~+ s5 |& K7 [
如果你的钱包通过核查 msg.sender 来验证发送方身份,你就会得到恶意钱包的地址,而不是所有者的地址。% N. a) O6 {1 W5 S' p
但是通过核查 tx.origin ,得到的就会是启动交易的原始地址,它仍然会是所有者的地址。9 _3 @4 ^# d# g, l
恶意钱包会立即将你的资金抽出。
细枝末节' }2 |" l! `, S7 v- v, E6 C
在 for (var i = 0; i 中, i 的类型会变为 uint8 ,
因为这是保存 0 值所需的最小类型。如果数组超过 255 个元素,则循环不会终止。不占用完整 32 字节的类型可能包含“脏高位”。这在当你访问 msg.data 的时候尤为重要 —— 它带来了延展性风险:0 R6 r9 K, m$ D6 g+ w7 }9 c( z, l4 K
你既可以用原始字节 0xff000001 也可以用 0x00000001 作为参数来调用函数 f(uint8 x) 以构造交易。 A s4 X7 O" s; m/ g% ]/ {' w7 w
这两个参数都会被正常提供给合约,并且 x 的值看起来都像是数字 1,0 n: s- I; K8 S" v5 E( B
但 msg.data 会不一样,所以如果你无论怎么使用 keccak256(msg.data),你都会得到不同的结果。" c# k6 i4 ~# J9 h
推荐做法+ i) I6 x# N3 } `& X4 k
认真对待警告
如果编译器警告了你什么事,你最好修改一下,即使你不认为这个特定的警告不会产生安全隐患,因为那也有可能埋藏着其他的问题。$ C+ H" h2 W- A6 x( r4 |
我们给出的任何编译器警告,都可以通过轻微的修改来去掉。
同时也请尽早添加 pragma experimental "v0.5.0"; 来允许 0.5.0 版本的安全特性。7 p T, A1 c# p6 a# Q: U
注意在这种情况下,experimental 并不意味着任何有风险的安全特性,4 U* J, X5 K6 f( P5 j
它只是可以允许一些在当前版本还不支持的 Solidity 特性,来提供向后的兼容。6 r3 r; w$ J$ m5 z* D# [
限定 |ether| 的数量, b! ~. X( l1 f/ D. w
限定 |storage| 在一个智能合约中 |ether| (或者其它通证)的数量。. [2 y2 l' ^# l, h" H' A
如果你的源代码、编译器或者平台出现了 bug,可能会导致这些资产丢失。
如果你想控制你的损失,就要限定 |ether| 的数量。2 j% Y7 y& n, E! M. X
保持合约简练且模块化
保持你的合约短小精炼且易于理解。
找出无关于其它合约或库的功能。
有关源码质量可以采用的一般建议:% R; j( ~, e) U, e2 @
限制局部变量的数量以及函数的长度等等。
将实现的函数文档化,这样别人看到代码的时候就可以理解你的意图,并判断代码是否按照正确的意图实现。. z. v* D- r4 ?0 E: ^3 D! w
使用“检查-生效-交互”(Checks-Effects-Interactions)模式
大多数函数会首先做一些检查工作(例如谁调用了函数,参数是否在取值范围之内,它们是否发送了足够的 |ether| ,用户是否具有通证等等)。- q/ W" I* }, Y
这些检查工作应该首先被完成。9 M/ E* t# f# t& R7 S# \& E
第二步,如果所有检查都通过了,应该接着进行会影响当前合约状态变量的那些处理。
与其它合约的交互应该是任何函数的最后一步。" ]# c0 J# u' A: X7 z! _
早期合约延迟了一些效果的产生,为了等待外部函数调用以非错误状态返回。
由于上文所述的重入问题,这通常会导致严重的后果。
请注意,对已知合约的调用反过来也可能导致对未知合约的调用,所以最好是一直保持使用这个模式编写代码。4 T9 \4 O2 q) }2 ?
包含故障-安全(Fail-Safe)模式7 ?: @9 ]& m! _
尽管将系统完全去中心化可以省去许多中间环节,但包含某种故障-安全模式仍然是好的做法,尤其是对于新的代码来说:: |8 T4 Z8 B3 h! B6 }; N- I
你可以在你的智能合约中增加一个函数实现某种程度上的自检查,比如“ |ether| 是否会泄露?”, ?. f m# L/ M
“通证的总和是否与合约的余额相等?”等等。, Y9 S: `& |& `3 P9 Z
请记住,你不能使用太多的 gas,所以可能需要通过 |off_chain| 计算来辅助。
如果自检查没有通过,合约就会自动切换到某种“故障安全”模式,
例如,关闭大部分功能,将控制权交给某个固定的可信第三方,或者将合约转换成一个简单的“退回我的钱”合约。
2 W; |4 J8 Y0 i# k* z5 Y2 C
形式化验证1 d" G- P' \7 K5 _6 h1 J
9 c$ A; V2 b+ a" P8 @8 I& e
使用形式化验证可以执行自动化的数学证明,保证源代码符合特定的正式规范。
规范仍然是正式的(就像源代码一样),但通常要简单得多。6 M7 u( v! {3 b4 }
请注意形式化验证本身只能帮助你理解你做的(规范)和你怎么做(实际的实现)的之间的差别。3 p% n, F& @$ \# N( z: O* d
你仍然需要检查这个规范是否是想要的,而且没有漏掉由它产生的任何非计划内的效果。
BitMere.com 比特池塘系信息发布平台,比特池塘仅提供信息存储空间服务。
声明:该文观点仅代表作者本人,本文不代表比特池塘立场,且不构成建议,请谨慎对待。
声明:该文观点仅代表作者本人,本文不代表比特池塘立场,且不构成建议,请谨慎对待。
成为第一个吐槽的人