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

Potential memory leak in ep_file_write_event error path #107970

Open
filipnavara opened this issue Sep 18, 2024 · 1 comment
Open

Potential memory leak in ep_file_write_event error path #107970

filipnavara opened this issue Sep 18, 2024 · 1 comment
Labels
area-System.Diagnostics untriaged New issue has not been triaged by the area owner

Comments

@filipnavara
Copy link
Member

metadata_instance gets allocated here:

metadata_instance = ep_build_event_metadata_event (event_instance, metadata_id);

This error path will presumably go to ep_on_error which skips the ep_event_metadata_event_free (metadata_instance); call:

ep_raise_error_if_nok (file_save_metadata_id (file, ep_event_instance_get_ep_event (event_instance), metadata_id));

@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Sep 18, 2024
@dotnet-policy-service dotnet-policy-service bot added the untriaged New issue has not been triaged by the area owner label Sep 18, 2024
@filipnavara filipnavara added area-System.Diagnostics and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Sep 18, 2024
Copy link
Contributor

Tagging subscribers to this area: @tommcdon
See info in area-owners.md if you want to be subscribed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-System.Diagnostics untriaged New issue has not been triaged by the area owner
Projects
None yet
Development

No branches or pull requests

1 participant