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

Security Warning: RISC-V CPUs from T-HEAD May Have Vulnerabilities on Several Extensions #1727

Open
Coekjan opened this issue Aug 10, 2024 · 5 comments

Comments

@Coekjan
Copy link
Contributor

Coekjan commented Aug 10, 2024

See https://ghostwriteattack.com/ for more details.

As reported in the paper, this vulnerability allows unprivileged attackers, even those with limited access, to read and write any part of the computer’s memory and to control peripheral devices like network cards.

Box64 is using these extensions, so it might generate harmful target code in cases.

@ksco
Copy link
Collaborator

ksco commented Aug 10, 2024

We use these extensions in a good way, not an evil way. I don't see any issues in box64 side by just using thead extensions.

@Coekjan
Copy link
Contributor Author

Coekjan commented Aug 10, 2024

We use these extensions in a good way, not an evil way. I don't see any issues in box64 side by just using thead extensions.

I am not familiar with how we use these extensions in box64, so report this as a "warning".

If we are currently using them in good way, I would still suggest that code changes in the future related with these extensions should be carefully checked.

@ksco
Copy link
Collaborator

ksco commented Aug 10, 2024

If you read the paper, you'll see that it uses some reserved vector instruction that accidentally implemented by thead CPUs to access physical memory directly. We would never use reserved encoding, and we do not support xtheadvector.

@archanox
Copy link
Contributor

we do not support xtheadvector.

I wish you would 😔 there's a lot of performance left on the table with my SG2042 not having the thead custom extensions supported.

@ksco
Copy link
Collaborator

ksco commented Aug 19, 2024

we do not support xtheadvector.

I wish you would 😔 there's a lot of performance left on the table with my SG2042 not having the thead custom extensions supported.

We will. But after the v1p0 support. I think the code can be largely shared with v0p7.

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

No branches or pull requests

3 participants