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

Add kernelCTF CVE-2023-3776 (lts) #50

Merged
merged 17 commits into from
Nov 13, 2023

Conversation

d4em0n
Copy link
Contributor

@d4em0n d4em0n commented Sep 8, 2023

No description provided.

@koczkatamas koczkatamas force-pushed the master branch 2 times, most recently from d8211d8 to 22b1cba Compare October 20, 2023 09:30
Copy link
Collaborator

@koczkatamas koczkatamas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey! I've made some comments, can you please take a look and made the appropriate changes, add more explanations?

We are trying to make the submissions as understandable as possible, so it makes easier for the community to learn from them!

Thank you!

first->cmsg_type = 0x41414141;

for(int i=0x40;i<0x80;i+=0x8)
*(size_t*)&payload[i] = 0xfffffe000003df58; //cpu_entry_area fixed kernel address
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That address is at CPU#1 cpu_entry_area's entry_stack_page (stack address) while it try to push r15 in function error_entry

@d4em0n
Copy link
Contributor Author

d4em0n commented Oct 30, 2023

Hi, thanks for the comment.
We will fix the rest of our PRs after this one is merged because it's similar.

@koczkatamas koczkatamas merged commit 45de264 into google:master Nov 13, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants