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: F3: fix buffer overflow identified by security scanner #3332

Merged
merged 7 commits into from
Jan 17, 2025

Conversation

aaron-kasten
Copy link
Contributor

Change Overview

see: https://github.com/kanisterio/kanister/security/code-scanning/224

Pull request type

Please check the type of change your PR introduces:

  • 🐛 Bugfix

Issues

There is no Github issue associated with this PR

Test Plan

Overflow may occur on 32 bit systems due to 32 bit integer size. Testing is system dependent. Possible failures will need to be identified by static analysis

@aaron-kasten aaron-kasten force-pushed the kanx-2-kubexec-F3 branch 4 times, most recently from 78d2ba5 to 8a62402 Compare January 17, 2025 13:44
@aaron-kasten aaron-kasten force-pushed the kanx-2-kubexec-F3 branch 2 times, most recently from b7b1017 to 9419a9b Compare January 17, 2025 18:55
Signed-off-by: Aaron Alpar <[email protected]>
Signed-off-by: Aaron Alpar <[email protected]>
@mergify mergify bot merged commit 5f653cb into kanisterio:master Jan 17, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants