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

fix: avoid SIGBUS when reading non-std series segment files (#24509) (#24520) #24521

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

davidby-influx
Copy link
Contributor

Some series files which are smaller than the standard sizes cause SIGBUS in influx_inspect and influxd, because entry iteration walks onto mapped memory not backed by the the file. Avoid walking off the end of the file while iterating series entries in oddly sized files.

closes #24508

Co-authored-by: Geoffrey Wossum gwossum@influxdata.com
(cherry picked from commit 969abf3)

closes #24511

(cherry picked from commit 081f951)

closes #24512

…24520)

Some series files which are smaller than the standard
sizes cause SIGBUS in influx_inspect and influxd, because
entry iteration walks onto mapped memory not backed by the
the file.  Avoid walking off the end of the file while
iterating series entries in oddly sized files.

closes #24508

Co-authored-by: Geoffrey Wossum <gwossum@influxdata.com>
(cherry picked from commit 969abf3)

closes #24511

(cherry picked from commit 081f951)

closes #24512
@davidby-influx davidby-influx merged commit 535e869 into 2.7 Dec 20, 2023
27 checks passed
@davidby-influx davidby-influx deleted the DSB_series_sigbux_2.7 branch December 20, 2023 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tsm area/2.x OSS 2.0 related issues and PRs kind/bug team/edge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants