ช่วยเหลือในการใช้งาน linux shell อย่างง่าย

ฉันกำลังใช้ linux shell เวอร์ชันธรรมดาใน c

ฉันเขียน parser สำเร็จแล้ว แต่ฉันประสบปัญหาในการแยกกระบวนการย่อย อย่างไรก็ตาม ฉันคิดว่าปัญหาเกิดจากอาร์เรย์ พอยน์เตอร์ และอื่นๆ เนื่องจากเพิ่งเริ่มใช้ภาษา 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 เพื่อดู back-trace ของสแต็ก สิ่งนี้จะช่วยให้คุณระบุได้อย่างชัดเจนว่าคุณกำลังขัดข้องตรงจุดใด   -  person i_am_jorf    schedule 15.04.2010
comment
str[strlen(str)-1] = '\0'; ไม่ปลอดภัย จะเกิดอะไรขึ้นถ้า str = ;   -  person Ernelli    schedule 15.04.2010
comment
แน่นอนว่านี่เป็นงานที่สมบูรณ์แบบสำหรับ gdb... และที่สำคัญกว่านั้น มันเป็นงานที่สมบูรณ์แบบในการทำความคุ้นเคยกับ gdb ความสามารถในการใช้ gdb เพื่อติดตามตำแหน่งของ segfault ถือเป็นหนึ่งในทักษะการดีบักที่สำคัญที่สุด โดยเฉพาะอย่างยิ่งสำหรับนักพัฒนา 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