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

featurize bpf function hash collision fix #24262

Merged
merged 1 commit into from
Apr 13, 2022

Conversation

jackcmay
Copy link
Contributor

Problem

There is a potential for conflicting hashes between syscalls and BPF functions could result in unexpected behavior

Summary of Changes

Error if detected

Fixes #
Feature gate issue #24261

@jackcmay jackcmay added the feature-gate Pull Request adds or modifies a runtime feature gate label Apr 11, 2022
@codecov
Copy link

codecov bot commented Apr 12, 2022

Codecov Report

Merging #24262 (b0bd538) into master (2da4e3e) will increase coverage by 0.2%.
The diff coverage is 78.4%.

❗ Current head b0bd538 differs from pull request most recent head 52b048f. Consider uploading reports for the commit 52b048f to get more accurate results

@@            Coverage Diff            @@
##           master   #24262     +/-   ##
=========================================
+ Coverage    81.8%    82.0%   +0.2%     
=========================================
  Files         581      581             
  Lines      158312   161383   +3071     
=========================================
+ Hits       129518   132367   +2849     
- Misses      28794    29016    +222     

@jackcmay jackcmay force-pushed the featurize-hash-collision-fix branch from b0bd538 to 52b048f Compare April 12, 2022 16:55
@jackcmay
Copy link
Contributor Author

@merify-io rebase

@jackcmay jackcmay force-pushed the featurize-hash-collision-fix branch from 52b048f to 06d1871 Compare April 12, 2022 20:56
@jackcmay jackcmay merged commit 138f04a into solana-labs:master Apr 13, 2022
@jackcmay jackcmay deleted the featurize-hash-collision-fix branch April 13, 2022 00:52
jeffwashington pushed a commit to jeffwashington/solana that referenced this pull request Jun 29, 2022
jeffwashington pushed a commit to jeffwashington/solana that referenced this pull request Jun 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-gate Pull Request adds or modifies a runtime feature gate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant