diff options
| author | Robbie Ferguson <robbieferguson139@gmail.com> | 2018-02-27 18:48:16 +0800 |
|---|---|---|
| committer | chriseth <chris@ethereum.org> | 2018-04-06 22:38:59 +0800 |
| commit | 49567b3f4c216b3345cd5e6c2f2a9a3b27b54fb6 (patch) | |
| tree | 9fc119be16c25614e5ff8add295f282c7b9e07d7 | |
| parent | 866df15daa21ff55ebd7084976a32cd698fcdb9e (diff) | |
| download | dexon-solidity-49567b3f4c216b3345cd5e6c2f2a9a3b27b54fb6.tar.gz dexon-solidity-49567b3f4c216b3345cd5e6c2f2a9a3b27b54fb6.tar.zst dexon-solidity-49567b3f4c216b3345cd5e6c2f2a9a3b27b54fb6.zip | |
Update Delegatecall Description in Docs
The previous description did not include the fact that the storage locations of the two contracts must align up until the storage variable(s) affected in order for the called contract to successfully write to the caller's storage. If they are misaligned, delegatecall will silently fail. This is difficult to debug without underlying knowledge of how delegatecall works, and clarity in the docs would certainly be helpful.
| -rw-r--r-- | docs/units-and-global-variables.rst | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/docs/units-and-global-variables.rst b/docs/units-and-global-variables.rst index 2571f20a..e7f41ed1 100644 --- a/docs/units-and-global-variables.rst +++ b/docs/units-and-global-variables.rst @@ -169,6 +169,13 @@ For more information, see the section on :ref:`address`. Use a pattern where the recipient withdraws the money. .. note:: + If storage variables are accessed via a low-level delegatecall, the storage layout of the two contracts + must align in order for the called contract to correctly access the storage variables of the calling contract by name. + This is of course not the case if storage pointers are passed as function arguments as in the case for + the high-level libraries. + + +.. note:: The use of ``callcode`` is discouraged and will be removed in the future. .. index:: this, selfdestruct |