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 atty potential unsoundness (unaligned read) advisory #1462

Merged
merged 4 commits into from
Nov 22, 2022

Conversation

niluxv
Copy link
Contributor

@niluxv niluxv commented Nov 4, 2022

See issue #1457.

@pinkforest Something like this?

@pinkforest pinkforest added Unsound Informational / Unsound Propose-Merge Propose-Merge labels Nov 7, 2022
@pinkforest
Copy link
Contributor

Thanks for the contribution - just couple of tiny cosmetic things please and one format fix :) thanks ❤️

I'll leave this for a week open for anyone to comment as this will cause a lot of traffic

@niluxv niluxv force-pushed the atty-unsound-unaligned-read branch 3 times, most recently from 498b287 to 0fd6705 Compare November 16, 2022 11:22
@niluxv
Copy link
Contributor Author

niluxv commented Nov 16, 2022

Thank you for the suggestions, and sorry for the delay! Addressed them now.

@pinkforest pinkforest merged commit 610c537 into rustsec:main Nov 22, 2022
@pinkforest pinkforest removed the Propose-Merge Propose-Merge label Nov 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Unsound Informational / Unsound
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants