Thread: Memory freed at the end of function?

Hi,

I was just wondering about this function I wrote:

Code:

/*Reverses a string*/
char *strrev(char *in_string) {
    char *out_string;
    int in_char,
        out_char;

    out_string = (char *) calloc(strlen(in_string), sizeof(char));
    
    in_char = strlen(in_string) - 1;
    
    for (out_char = 0; out_char < in_char+1; out_char ++) {
        out_string[out_char] = in_string[in_char - out_char];
    }
    
    return out_string;
}

I never free the memory pointed to by out_string, but I can't, because I have to return that. Am I screwing up somewhere, or is the memory freed when the function returns?

Thanks guys. I just came to the conclusion I should be freeing the memory in the calling function, but I thought I'd check.

Methinks too! anon has a good point about optimal memory utilization. Why use a buffer as long as the string when it can be reversed in situ by using just one temporary buffer; albeit it'll make your program computationally intensive ie resource tradeoffs.