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.

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

Don't use global storage account #2909

Merged
merged 1 commit into from
Feb 22, 2019

Conversation

sakridge
Copy link
Member

Problem

Other accounts would not be able to modify the system accounts userdata.

Summary of Changes

Use accounts[0] keyed account as storage state holder. Each validator will have to create this storage account.

Fixes #

@sakridge sakridge requested a review from garious February 22, 2019 22:49
@sakridge sakridge force-pushed the remove-storage-system-account branch 2 times, most recently from ab10549 to 3536572 Compare February 22, 2019 22:51
@garious
Copy link
Contributor

garious commented Feb 22, 2019

Awesome, thanks for the quick work!

Other accounts would not be able to modify the system accounts userdata.
@sakridge sakridge force-pushed the remove-storage-system-account branch from 3536572 to 39632c1 Compare February 22, 2019 23:22
@codecov
Copy link

codecov bot commented Feb 22, 2019

Codecov Report

Merging #2909 into master will increase coverage by <.1%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #2909     +/-   ##
========================================
+ Coverage    78.3%   78.3%   +<.1%     
========================================
  Files         127     127             
  Lines       19534   19478     -56     
========================================
- Hits        15299   15269     -30     
+ Misses       4235    4209     -26

@sakridge sakridge merged commit 66891d9 into solana-labs:master Feb 22, 2019
@sakridge sakridge deleted the remove-storage-system-account branch February 22, 2019 23:59
yihau pushed a commit to yihau/solana that referenced this pull request Sep 28, 2024
…solana-labs#2687) (solana-labs#2909)

remove disable_block_production_forwarding cli flag (solana-labs#2687)

(cherry picked from commit df892c4)

Co-authored-by: Andrew Fitzgerald <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants