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

ucx utils: fix init bug in am test #20

Merged
merged 1 commit into from
Mar 13, 2025
Merged

Conversation

tstamler
Copy link
Contributor

Found by the vulnerability analysis

@tstamler tstamler requested review from artpol84 and mkhazraee March 13, 2025 17:06
@tstamler tstamler self-assigned this Mar 13, 2025
@tstamler tstamler force-pushed the tstamler/ucx_am_test_fix branch from f297703 to 3501650 Compare March 13, 2025 17:08
@tstamler tstamler force-pushed the tstamler/ucx_am_test_fix branch from 3501650 to 7aff645 Compare March 13, 2025 17:09
@tstamler tstamler force-pushed the tstamler/ucx_am_test_fix branch from 7aff645 to de576e5 Compare March 13, 2025 17:35
@mkhazraee mkhazraee force-pushed the tstamler/ucx_am_test_fix branch from de576e5 to d27690a Compare March 13, 2025 18:08
@mkhazraee mkhazraee force-pushed the tstamler/ucx_am_test_fix branch from d27690a to 135584c Compare March 13, 2025 18:25
@mkhazraee mkhazraee force-pushed the tstamler/ucx_am_test_fix branch from 135584c to 40755e2 Compare March 13, 2025 18:37
@mkhazraee mkhazraee force-pushed the tstamler/ucx_am_test_fix branch from 40755e2 to df40dfe Compare March 13, 2025 18:38
@mkhazraee mkhazraee force-pushed the tstamler/ucx_am_test_fix branch from df40dfe to b03b0f9 Compare March 13, 2025 18:40
* Fix init bug in am test of ucx utils
* Add missing free/delete for memories allocated.
* Fixed a type conversion.
* Styling fixes.

Signed-off-by: Timothy Stamler <[email protected]>
Signed-off-by: Moein Khazraee <[email protected]>
@artpol84 artpol84 self-requested a review March 13, 2025 20:12
@mkhazraee mkhazraee merged commit e92f335 into main Mar 13, 2025
6 checks passed
@mkhazraee mkhazraee deleted the tstamler/ucx_am_test_fix branch March 13, 2025 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants