Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

make memoffset advisory informational #317

Merged
merged 1 commit into from
Jun 30, 2020
Merged

Conversation

RalfJung
Copy link
Contributor

To cause the drop of uninit memory clients had to panic in a Deref implementation. That sounds like a sufficiently "uncommon" library usage to me that this probably should be just informational.

@tarcieri
Copy link
Member

We need to update the rustsec-admin linter to allow unsound as an informational advisory type. Let me do that and cut another release (we've needed to do that for the new assign-id command anyway)

@tarcieri
Copy link
Member

@RalfJung can you rebase so it pulls in the new linter? /cc @alex

@RalfJung
Copy link
Contributor Author

Sure, done.

@tarcieri tarcieri merged commit 9cd619f into rustsec:master Jun 30, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants