Skip to content

Fix use-of-unitialized-value and crash introduced in #85063#86171

Merged
Avogar merged 1 commit intoClickHouse:masterfrom
Avogar:fix-unitialized-memory-string
Aug 26, 2025
Merged

Fix use-of-unitialized-value and crash introduced in #85063#86171
Avogar merged 1 commit intoClickHouse:masterfrom
Avogar:fix-unitialized-memory-string

Conversation

@Avogar
Copy link
Copy Markdown
Member

@Avogar Avogar commented Aug 26, 2025

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

Closes #86134
Closes #86133

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Aug 26, 2025 •
edited
Loading

Workflow [PR], commit [3db2390]

Summary: ❌

job_name test_name status info comment
Stress test (arm_asan) failure
Server died FAIL
Hung check failed, possible deadlock found (see hung_check.log) FAIL
Killed by signal (in clickhouse-server.log) FAIL
Fatal message in clickhouse-server.log (see fatal_messages.txt) FAIL
Killed by signal (output files) FAIL

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Aug 26, 2025
@nikitamikhaylov nikitamikhaylov self-assigned this Aug 26, 2025
@Avogar Avogar added this pull request to the merge queue Aug 26, 2025
Merged via the queue into ClickHouse:master with commit 9d39fcf Aug 26, 2025
120 of 122 checks passed
@Avogar Avogar deleted the fix-unitialized-memory-string branch August 26, 2025 19:09
@robot-ch-test-poll1 robot-ch-test-poll1 mentioned this pull request Aug 26, 2025
Merged
robot-ch-test-poll1 added a commit that referenced this pull request Aug 26, 2025
@robot-ch-test-poll1
Merge pull request #86234 from ClickHouse/cherrypick/25.8/86171
e00587b 
Cherry pick #86171 to 25.8: Fix use-of-unitialized-value and crash introduced in #85063
robot-clickhouse added a commit that referenced this pull request Aug 26, 2025
@robot-clickhouse
Backport #86171 to 25.8: Fix use-of-unitialized-value and crash intro…
8570e42 
…duced in #85063
@robot-ch-test-poll1 robot-ch-test-poll1 mentioned this pull request Aug 26, 2025
Merged
@robot-clickhouse robot-clickhouse added the pr-synced-to-cloud The PR is synced to the cloud repo label Aug 26, 2025
@robot-ch-test-poll robot-ch-test-poll added pr-backports-created-cloud deprecated label, NOOP pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR labels Aug 26, 2025
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Aug 26, 2025
clickhouse-gh bot added a commit that referenced this pull request Aug 26, 2025
@clickhouse-gh
Merge pull request #86235 from ClickHouse/backport/25.8/86171
1df1b7f 
Backport #86171 to 25.8: Fix use-of-unitialized-value and crash introduced in #85063
@Avogar Avogar mentioned this pull request Aug 27, 2025
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nikitamikhaylov nikitamikhaylov nikitamikhaylov approved these changes

Assignees

@nikitamikhaylov nikitamikhaylov

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-backports-created-cloud deprecated label, NOOP pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo v25.8-must-backport

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

MemorySanitizer: use-of-uninitialized-value in DB::ColumnString::sizeAt(long) MemorySanitizer: use-of-uninitialized-value in DB::getURLHost

5 participants

@Avogar @nikitamikhaylov @robot-clickhouse @robot-ch-test-poll @robot-ch-test-poll1