[wip] Spurious Dragon: Eip161 state cleaning #288
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pirapira
force-pushed
the
eip161_state_clean
branch
2 times, most recently
from
e501b3c to
120fd5d
Compare
pirapira
commented
Apr 28, 2017
Paper.tex
Outdated
| \wedge \boldsymbol{\sigma}[a]_n = 0 | ||
| \wedge \boldsymbol{\sigma}[a]_b = 0 | ||
| \end{equation} | ||
| Even callable precompiled contracts can have an empty account state. This is because their account states do not usually contain the code describing its behavior. |
There was a problem hiding this comment.
Need to remove one of the double spaces.
pirapira
commented
Apr 28, 2017
Paper.tex
Outdated
|
|
||
| An account is \textit{dead} when its account state is non-existent or empty: | ||
| \begin{equation} | ||
| \mathtt{\tiny DEAD}(\boldsymbol{\sigma}, a) |
There was a problem hiding this comment.
There is a spurious newline here.
pirapira
commented
Apr 28, 2017
Paper.tex
Outdated
| \boldsymbol{\sigma}^*[a] &\equiv& \big( 0, v + v', \mathtt{\tiny TRIE}(\varnothing), \mathtt{\tiny KEC}\big(()\big) \big) \\ | ||
| \boldsymbol{\sigma}^*[s]_b &\equiv& \boldsymbol{\sigma}[s]_b - v | ||
| \boldsymbol{\sigma}^*[a] &\equiv& \big( 1, v + v', \mathtt{\tiny TRIE}(\varnothing), \mathtt{\tiny KEC}\big(()\big) \big) \\ | ||
| \boldsymbol{\sigma}^*[s] &\equiv& |
There was a problem hiding this comment.
There is a spurious newline here.
pirapira
commented
Apr 28, 2017
Paper.tex
Outdated
| \boldsymbol{\sigma}^*[s] &\equiv& | ||
| \begin{cases} | ||
| \varnothing & \text{if}\ \boldsymbol{\sigma}[s] = \varnothing \ \wedge\ v = 0\\ | ||
| \mathbf{a}^* & \text{otherwise} |
There was a problem hiding this comment.
There is a spurious space here.
pirapira
commented
Apr 28, 2017
Paper.tex
Outdated
| I_a & \equiv & r \\ | ||
| I_o & \equiv & o \\ | ||
| I_p & \equiv & p \\ | ||
| I_\mathbf{d} & \equiv & \mathbf{d} \\ | ||
| I_s & \equiv & s \\ | ||
| I_v & \equiv & \tilde{v} \\ | ||
| I_e & \equiv & e \\ | ||
| \mathbf{t} & \equiv & \{s, r\} \\ |
There was a problem hiding this comment.
There are spurious spaces here.
pirapira
commented
Apr 28, 2017
Paper.tex
Outdated
| \Omega(B, \boldsymbol{\sigma}) & \equiv & \boldsymbol{\sigma}': \boldsymbol{\sigma}' = \boldsymbol{\sigma} \quad \text{except:} \\ | ||
| \boldsymbol{\sigma}'[{B_H}_c]_b & = & \boldsymbol{\sigma}[{B_H}_c]_b + (1 + \frac{\lVert B_\mathbf{U}\rVert}{32})R_b \\ | ||
| \forall_{U \in B_\mathbf{U}}: \\ \nonumber | ||
| \boldsymbol{\sigma}'[U_c]_b & = & \boldsymbol{\sigma}[U_c]_b + (1 + \frac{1}{8} (U_i - {B_H}_i)) R_b | ||
| \boldsymbol{\sigma}'[U_c] & = & \begin{cases} | ||
| \varnothing &\text{if}\ \boldsymbol{\sigma}[U_c] = \varnothing\ \wedge\ R = 0\\ |
There was a problem hiding this comment.
There is a spurious space here and on the next line.
pirapira
commented
Apr 28, 2017
Paper.tex
Outdated
| \varnothing &\text{if}\ \boldsymbol{\sigma}[U_c] = \varnothing\ \wedge\ R = 0\\ | ||
| \mathbf{a}' &\text{otherwise} | ||
| \end{cases}\\ | ||
| \mathbf{a}' &\equiv& (\boldsymbol{\sigma}[U_c]_n, \boldsymbol{\sigma}[U_c]_b + R, |
There was a problem hiding this comment.
Spurious newline and spaces.
pirapira
commented
Apr 28, 2017
Paper.tex
Outdated
| At block 2675119, in the transaction \texttt{0xcf416c536ec1a19ed1fb89e4ec7ffb3cf73aa413b3aa9b77d60e4fd81a4296ba}, | ||
| an account at address 0x03 was called and an out-of-gas occurred during the call. | ||
| Against the equation (\ref{eq:pre}), this added 0x03 in the set of touched transactions, and | ||
| this transaction turned $\boldsymbol{\sigma}[0x03]$ into $\varnothing$. |
There was a problem hiding this comment.
The whole paragraph should occupy just one line.
pirapira
commented
Apr 28, 2017
Paper.tex
Outdated
|
|
||
| At block 2675119, in the transaction \texttt{0xcf416c536ec1a19ed1fb89e4ec7ffb3cf73aa413b3aa9b77d60e4fd81a4296ba}, | ||
| an account at address 0x03 was called and an out-of-gas occurred during the call. | ||
| Against the equation (\ref{eq:pre}), this added 0x03 in the set of touched transactions, and |
There was a problem hiding this comment.
Not "touched transactions" but "touched addresses".
The description is based on ethereum/go-ethereum#3341 (comment)
pirapira
force-pushed
the
eip161_state_clean
branch
from
120fd5d to
09aa9bf
Compare
pirapira
changed the title
This has to be fixed in earlier versions as well
fix a typo
|
The formulation here depends on ethereum/EIPs#716 |
pirapira
changed the title
|
This one should target |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This implements ethereum/EIPs#161 as part of the Spurious Dragon changes.