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

Fix potential integer overflow in SGX memory range calculation. #60496

Merged
merged 1 commit into from
May 3, 2019

Conversation

jethrogb
Copy link
Contributor

@jethrogb jethrogb commented May 3, 2019

Thanks to Eduard Marin and David Oswald at the University of Burmingham, and Jo Van Bulck at KU Leuven for discovering this issue.

Thanks to Eduard Marin and David Oswald at the University of Burmingham,
and Jo Van Bulck at KU Leuven for discovering this issue.
@rust-highfive
Copy link
Collaborator

r? @sfackler

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 3, 2019
@jethrogb
Copy link
Contributor Author

jethrogb commented May 3, 2019

r? @alexcrichton

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented May 3, 2019

📌 Commit 1dc4a38 has been approved by alexcrichton

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 3, 2019
@jseyfried
Copy link
Contributor

@bors p=2 (security issue)

@bors
Copy link
Contributor

bors commented May 3, 2019

@jseyfried: 🔑 Insufficient privileges: not in try users

Centril added a commit to Centril/rust that referenced this pull request May 3, 2019
…w, r=alexcrichton

Fix potential integer overflow in SGX memory range calculation.

Thanks to Eduard Marin and David Oswald at the University of Burmingham, and Jo Van Bulck at KU Leuven for discovering this issue.
@Centril
Copy link
Contributor

Centril commented May 3, 2019

@bors p=2

Included in a rollup but in case that fails.

@Centril Centril mentioned this pull request May 3, 2019
@bors
Copy link
Contributor

bors commented May 3, 2019

⌛ Testing commit 1dc4a38 with merge a340455...

bors added a commit that referenced this pull request May 3, 2019
…ichton

Fix potential integer overflow in SGX memory range calculation.

Thanks to Eduard Marin and David Oswald at the University of Burmingham, and Jo Van Bulck at KU Leuven for discovering this issue.
@bors bors mentioned this pull request May 3, 2019
@bors
Copy link
Contributor

bors commented May 3, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: alexcrichton
Pushing a340455 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label May 3, 2019
@bors bors merged commit 1dc4a38 into rust-lang:master May 3, 2019
@workingjubilee workingjubilee added the O-SGX Target: SGX label Jul 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. O-SGX Target: SGX S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants