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

getUseridByExternalInfo can leak under LE 64-bit #467

Closed
ifakhrutdinov opened this issue Jul 31, 2024 · 3 comments
Closed

getUseridByExternalInfo can leak under LE 64-bit #467

ifakhrutdinov opened this issue Jul 31, 2024 · 3 comments
Assignees
@ifakhrutdinov
Labels
bug Something isn't working new not yet triaged

Comments

@ifakhrutdinov
Copy link
Contributor

ifakhrutdinov commented Jul 31, 2024 •
edited
Loading

Description

The getUseridByExternalInfo function in c/rsusermap.c will leak storage in case a bad function code is provided and the code is compiled for LE 64-bit. The function needs to use FREE_STRUCT31 for every exit point (which we should probably make fewer of).
@ifakhrutdinov ifakhrutdinov added bug Something isn't working new not yet triaged labels Jul 31, 2024
Gautham-coder added a commit that referenced this issue Aug 28, 2024
@Gautham-coder
Correcting changelog of PR #467
55c3fa4 
Signed-off-by: Gautham Kuppuswamy <gkuppuswamy@rocketsoftware.com>
@github-actions GitHub Actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs, but can be reopened if needed. Thank you for your contributions.

@github-actions github-actions bot added the stale-reopen-if-needed An issue closed due to inactivity. No indication of completion or validity. label Jan 28, 2025
@Gautham-coder
Copy link
Contributor

Please ignore the commit reference done earlier by me. It is not related to this ticket.

@github-actions github-actions bot removed the stale-reopen-if-needed An issue closed due to inactivity. No indication of completion or validity. label Jan 29, 2025
This was referenced Feb 10, 2025
Merged
Merged
@ifakhrutdinov
Copy link
Contributor Author

Fixed in zowe-common-c v2 and v3.

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

@ifakhrutdinov ifakhrutdinov

Labels
bug Something isn't working new not yet triaged
Projects
zOS Squad Board
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ifakhrutdinov @Gautham-coder