Tu código contenía bastantes errores.
Esta es mi versión, manteniendo la interfaz de tus funciones intactas.
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <errno.h>
#include <unistd.h>
#define PROMPT "$"
#define MAX_LINE 512
#define DELIMS " \t\r\n\v\f"
int parse_args( char **args, char *line){
int n = 0; // Contador de palabras.
char *token = strtok( line, DELIMS );
while( token ) {
args[n] = token;
++n;
token = strtok( NULL, DELIMS );
}
return n;
}
char *read_line( char *line ){
printf( "%s%s ", getenv( "USER" ), PROMPT );
fflush( stdout );
return fgets( line, MAX_LINE, stdin );
}
int execute_line( char *line ){
char *args[MAX_LINE/2]; // NO puede haber mas de MAX_LINE / 2 ordenes.
memset( args, 0, MAX_LINE ); // Por seguridad.
parse_args( args, line );
return check_internal( args );
}
int check_internal( char **args ) {
if( !strcmp( args[0], "cd" ) ) {
return internal_cd( args );
} else
if( !strcmp(args[0], "export" ) ) {
return internal_export( args );
} else
if( !strcmp( args[0], "source" ) ) {
return internal_source();
} else
if( !strcmp(args[0], "jobs" ) ) {
return internal_jobs( );
} else
return -1;
}
int internal_cd( char **args ){
char buff[MAX_LINE];
int ret;
if( !args[1] ) {
printf( "Debe indicar una ruta.\n" );
} else {
if( chdir( args[1] ) == -1 ) {
fprintf( stderr, "Error: %s\n", strerror( errno ) );
return -1;
}
getcwd( buff, MAX_LINE );
printf( "Estas en la ruta %s\n", buff );
}
return 1;
}
int internal_export( char **args ) {
char buff[MAX_LINE];
char *old;
int ret;
if( args[1] && args[2] ) {
old = getenv( args[1] );
printf( "Valor antiguo: %s\n", old ? old : "SIN ASIGNAR" );
if( setenv( args[1], args[2], 1 ) == -1 ) {
fprintf( stderr, "Error: %s\n", strerror( errno ) );
return -1;
}
printf( "Valor nuevo: %s\n", args[2] );
} else {
fprintf( stderr, "Error. Uso: export <nombre> <valor>\n" );
}
return 1;
}
int internal_source(char **args) {
printf("%s","éste es el source\n");
return 1;
}
int internal_jobs(char **args){
printf("%s","éste es el jobs\n");
return 1;
}
void main( ) {
char line[MAX_LINE];
while( read_line( line ) ) {
execute_line( line );
}
}
No te detallo todas las modificaciones porque, sinceramente, han sido bastantes y no las recuerdo.
Como se mantiene la interfaz de tus funciones, puedes compararlas.
EDITO
El gran truco está en strtok( str, delims )
. Esta función modifica str, colocando 0 en las posiciones en las que encuentre delimitadores. No es necesario reservar memória ni nada, usamos directamente la línea leída por fgets( )
.