CODEWITHSUNDEEP
cpointersmallocfree
KiL
KiL

Reputation: 1550

Free allocated memory before return a function

I am trying to return an array using malloc in a function:

char* queueBulkDequeue(queueADT queue, unsigned int size)
{
    unsigned int i;
    char* pElements=(char*)malloc(size * sizeof(char));
    for (i=0; i<size; i++)
    {
        *(pElements+i) = queueDequeue(queue);
    }
    return pElements;
}

The problem is that I need to free it because my MCU's heap size is limited. But I want to return it so I cannot free it in the function, right?. Can I free the allocated memory outside the function (where I call the function). Is there any best practices for this? Thank you in advance!

Upvotes: 15

Views: 22263

Answers (4)

user529758
user529758

Reputation:

As the memory allocated by malloc() is on the heap and not on the stack, you can access it regardless of which function you are in. If you want to pass around malloc()'ed memory, you have no other option than freeing it from the caller. (in reference counting terms, that's what is called an ownership transfer.)

Upvotes: 11

Jay
Jay

Reputation: 24895

Ofcourse you can free the memory allocated in a function outside of that function provided you return it.

But, an alternative would be to modify your function like below, where the caller only allocates & frees the memory. This will be inline with concept of the function which allocates the memory takes responsibility for freeing the memory.

void queueBulkDequeue(queueADT queue, char *pElements, unsigned int size) 
{     
   unsigned int i;     
   for (i=0; i<size; i++)     
   {         
      *(pElements+i) = queueDequeue(queue);     
   }     
   return; 
}

//In the caller

char *pElements = malloc(size * sizeof(char));
queueBulkDequeue(queue, pElements, size);
//Use pElements
free(pElements);

Upvotes: 9

tbert
tbert

Reputation: 2097

1) Yes, you can free() the malloc'ed memory outside the function

2) No, you cannot free it inside the function and have the data passed outside the function, so you must do 1) here

3) If you're concerned about scarce memory, you need to check for failure from memory allocations always, which you fail to do here, which is then likely to lead to a segfault

Upvotes: 10

Sergey Kalinichenko
Sergey Kalinichenko

Reputation: 726489

Yes, you can free memory allocated in a function that you call outside the function; this is precisely what you need to do in this case.

Alternatives include passing a buffer and its length into the function, and returning the actual length to the caller, the way fgets does. This may not be the best alternative, because the callers would need to call your function in a loop.

Upvotes: 5

Related Questions