Помогите с простой реализацией оболочки Linux

Я реализую простую версию оболочки Linux в c.

Я успешно написал синтаксический анализатор, но у меня возникли проблемы с разветвлением дочернего процесса. Однако я думаю, что проблема связана с массивами, указателями и тому подобным, потому что я только начал C с этим проектом и еще не очень хорошо разбираюсь в них.

Я получаю ошибку сегментации и не знаю, откуда. Любая помощь приветствуется.

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <string.h>
#include <sys/wait.h>
#include <sys/types.h>

#define MAX_COMMAND_LENGTH 250
#define MAX_ARG_LENGTH 250

typedef enum {false, true} bool;

typedef struct {
    char **arg;     
    char *infile;   
    char *outfile;  
    int background; 
} Command_Info;

int parse_cmd(char *cmd_line, Command_Info *cmd_info)
{
    char *arg;
    char *args[MAX_ARG_LENGTH]; 

    int i = 0;
    arg = strtok(cmd_line, " ");
    while (arg != NULL) {
        args[i] = arg;
        arg = strtok(NULL, " ");
        i++;
    }

    int num_elems = i;
    if (num_elems == 0)
        return -1;

    cmd_info->infile = NULL;
    cmd_info->outfile = NULL;
    cmd_info->background = 0;

    int iarg = 0;
    for (i = 0; i < num_elems-1; i++)
    {                   
        if (!strcmp(args[i], "<"))
        {
            if (args[i+1] != NULL)
                cmd_info->infile = args[++i];
            else
                return -1;                      
        }

        else if (!strcmp(args[i], ">"))
        {
            if (args[i+1] != NULL)
                cmd_info->outfile = args[++i];
            else 
                return -1;                          
        }

        else
            cmd_info->arg[iarg++] = args[i];
    }

    if (!strcmp(args[i], "&"))
        cmd_info->background = true;
    else
        cmd_info->arg[iarg++] = args[i];

    cmd_info->arg[iarg] = NULL; 

    return 0;   
}


void print_cmd(Command_Info *cmd_info)
{
    int i;  
    for (i = 0; cmd_info->arg[i] != NULL; i++)
        printf("arg[%d]=\"%s\"\n", i, cmd_info->arg[i]);
    printf("arg[%d]=\"%s\"\n", i, cmd_info->arg[i]);    
    printf("infile=\"%s\"\n", cmd_info->infile);
    printf("outfile=\"%s\"\n", cmd_info->outfile);
    printf("background=\"%d\"\n", cmd_info->background);
}

void get_cmd(char* str)
{
    fgets(str, MAX_COMMAND_LENGTH, stdin);
    str[strlen(str)-1] = '\0'; //apaga o '\n' do fim
}

pid_t exec_simple(Command_Info *cmd_info)

{
    pid_t pid = fork();


    if (pid < 0)
    {
        perror("Fork Error");
        return -1;
    }



    if (pid == 0)

    {
        execvp(cmd_info->arg[0], cmd_info->arg);

        perror(cmd_info->arg[0]);
        exit(1);
    }


    return pid;

}

int main(int argc, char* argv[])
{

    while (true)
    {
        char cmd_line[MAX_COMMAND_LENGTH];
        Command_Info cmd_info;

        printf(">>> ");

        get_cmd(cmd_line);

        if ( (parse_cmd(cmd_line, &cmd_info) == -1) )   
            return -1;

        parse_cmd(cmd_line, &cmd_info);



        if (!strcmp(cmd_info.arg[0], "exit"))
            exit(0);

        pid_t pid = exec_simple(&cmd_info);

        waitpid(pid, NULL, 0);  
    }

    return 0;
} 

Спасибо.


person nunos    schedule 15.04.2010    source источник
comment
Вы можете запустить свою программу с помощью gdb, и когда она выйдет из строя, вы можете ввести bt, чтобы получить обратную трассировку стека. Это должно помочь вам определить, где именно происходит сбой.   -  person i_am_jorf    schedule 15.04.2010
comment
стр[стрлен(стр)-1] = '\0'; небезопасно, что если str = ;   -  person Ernelli    schedule 15.04.2010
comment
Действительно, это идеальная задача для gdb... и, что более важно, это идеальная задача для ознакомления с gdb. Возможность использовать gdb для отслеживания местоположения ошибок сегментации будет одним из самых важных навыков отладки, особенно для новых разработчиков C.   -  person Rakis    schedule 15.04.2010
comment
Пожалуйста, также рассмотрите возможность использования strtok_r() по умолчанию, это избавит вас от многих головных болей, если/когда вы обрабатываете команды в потоках. Требуется всего один дополнительный указатель для strtok(), чтобы сохранить свою работу в локальном хранилище потока.   -  person Tim Post♦    schedule 16.04.2010


Ответы (1)


Проблема с cmd_info->arg, который объявлен как char **arg, и вы никогда не выделяете для него память.

Таким образом, когда вы пытаетесь получить к нему доступ для сохранения таких аргументов, как этот cmd_info->arg[iarg++] = args[i], вы разыменовываете неинициализированный указатель, вызывая ошибку сегментации.

Решением было бы изменить структуру Command_Info, чтобы объявить arg следующим образом:

char *arg[MAX_ARG_LENGTH];
person Laurent Parenteau    schedule 15.04.2010