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

Leak in stdMem interface #1000

Open
bwhitchurch opened this issue Oct 3, 2023 · 1 comment
Open

Leak in stdMem interface #1000

bwhitchurch opened this issue Oct 3, 2023 · 1 comment
Assignees
@gvoskuilen
Labels
Bug SST-Core

Comments

@bwhitchurch
Copy link
Contributor

The CustomData member of CustomRequest and CustomResponse is prone to leaking. Should the lifetime of CustomData be tied to the CustomRequest/CustomResponse object?
@bwhitchurch bwhitchurch mentioned this issue Oct 3, 2023
Open
@gvoskuilen gvoskuilen self-assigned this Oct 3, 2023
@gvoskuilen
Copy link
Contributor

The lifetime can be tied to the CustomRequest/Response unless the CustomData object is explicitly transferred. As is, request/response don't have a way to easily specify that the CustomData ownership has been transferred so that does need fixing.

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

@gvoskuilen gvoskuilen

Labels
Bug SST-Core
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gvoskuilen @bwhitchurch