Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Conversation

@jstarry
Copy link
Contributor

@jstarry jstarry commented Jun 29, 2021

Problem

Simulation allows processing invalid transactions with duplicate accounts.

@jstarry jstarry added the v1.7 label Jun 29, 2021
@jstarry jstarry force-pushed the simulation-dupes branch from 6beca20 to 76a8779 Compare June 29, 2021 15:59
@jstarry jstarry requested a review from mvines June 29, 2021 16:39
@codecov
Copy link

codecov bot commented Jun 29, 2021

Codecov Report

Merging #18304 (76a8779) into master (772f9d1) will decrease coverage by 0.0%.
The diff coverage is 85.7%.

@@            Coverage Diff            @@
##           master   #18304     +/-   ##
=========================================
- Coverage    82.3%    82.3%   -0.1%     
=========================================
  Files         433      433             
  Lines      121157   121168     +11     
=========================================
- Hits        99813    99785     -28     
- Misses      21344    21383     +39     

@jstarry
Copy link
Contributor Author

jstarry commented Jun 30, 2021

low risk change, merging!

@jstarry jstarry merged commit b08f8bd into solana-labs:master Jun 30, 2021
@jstarry jstarry deleted the simulation-dupes branch June 30, 2021 18:50
mergify bot pushed a commit that referenced this pull request Jun 30, 2021
mergify bot added a commit that referenced this pull request Jun 30, 2021
@brooksprumo brooksprumo mentioned this pull request Aug 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant