Skip to content

Memory leak in Cellular_Cleanup #119

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

Closed
renesas-adam-benson opened this issue Dec 12, 2022 · 3 comments
Closed

Memory leak in Cellular_Cleanup #119

renesas-adam-benson opened this issue Dec 12, 2022 · 3 comments
Assignees

Comments

@renesas-adam-benson
Copy link

The task that is created in cellular_pktio on a Cellular_Init call is never deleted on Cellular_Cleanup/_Cellular_PktioShutdown.

status = Platform_CreateDetachedThread( _pktioReadThread,

@chinglee-iot
Copy link
Member

Hi Adam,

Platform_CreateDetachedThread is supposed to delete the thread info after the thread function returns.
The implementation in demo can be referenced.
https://github.com/FreeRTOS/FreeRTOS/blob/391c79958f635ee5476dcf2774dab59e2b151eff/FreeRTOS-Plus/Demo/FreeRTOS_Cellular_Interface_Windows_Simulator/Common/cellular_platform.c#L64-L73

@renesas-adam-benson
Copy link
Author

@chinglee-iot thanks for the response. Missed this when looking through it, closing the issue.

@ahmedsayed2014
Copy link

Hi Adam,

I'm still encountering the same issue—after calling Cellular_Cleanup, the memory reserved for the thread remains allocated in the heap. Could you please let me know if I'm overlooking something?

Thanks!

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

No branches or pull requests

3 participants