8000 [collectd 6] fix: df plugin: Fix an array out of bounds crash. by octo · Pull Request #4252 · collectd/collectd · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[collectd 6] fix: df plugin: Fix an array out of bounds crash. #4252

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

Merged
merged 1 commit into from
Jan 24, 2024

Conversation

octo
Copy link
Member
@octo octo commented Jan 24, 2024

The second argument of the for loop always evaluated to true, causing the code to read past the end of the array.

ChangeLog: df plugin: An out of bounds read, causing segmentation faults, has been fixed.

The second argument of the `for` loop always evaluated to true, causing the
code to read past the end of the array.

ChangeLog: df plugin: An out of bounds read, causing segmentation faults, has been fixed.
@octo octo requested a review from a team as a code owner January 24, 2024 14:11
@collectd-bot collectd-bot added the Fix A pull request fixing a bug label Jan 24, 2024
@collectd-bot collectd-bot added this to the 6.0 milestone Jan 24, 2024
@octo octo added the Automerge Labels PRs to be merged by a bot once approved label Jan 24, 2024
Copy link
Contributor
@eero-t eero-t left a comment

Choose a reason for hiding this comment

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

Approved. Commit message could include:
Fixes: c6f5e418674f49

PS. sorry for missing this!

@collectd-bot collectd-bot merged commit 18b7775 into collectd:collectd-6.0 Jan 24, 2024
@octo octo deleted the 6/df branch January 25, 2024 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Automerge Labels PRs to be merged by a bot once approved Fix A pull request fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0