Skip to content

v3.1: svm: remove NativeLoader-owned rent assert (backport of #9574)#9583

Merged
2501babe merged 1 commit intov3.1from
mergify/bp/v3.1/pr-9574
Dec 17, 2025
Merged

v3.1: svm: remove NativeLoader-owned rent assert (backport of #9574)#9583
2501babe merged 1 commit intov3.1from
mergify/bp/v3.1/pr-9574

Conversation

@mergify
Copy link

@mergify mergify bot commented Dec 16, 2025

Problem

TransactionAccountStateInfo::new() contains a debug assert that a writable account is not owned by NativeLoader. this is overly general, as the condition we actually value is that a reserved account key is not writable

Summary of Changes

remove it. we could check against ReservedAccountKeys here instead but this would require pulling it from Bank through svm. since it is only is debug assert and the condition itself is checked in sanitization, this does not seem necessary


This is an automatic backport of pull request #9574 done by Mergify.

@mergify mergify bot requested a review from a team as a code owner December 16, 2025 16:27
@codecov-commenter
Copy link

codecov-commenter commented Dec 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.2%. Comparing base (5347b32) to head (c65f274).

Additional details and impacted files
@@           Coverage Diff           @@
##             v3.1    #9583   +/-   ##
=======================================
  Coverage    83.2%    83.2%           
=======================================
  Files         865      865           
  Lines      376060   376059    -1     
=======================================
+ Hits       313042   313046    +4     
+ Misses      63018    63013    -5     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@2501babe 2501babe force-pushed the mergify/bp/v3.1/pr-9574 branch from c8d2808 to c65f274 Compare December 17, 2025 10:58
@2501babe 2501babe requested a review from Lichtso December 17, 2025 11:53
@2501babe 2501babe added the automerge automerge Merge this Pull Request automatically once CI passes label Dec 17, 2025
@2501babe 2501babe merged commit 7abb6df into v3.1 Dec 17, 2025
45 checks passed
@2501babe 2501babe deleted the mergify/bp/v3.1/pr-9574 branch December 17, 2025 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge automerge Merge this Pull Request automatically once CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants