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.
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: Add check for empty inline initcode for contract creation #10449
Conversation
Signed-off-by: lukelee-sl <luke.lee@swirldslabs.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #10449 +/- ##
==========================================
Coverage 63.01% 63.02%
- Complexity 30800 30804 +4
==========================================
Files 3360 3360
Lines 135281 135283 +2
Branches 14074 14075 +1
==========================================
+ Hits 85253 85267 +14
+ Misses 46679 46670 -9
+ Partials 3349 3346 -3 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
.../main/java/com/hedera/node/app/service/mono/txns/contract/ContractCreateTransitionLogic.java
Show resolved
Hide resolved
...pl/src/main/java/com/hedera/node/app/service/contract/impl/infra/HevmTransactionFactory.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense to me. Thanks @lukelee-sl !
Description:
Add a check to ensure that contracts with empty initcode cannot be created. There was a check for checking empty smart contract byte code files but not for inline transactions.
Checklist