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

Fix - Revert and feature gate incorrect error message in BPF loader #30748

Conversation

Lichtso
Copy link
Contributor

@Lichtso Lichtso commented Mar 16, 2023

Problem

#30614 removed an incorrect error condition partially and without a feature gate.

Summary of Changes

Add a feature gate remove_bpf_loader_incorrect_program_id which temporarily reverts the behavior back to what it was before.

Feature Gate Issue: #30747

@Lichtso Lichtso added bug Something isn't working feature-gate Pull Request adds or modifies a runtime feature gate runtime Issues related to runtime, BPF, and LLVM labels Mar 16, 2023
@Lichtso Lichtso force-pushed the fix/revert_and_feature_gate_incorrect_error_message branch from 891d8f7 to 955686a Compare March 16, 2023 14:26
@codecov
Copy link

codecov bot commented Mar 16, 2023

Codecov Report

Merging #30748 (7e3f9a0) into master (f42f618) will decrease coverage by 0.1%.
The diff coverage is 69.5%.

@@            Coverage Diff            @@
##           master   #30748     +/-   ##
=========================================
- Coverage    81.3%    81.3%   -0.1%     
=========================================
  Files         726      726             
  Lines      203490   203536     +46     
=========================================
- Hits       165599   165594      -5     
- Misses      37891    37942     +51     

@Lichtso Lichtso requested a review from pgarg66 March 17, 2023 10:55
@Lichtso Lichtso force-pushed the fix/revert_and_feature_gate_incorrect_error_message branch from 955686a to 3933f83 Compare March 17, 2023 10:57
pgarg66
pgarg66 previously approved these changes Mar 17, 2023
@Lichtso Lichtso merged commit 66da71f into solana-labs:master Mar 21, 2023
@Lichtso Lichtso deleted the fix/revert_and_feature_gate_incorrect_error_message branch March 21, 2023 10:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working feature-gate Pull Request adds or modifies a runtime feature gate runtime Issues related to runtime, BPF, and LLVM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants