Punteros en funciones c

[manolo669]

TEMA, tengo lo siguiente:

char cad1[20] = "pepe";
char cad2[20] = "pepe2";

char **com = (char **)malloc (sizeof(char *)*2);

com[0] = (char *)malloc (sizeof(char)*(strlen(cad1)+1));
com[1] = (char *)malloc (sizeof(char)*(strlen(cad2)+1));

limpiarTabla(&com,1);


funcion limpiarTabla:

void limpiarTabla(char ***tabla,pos){

for ( ; pos >= 0 ; pos--) free ( *(tabla[pos]) );

free(*tabla);

}


PROBLEMA :
El caso es que al intentar limpiar la posicion 1 de la tabla, la funcion peta por violacion del segmento.

Mirando la direccion de los punbteros con [ printf ("%p",*(tabla+pos)); ]
me doy cuenta que el puntero en la direccion 1 de la tabla esta a nulo. ????????

Haber si alguien me podria explicar este caso, que escapa a mi saber.

GRACIAS.

[define]

com[0] = (char *)malloc (sizeof(char)*(strlen(cad1)+1));

no le aplices malloc a una posicion especifica.... porque eso es lo que produce la violacion de segmento, debes aplicarlo al puntero... (al menos es lo que recuerdo.. tengo un poco espeso el C, pero vamos :) estoy casi sguro de que es eso!, espero haberte sido de ayuda)

[manolo669]

Ja tengo la solucion:

El problema esta aquí:

limpiarTabla(&com,1); // No se puede pasar la direccion de u puntero, ya que el puntero ya es una direccion.

Ade+ hay que dajar los punteros liverados a NULL.

Entronces el tema quedaria así:

limpiarTabla(com,1);


FUNCION LIMPIAR TABLA:

void limpiarTabla(char **tabla,pos){

for ( ; pos >= 0 ; pos--){
free ( tabla[pos] );
tabla[pos] = NULL;
}

free (tabla);
tabla = NULL;
}

GARCIAS POR EL INTERES.

[PCKing]

Buenasss

Solo una cosita, en tu funcion limpiarTabla, el ultimo tabla = NULL estaría de más, porque lo que recibe una funcion es una copia del char **, el cambio no se va a mantener, esa variable automatica desaparece una vez que la funcion retorne, si queres que tabla quede seteada en NULL deberias hacer algo asi:

Código HTML:

void limpiarTabla(char *** tabla,pos)
{
    for ( ; pos >= 0 ; pos--)
    {
        free ( (*tabla)[pos] );
        (*tabla)[pos] = NULL;
    }

    free (*tabla);
    *tabla = NULL;
}

Saludos

[manolo669]

Buenas:
Con respecto a lo que comntas PCKing.

Creo que no llevas razon , ya que no se pasa una copia de la variable tabla, sinó que es un puntero a una direccion de memoria donde empieza la cadena de caracteres (en es te caso).

Entonces este tipo de variables no les hace una copia, ya que se le pasa la direccion que ocupa en la memoria, entonces simplemente hay qye liberar esa direccion de memoria i listo.

[PCKing]

Para modificar una variable tenes que enviar su direccion de memoria, si queres modificar tabla tenes que mandar &tabla en la invocación por ejemplo, hacé la prueba...

[manolo669]

Yo creia que era así, como indico en mi primer posteo, pero no me funcionaba correctamente, ya que me perdía la dirección de memoria y me intentaba liberar una dirección nula.

[PCKing]

Si, pero esa es porque pusiste mal los parentesis, nuestras dos funciones no son iguales fijate...

Código HTML:

void limpiarTabla(char ***tabla,pos){

for ( ; pos >= 0 ; pos--) free ( *(tabla[pos]) );

free(*tabla);

}

void limpiarTabla(char *** tabla,pos)
{
    for ( ; pos >= 0 ; pos--)
    {
        free ( (*tabla)[pos] );
        (*tabla)[pos] = NULL;
    }

    free (*tabla);
    *tabla = NULL;
}

free ( *(tabla[pos]) ); y la mia free ( (*tabla)[pos] );

Por eso te falla.

Saludos