Skip to main content
replaced http://es.stackoverflow.com/ with https://es.stackoverflow.com/
Origen Enlace

¿Por qué necesitas un puntero triple? Porque si no los cambios serán locales (Esto lo he explicado ya en otras preguntas, como por ejemplo estaesta

¿Por qué necesitas un puntero triple? Porque si no los cambios serán locales (Esto lo he explicado ya en otras preguntas, como por ejemplo esta

¿Por qué necesitas un puntero triple? Porque si no los cambios serán locales (Esto lo he explicado ya en otras preguntas, como por ejemplo esta

Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75

char *read_line(char *line){
    printf("%s%s ",getenv("USER"),PROMPT);
    fflush(stdout);
    line=fgets(line,MAX_LINE,stdin);
    return line;
}

Siendo estrictos, esa función debería quedar así:

char *read_line(char *line){
    printf("%s%s ",getenv("USER"),PROMPT);
    fflush(stdout);
    return fgets(line,MAX_LINE,stdin);
}

Si no el resultado es un poco engañoso, porque al llegar al final del fichero, ´line´ apuntará a 0 pero dicho cambio de puntero no se va a reflejar fuera de la función:

void func(int* ptr)
{
  ptr = 0;
}

int main()
{
  int* ptr = 1000;
  func(ptr);

  // Pregunta: ¿Cuanto vale ptr aqui?
  // Respuesta: 100
}

Y ahora sí nos centramos en el error:

int execute_line(char *line){
    char **args; // (1)
    parse_args(args,line);
    check_internal(args);
    return 0;
}

int parse_args(char **args, char *line){
    int n=0;
    char* token;
    char delimit[]=" \t\r\n\v\f";
    token=strtok(line,delimit);
    while(token!=NULL){
        printf("token%i: %s\n",n,token);
        args=token; // (2) (3)
        n++;
        args++;
        token=strtok(NULL,delimit);
    }
    printf("token%i: %s\n",n,token);
    args=token; // (4)
    return n;
}
  • (1): args es un puntero doble y no está inicializado. ¿A dónde apunta? A cualquier parte menos a una posición de memoria debidamente inicializada
  • (2) asignas un valor a args, que no está inicializado luego estás sobreescribiendo memoria que no te pertenece.
  • (3) args es char** y token es char*. Son tipos diferentes y el resultado de esa asignación no puede ser bueno.
  • (4) ¿Qué pretendes con esta última asignación?

Si quieres almacenar una lista en args tienes varias opciones:

  1. Calculas previamente el número de elementos a almacenar y haces la reserva de memoria que corresponda. En este caso junto con args tendrás que facilitar una variable que diga cual es el número de elementos que contiene.
  2. Asumes un número máximo y haces que args tenga un tamaño fijo. En este caso también sería recomendable indicar el número efectivo de valores que almacena.

Un ejemplo con memoria dinámica:

int execute_line(char *line){
    char **args;
    int nargs;
    parse_args(&args,&nargs,line); // parse_args requiere un puntero triple
    check_internal(args);
    free(args);
    return 0;
}

int parse_args(char ***args, int* nargs char *line){
    int n=0;
    const char delimit[]=" \t\r\n\v\f";

    *nargs = 0;
    char cadTemp[MAX_LINE];
    strcpy(cadTemp,line);
    char* token=strtok(cadTemp,delimit)
    while( token )
    {
      *nargs++;
      token=strtok(NULL,delimit);
    }

    *args = malloc(*nargs * sizeof(char*);
    token=strtok(linea,delimit)
    while(token!=NULL){
        printf("token%i: %s\n",n,token);
        **args=token;
        n++;
        (*args)++;
        token=strtok(NULL,delimit);
    }
    printf("token%i: %s\n",n,token);
    return n;
}

¿Por qué necesitas un puntero triple? Porque si no los cambios serán locales (Esto lo he explicado ya en otras preguntas, como por ejemplo esta