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 MSan report in AES functions #19397

Merged
merged 2 commits into from
Jan 23, 2021
Merged

Fix MSan report in AES functions #19397

merged 2 commits into from
Jan 23, 2021

Conversation

alexey-milovidov
Copy link
Member

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Uninitialized memory read was possible in encrypt/decrypt functions if empty string was passed as IV. This closes #19391.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Jan 22, 2021
@alexey-milovidov alexey-milovidov self-assigned this Jan 23, 2021
@alexey-milovidov alexey-milovidov merged commit feebf64 into master Jan 23, 2021
@alexey-milovidov alexey-milovidov deleted the aes-msan branch January 23, 2021 01:47
alexey-milovidov added a commit that referenced this pull request Jan 23, 2021
Backport #19397 to 21.1: Fix MSan report in AES functions
vitlibar pushed a commit that referenced this pull request Feb 1, 2021
…73018039909fd800efbae273e1152f

Cherry pick #19397 to 20.12: Fix MSan report in AES functions
vitlibar pushed a commit that referenced this pull request Feb 1, 2021
…73018039909fd800efbae273e1152f

Cherry pick #19397 to 20.11: Fix MSan report in AES functions
alexey-milovidov added a commit that referenced this pull request Feb 19, 2021
Backport #19397 to 20.12: Fix MSan report in AES functions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-bugfix Pull request with bugfix, not backported by default
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MSan report in AES functions
2 participants