Skip to content

Buffer Overflow in toodee DrainCol destructor #2325

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 8, 2025
Merged

Buffer Overflow in toodee DrainCol destructor #2325

merged 2 commits into from
Sep 8, 2025

Conversation

GeorgeAndrou
Copy link
Contributor

No description provided.

djc
djc approved these changes May 26, 2025
View reviewed changes
Copy link
Contributor

@djc djc left a comment

Choose a reason for hiding this comment

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

@antonmarsden any feedback on the proposed advisory here?

@djc djc mentioned this pull request Aug 13, 2025
Closed
@GeorgeAndrou GeorgeAndrou changed the title HBOF in toodee DrainCol destructor Buffer Overflow in toodee DrainCol destructor Aug 15, 2025
@GeorgeAndrou
Copy link
Contributor Author

@djc, this pull request has been open for quite some time now (~4 months). The safety issue has been patched, so should we publish the advisory?

@djc djc merged commit 6c259b0 into rustsec:main Sep 8, 2025
1 check passed
@djc
Copy link
Contributor

djc commented Sep 8, 2025

Yup -- thanks for the ping.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@djc djc djc approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@GeorgeAndrou @djc