Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(evm): commit temporary state on precompile start, avoiding bug stemming from uncommitted, dirty journal entries in the EVM StateDB #2086

Merged
merged 17 commits into from
Oct 24, 2024

fix(precompile-funtoken): modify StateDB after bank send operations

0e6afd8
Select commit
Loading
Failed to load commit list.
Merged

fix(evm): commit temporary state on precompile start, avoiding bug stemming from uncommitted, dirty journal entries in the EVM StateDB #2086

fix(precompile-funtoken): modify StateDB after bank send operations
0e6afd8
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 24, 2024 in 1s

71.66% of diff hit (target 0.00%)

View this Pull Request on Codecov

71.66% of diff hit (target 0.00%)

Annotations

Check warning on line 79 in x/evm/errors.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/errors.go#L77-L79 

Added lines #L77 - L79 were not covered by tests

Check warning on line 87 in x/evm/errors.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/errors.go#L81-L87 

Added lines #L81 - L87 were not covered by tests

Check warning on line 106 in x/evm/evmtest/erc20.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/evmtest/erc20.go#L104-L106 

Added lines #L104 - L106 were not covered by tests

Check warning on line 130 in x/evm/evmtest/tx.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/evmtest/tx.go#L128-L130 

Added lines #L128 - L130 were not covered by tests

Check warning on line 170 in x/evm/evmtest/tx.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/evmtest/tx.go#L170 

Added line #L170 was not covered by tests

Check warning on line 251 in x/evm/evmtest/tx.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/evmtest/tx.go#L249-L251 

Added lines #L249 - L251 were not covered by tests

Check warning on line 257 in x/evm/evmtest/tx.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/evmtest/tx.go#L255-L257 

Added lines #L255 - L257 were not covered by tests

Check warning on line 279 in x/evm/evmtest/tx.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/evmtest/tx.go#L279 

Added line #L279 was not covered by tests

Check warning on line 291 in x/evm/evmtest/tx.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/evmtest/tx.go#L291 

Added line #L291 was not covered by tests

Check warning on line 58 in x/evm/keeper/erc20.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/erc20.go#L58 

Added line #L58 was not covered by tests

Check warning on line 79 in x/evm/keeper/erc20.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/erc20.go#L79 

Added line #L79 was not covered by tests

Check warning on line 89 in x/evm/keeper/erc20.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/erc20.go#L89 

Added line #L89 was not covered by tests

Check warning on line 214 in x/evm/keeper/erc20.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/erc20.go#L213-L214 

Added lines #L213 - L214 were not covered by tests

Check warning on line 234 in x/evm/keeper/erc20.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/erc20.go#L233-L234 

Added lines #L233 - L234 were not covered by tests

Check warning on line 255 in x/evm/keeper/erc20.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/erc20.go#L255 

Added line #L255 was not covered by tests

Check warning on line 261 in x/evm/keeper/erc20.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/erc20.go#L261 

Added line #L261 was not covered by tests

Check warning on line 275 in x/evm/keeper/msg_server.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/msg_server.go#L275 

Added line #L275 was not covered by tests

Check warning on line 336 in x/evm/keeper/msg_server.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/msg_server.go#L336 

Added line #L336 was not covered by tests

Check warning on line 355 in x/evm/keeper/msg_server.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/msg_server.go#L355 

Added line #L355 was not covered by tests

Check warning on line 364 in x/evm/keeper/msg_server.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/msg_server.go#L364 

Added line #L364 was not covered by tests

Check warning on line 368 in x/evm/keeper/msg_server.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/keeper/msg_server.go#L368 

Added line #L368 was not covered by tests

Check warning on line 74 in x/evm/precompile/funtoken.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/precompile/funtoken.go#L73-L74 

Added lines #L73 - L74 were not covered by tests

Check warning on line 166 in x/evm/precompile/funtoken.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/precompile/funtoken.go#L166 

Added line #L166 was not covered by tests

Check warning on line 212 in x/evm/precompile/funtoken.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/precompile/funtoken.go#L200-L212 

Added lines #L200 - L212 were not covered by tests

Check warning on line 214 in x/evm/precompile/funtoken.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

x/evm/precompile/funtoken.go#L214 

Added line #L214 was not covered by tests
View more details on Codecov