-
Notifications
You must be signed in to change notification settings - Fork 510
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
[BUG] Potential for memory leaks #255
Comments
Hey, thank you for reading the code so carefully, and you are of course 100% correct! however, you’re also correct in your hypothesis that the intent is that this only happens once, globally. while of course we can’t see exclude the possibility of other use cases, we do expect that normally current sensing and PWM are configured once, during Initialization, and then never changed. This is also because they are closely linked to the hardware used. but in the longer term we should probably clean this up, so thank you for reporting it! |
Unless I'm mistaken I've noticed a potential for a memory leak when instantiating current sense parameters for example looking at
stm32g4_mcu.cpp I see:
Stm32CurrentSenseParams are instantiated on the heap and the pointer is assigned to a void* member in the calling class. I don't see any way that if the caller is destructed that this memory will be freed appropriately. If the top level class is only instantiated once and has a lifetime of the entire program then it's not really a issue, but if one would happen to want to use these functions dynamically then this would be bad.
Presumably the most workable solution would be for these drivers to implement a function that frees the memory, to be called in the destructor of the top level class?
something like:
and then
_deinitADCLowSide()
would be called in theLowsideCurrentSense
destructor.Describe the hardware setup
For us it is very important to know what is the hardware setup you're using in order to be able to help more directly
NA
LowSideCurrentSense (maybe others?)
STM32G431RB
NA
-Yes
IDE you are using
Platformio
The text was updated successfully, but these errors were encountered: