Función y devolver puntero

[guzzano]

Buenas, ya siento que abuso un poco del foro, hago preguntas a las tantas. La cosa es, tengo una función, un ejemplo hipotético y no real a lo que quiero hacer aquí:

Código C:

int main (void)
{
    char *nombre = char *depends = depends_parser();
}
 
char *mi_nombre()
{
    char *nombre;
    nombre = malloc(sizeof(char)*strlen("Beto")+1);
    if (nombre == NULL)
    {
        exit(EXIT_FAILURE);
    }
    
    sprintf(nombre, "Beto");
 
    return nombre;
}
 

Todo perfecto, el problema es que la función esta retornando un puntero, si lo dejo así funcionara perfecto, pero no estoy haciéndole free() a nombre, entonces queda el espacio sin liberarse. ¿En este caso, como haría?

Saludos y gracias de ante mano.

[Heimish2000]

Supongo que cuando llames a esa función recoges el valor en una variable de tipo char*, esa es la que debes liberar.

[guzzano]

Iniciado por Heimish2000

Supongo que cuando llames a esa función recoges el valor en una variable de tipo char*, esa es la que debes liberar.

Gracias por la respuesta. Curiosamente, funciona perfecto, pero el debuger me dice que todavía esta bloqueado ese espacio en la memoria, así libere al puntero que esta recibiendo el retorno de la función.

PD; Me equivoque en el código de arriba, es:

Código C:

char *nombre = depends_parser();
 

[Malenko]

Efectivamente, lo que estas haciendo es reservar un bloque de memoria en la linia 9 y guardas la dirección del bloque en la variable nombre. Esta dirección es la que devuelve la función. Si quieres liberar esa memoria bastaría con algo así:

Código:

// Guardamos la dirección de memoria en la varibale "name"
char *name = mi_nombre();

// Trabajamos con la variable "name"
// ...

// liberamos la memoria
free(name);

[guzzano]

Listo! muchisimas gracias, fue un error mio con los datos que le di el debugger. Ahora, el problema como tal es hacer esto:

Código C:

char *nombre = mi_nombre();
puts(mi_nombre)
 

Según leí es muy mala idea, de paso el debugger (ahora si, con los parametros bien, me manda error) preguntaré, ¿por qué es malo esto?

Los datos del debugger:

Código HTML:

==7991== HEAP SUMMARY:
==7991==     in use at exit: 0 bytes in 0 blocks
==7991==   total heap usage: 16 allocs, 16 frees, 197,588 bytes allocated
==7991== 
==7991== All heap blocks were freed -- no leaks are possible
==7991== 
==7991== Use --track-origins=yes to see where uninitialised values come from
==7991== ERROR SUMMARY: 2 errors from 2 contexts (suppressed: 0 from 0)
==7991== 
==7991== 1 errors in context 1 of 2:
==7991== Conditional jump or move depends on uninitialised value(s)
==7991==    at 0x40BB529: ??? (in /lib/libc-2.16.so)
==7991==    by 0x80488CF: main (in /home/guzzano/pkgrmd/pkgrmd)
==7991== 
==7991== 
==7991== 1 errors in context 2 of 2:
==7991== Invalid read of size 4
==7991==    at 0x40BB51B: ??? (in /lib/libc-2.16.so)
==7991==    by 0x80488CF: main (in /home/guzzano/pkgrmd/pkgrmd)
==7991==  Address 0x42176ec is 4 bytes inside a block of size 6 alloc'd
==7991==    at 0x402A66D: malloc (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==7991==    by 0x8048BE4: search_package (in /home/guzzano/pkgrmd/pkgrmd)
==7991==    by 0x80488CF: main (in /home/guzzano/pkgrmd/pkgrmd)
==7991== 
==7991== ERROR SUMMARY: 2 errors from 2 contexts (suppressed: 0 from 0)
 

Funciona perfecto, puts envia la cadena perfectamente a la pantalla, pero en lo que es el debugger me dice que tengo unos errores, ¿por qué pasa tal cosa?


Muchas gracias a todos por las respuestas,
saludos.

[Malenko]

Somos programadores, no adivinos. Sin poner el código del programa poco vamos a poder decirte...

[guzzano]

Disculpa, igual esta de más lo de adivino. El código es el mismo.

Código C:

int main (void)
{
    char *nombre = depends_parser();
    puts(nombre);
}
 
char *mi_nombre()
{
    char *nombre;
    nombre = malloc(sizeof(char)*strlen("Beto")+1);
    if (nombre == NULL)
    {
        exit(EXIT_FAILURE);
    }
    
    sprintf(nombre, "Beto");
 
    return nombre;
}
 

Saludos y disculpen.

[cesar_casla]

Me perdi un poco, el problema lo tenes en el ultimo codigo que subiste??? o al querer hacer:

Código C:

char *nombre = mi_nombre();
    puts(mi_nombre);  /* <--- ojo le estas pasando la direccion de una funcion */
 

??

Saludos

PD: faltaria hacer un free(nombre);