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:
- 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.
- 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