Skip to main content
2 de 2
Mejorando gramática y formato. Indentando código.

Problemas con punteros y violación de segmento

Estoy teniendo problemas con un proyecto que consiste en simular un minishell. El usuario teclea un comando y el programa de momento separa con strtok() la línea introducida y mira a ver si es un comando interno como cd, export, jobs y source, y de momento da una explicación de lo que hace ese comando, que aún no he acabado de completar.

Cuando lo ejecuto en CodeBlocks el programa funciona perfectamente, pero al compilarlo y ejectutarlo por consola de Linux me de violación de segmento.

Al parecer, al compilar, reporta avisos en una serie de líneas detalladas más abajo. Estos avisos tienen que ver con el manejo de punteros y he estado leyendo en foros y tutoriales pero no doy con la solución.

Nota: Los errores y los warnings están comentados en el código.

#include <stdio.h>
#include <stdlib.h>
#define PROMPT "$"

int parse_args(char **args, char *line){
    char *token;
    int n=0;

    //Aquí da warning: assignment makes pointer from integer without a cast
    token=strtok(line," "); 

    while(token!=NULL){
        printf("token%i: %s\n",n,token);
        *args=token;
        n++;
        *args++;

        //Aquí da warning: assignment makes pointer from integer without a cast
        token=strtok(NULL," ");
    }
    printf("token%i: %s\n",n,token);
    *args=token;
    return n;
}

//Aquí me dice que espera un char* pero que recibe un char**
char *read_line(char *line){ 
    printf("%s ",PROMPT);

        //Aquí da warning: assignment makes pointer from integer without a cast
        *line = malloc(sizeof(500));
    fgets(line,500,stdin);
    return line;
}

//Aquí me dice que espera un char* pero que recibe un char**
int execute_line(char *line){
    char**args;
    parse_args(args,line);
    check_internal(args);
    return 0;
}

int check_internal(char **args){
    if( strcmp(*args, "cd")==0 ){
        internal_cd();
    } else{
        if( strcmp(*args, "export")==0 ){
            internal_export();
        }else{
            if( strcmp(*args, "source")==0 ){
                internal_source();
            }else{
                if( strcmp(*args, "jobs")==0 ){
                    internal_jobs();
                }else{
                    printf("%s","pasa los ifelse\n");
                    return 0;
                }
            }
        }
    }
}

int internal_cd(char **args){
    printf("%s","cambio de directorio\n");
    return 1;
}

int internal_export(char **args) {
    printf("%s","éste es el export\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;
    while(read_line(&line)){//warning: tipo de puntero incompatible
        execute_line(&line);//warning: tipo de puntero incompatible
    }
}