I’ve got a function that returns a pointer that is malloc’d within that function, and then free that memory outside that function from within the calling function like so:
It would appear that malloc is working but it is never freeing. Eventually, after enough iterations my MallocFailed hook kicks in. I am using heap_4.c
This execution is happening within the same task. Please help!
void doSomething(){
char *str = NULL; //Local string buffer
str = mallocInside(4);
if (str != NULL)
{
// Do lots of stuff, then:
vPortFree(str);
}
}
char* mallocInside(int x)
{
char* ret = (char*) pvPortMalloc(sizeof(char) * 255);
if(ret != NULL){
Initialize response buffer to 0/null
memset(ret,0,sizeof(ret));
// Then fill this with good stuff :-)
}
return ret;
}
We have run much more stringent tests than those shown in your code with no issues, so I’m guessing something in the “do lots of stuff, then” block is clobbering the memory somehow so it doens’t get freed correct.y.
You’re correct, I found the problem. It was indeed an errant malloc Also, the ret issue was due to some copied C++ std::string code, so my bad on the example… I just hadn’t fixed it yet at some point. Do you have an article as to how to effectively user configASSERT() and describe what it does/how to use it in practice? I haven’t really seen a good description of this so far.