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

Gate pointer alignment enforcement #12176

Merged
merged 4 commits into from
Sep 11, 2020

Conversation

jackcmay
Copy link
Contributor

Problem

Program syscalls were enforcing that the pointed passed from a program be aligned. Programs using the deprecated loader may be using unaligned pointers. This is causing a compatibility issue with mainnet-beta (#11973). Thanks, @ryoqun for finding this.

Summary of Changes

Gate the enforcement of pointer alignment on which loader is being used

Fixes #

@mvines mvines added the v1.3 label Sep 11, 2020
@jackcmay jackcmay added the automerge Merge this Pull Request automatically once CI passes label Sep 11, 2020
@codecov
Copy link

codecov bot commented Sep 11, 2020

Codecov Report

Merging #12176 into master will decrease coverage by 0.0%.
The diff coverage is 29.5%.

@@            Coverage Diff            @@
##           master   #12176     +/-   ##
=========================================
- Coverage    82.1%    82.0%   -0.1%     
=========================================
  Files         338      338             
  Lines       79397    79464     +67     
=========================================
+ Hits        65186    65192      +6     
- Misses      14211    14272     +61     

@jackcmay jackcmay merged commit ae7b15f into solana-labs:master Sep 11, 2020
@jackcmay jackcmay deleted the gate-align-checks branch September 11, 2020 18:07
mergify bot pushed a commit that referenced this pull request Sep 11, 2020
(cherry picked from commit ae7b15f)

# Conflicts:
#	programs/bpf/tests/programs.rs
mergify bot added a commit that referenced this pull request Sep 12, 2020
* Gate pointer alignment enforcement (#12176)

(cherry picked from commit ae7b15f)

# Conflicts:
#	programs/bpf/tests/programs.rs

* Fix conflicts

Co-authored-by: Jack May <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants