Skip to content

Maybe fix use-after-free inside S3 upload thread#35305

Merged
KochetovNicolai merged 1 commit intomasterfrom
try-to-fix-using-deleted-memory-tracker-inside-s3-disk
Mar 15, 2022
Merged

Maybe fix use-after-free inside S3 upload thread#35305
KochetovNicolai merged 1 commit intomasterfrom
try-to-fix-using-deleted-memory-tracker-inside-s3-disk

Conversation

@KochetovNicolai
Copy link
Copy Markdown
Member

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Mar 15, 2022
@KochetovNicolai
Copy link
Copy Markdown
Member Author

Stateless tests (release, s3 storage, actions) are ok, will merge

@KochetovNicolai KochetovNicolai merged commit cf260b8 into master Mar 15, 2022
@KochetovNicolai KochetovNicolai deleted the try-to-fix-using-deleted-memory-tracker-inside-s3-disk branch March 15, 2022 21:06
@tavplubix
Copy link
Copy Markdown
Member

The way to fix it probably was wrong, see #41011

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

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants