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

Fix subtraction overflow#21871

Merged
carllin merged 1 commit intosolana-labs:masterfrom
carllin:FixAncestorHashes
Dec 14, 2021
Merged

Fix subtraction overflow#21871
carllin merged 1 commit intosolana-labs:masterfrom
carllin:FixAncestorHashes

Conversation

@carllin
Copy link
Copy Markdown
Contributor

@carllin carllin commented Dec 14, 2021

Problem

AncestorHashesService can overflow on bad response

Summary of Changes

Fixes #

@carllin carllin added the v1.9 label Dec 14, 2021
@carllin carllin requested a review from AshwinSekar December 14, 2021 06:04
AshwinSekar
AshwinSekar previously approved these changes Dec 14, 2021
@mergify mergify Bot dismissed AshwinSekar’s stale review December 14, 2021 08:13

Pull request has been modified.

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 14, 2021

Codecov Report

Merging #21871 (62807e9) into master (a400b5e) will decrease coverage by 0.0%.
The diff coverage is 78.4%.

@@            Coverage Diff            @@
##           master   #21871     +/-   ##
=========================================
- Coverage    81.3%    81.2%   -0.1%     
=========================================
  Files         516      516             
  Lines      144033   144225    +192     
=========================================
+ Hits       117139   117249    +110     
- Misses      26894    26976     +82     

@carllin carllin merged commit cb395ab into solana-labs:master Dec 14, 2021
mergify Bot pushed a commit that referenced this pull request Dec 14, 2021
(cherry picked from commit cb395ab)
mergify Bot added a commit that referenced this pull request Dec 14, 2021
(cherry picked from commit cb395ab)

Co-authored-by: carllin <carl@solana.com>
@brooksprumo brooksprumo mentioned this pull request Jan 5, 2022
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.

2 participants