Skip to content

[receiver/mysql] fix oob overflow for Innodb_buffer_pool_pages_misc #36914

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 2 commits into from
Jan 6, 2025

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented Dec 20, 2024

Description

Ignore the value returned by the stat Innodb_buffer_pool_pages_misc if out-of-bounds.

Link to tracking issue

Fixes #35495

Testing

Add an integration test.

@mx-psi
Copy link
Member

mx-psi commented Dec 23, 2024

Error: mysqlreceiver/scraper_test.go:128:62: unnecessary leading newline (whitespace)

@djaglowski djaglowski merged commit cce2a3e into open-telemetry:main Jan 6, 2025
161 checks passed
@github-actions github-actions bot added this to the next release milestone Jan 6, 2025
AkhigbeEromo pushed a commit to sematext/opentelemetry-collector-contrib that referenced this pull request Jan 13, 2025
…pen-telemetry#36914)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Ignore the value returned by the stat Innodb_buffer_pool_pages_misc if
out-of-bounds.

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#35495

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Add an integration test.
zeck-ops pushed a commit to zeck-ops/opentelemetry-collector-contrib that referenced this pull request Apr 23, 2025
…pen-telemetry#36914)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Ignore the value returned by the stat Innodb_buffer_pool_pages_misc if
out-of-bounds.

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#35495

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Add an integration test.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[receiver/mysqlreceiver] Mysql receiver - Uint64 to Int64 overflow
3 participants