空行段。错误（编写自己的外壳）

Question

我还在学习编写简单的 shell。

我希望这个 shell 允许空行和 cmets。

我做了一些编码，遇到了一个问题，如果我直接输入enter（空白行），它会直接seg.fault core dumped。

我不知道到底哪里出错了，因为我打印了所有内容，一切看起来都很好。我在这行中唯一怀疑的事情

if (args[0] == NULL || !(strncmp(args[0],"#",1))) {
    exitstat = 0;
}

我从基本的 split 命令函数中得到了参数。奇怪的是 cmets 工作得很好。

下面是我的函数，用于读取用户输入并拆分它们（如果我没记错的话，进行标记）。它们非常基础，因为我是从互联网教程中学习这些功能的。

char *commandInput() {
    char *command = NULL;
    ssize_t bufsize = 0;
    getline(&command, &bufsize, stdin);
    return command;
}

char **splitLine(char *command) {
    int bufsize = 64, 
    int position = 0;
    char **tokens = malloc(bufsize * sizeof(char*));
    char *token;

    token = strtok(command, DELIMITER); 
    while (token != NULL) { 
        tokens[position] = token; 
        position++;

        if (position >= bufsize) { 
            bufsize += 64;     
            tokens = realloc(tokens, bufsize * sizeof(char*));
        }
            token = strtok(NULL, DELIMITER); 
    }
    tokens[position] = NULL; 
    return tokens;
}

如果我输入空行，任何人都可以帮助我识别导致 seg.fault 的原因吗？谢谢。

编辑

我使用了调试器（经过多次尝试终于成功使用它），结果发现错误位于我没想到会导致任何问题的行（参见---UPDATE----）。

他们处理我的 commandInput 函数的方式是在 main() 函数中，我写了

int main () {
......
    char * command = NULL
    char **args;
    command = commandInput();
    args= splitLine(command);

    ------------------ UPDATE!(CAUSING ERROR IF STATEMENT) ---------------
    background = 0
    numbarguments = 0

    // Condition to check whether there is a start program running in       backgrond
    if (!(strncmp(args[numbarguments - 1], "&",1))) {
       background = 1;
       args[numbarguments - 1] = NULL;
    }

    ----------------------------------------------

    if (args[0] == NULL || !(strncmp(args[0],"#",1))) {
        exitstat = 0;
    }
    ....... //(comparing the arguments other than null)
}

因此，有关导致我出现 seg.fault 的 if 条件的任何建议。谢谢。

Answer 1

你传递给 splitline 的参数被修改了。 strtok 具有通过插入 \0 并返回指向子字符串的指针来修改它获得的字符串的效果。 strtok 返回的不是你可以直接存储以备后用的东西，而是你需要制作它的副本。

 token = strtok(command, DELIMITER); 
 while (token != NULL) 
 { 
   tokens[position] = malloc(strlen(token)+1); 
   strcpy(tokesn[position],token);
...

因此，换句话说，仅分配指向字符串的指针数组是不够的，您还需要分配空间来保存您使用 strtok 标记的字符串。

---

代码

if (!(strncmp(args[numbarguments - 1], "&",1))) {
   background = 1;
   args[numbarguments - 1] = NULL;
}

看起来不对，numberarguments 最初为 0，因此您将 args[-1] 与 "&" 进行比较，然后您分配 args[-1] = NULL，这可能会导致 seg 错误。