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.

Sign up for GitHub

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

Jump to bottom

Upgrade zlib to 1.3 #45

Merged
merged 2 commits into from
Nov 16, 2023
Merged

Upgrade zlib to 1.3 #45

merged 2 commits into from
Nov 16, 2023

Conversation

5abeel
Copy link
Collaborator

@5abeel 5abeel commented Nov 16, 2023

Upgrading zlib to v1.3 to fix CVE identified in 1.2.13

@5abeel
Upgrade zlib to 1.3
be8071b 
Signed-off-by: Sabeel Ansari <sabeel.ansari@intel.com>
@5abeel
Updating components.json with tag version and SHA
8a02bfc 
Signed-off-by: Sabeel Ansari <sabeel.ansari@intel.com>
@ffoulkes
Copy link
Collaborator

👍for including the CVE link in the DL description.

ffoulkes
ffoulkes approved these changes Nov 16, 2023
View reviewed changes
Copy link
Collaborator

@ffoulkes ffoulkes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please update the components.json file, too, and include it in the commit?

bharticemk
bharticemk approved these changes Nov 16, 2023
View reviewed changes
Copy link
Contributor

@bharticemk bharticemk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@5abeel 5abeel merged commit f907b2e into main Nov 16, 2023
2 checks passed
@5abeel
Copy link
Collaborator Author

5abeel commented Nov 16, 2023

Could you please update the components.json file, too, and include it in the commit?

Updated components.json and merged. Thanks.

@5abeel 5abeel deleted the upgrade_zlib branch November 16, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bharticemk bharticemk bharticemk approved these changes

@ffoulkes ffoulkes ffoulkes approved these changes

@nupuruttarwar nupuruttarwar Awaiting requested review from nupuruttarwar

@n-sandeep n-sandeep Awaiting requested review from n-sandeep n-sandeep is a code owner

@vsureshkumarp vsureshkumarp Awaiting requested review from vsureshkumarp vsureshkumarp is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@5abeel @ffoulkes @bharticemk