Fix ml-service memory leak #672
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
NOTE: This does not fix the client's reported bug.
Fixes a memory leak by ensuring proper pipeline destruction and state management in
ml-api-service-agent-client.The previous implementation could lead to memory leaks on the daemon side when
ml_service_pipeline_launch()and_ml_service_pipeline_release_internal()were called repeatedly from multiple threads or processes. This was due toserver->idnot being correctly initialized to indicate an unlaunched state and not properly handling pipeline ID 0 during destruction, or not resetting the ID after destruction, leading to orphaned pipelines.