C函数中的分段错误[关闭]

Question

我正在尝试开发一个基本的外壳。对于那个 shell，我需要一个 C 函数来解析一个字符串。由于我是 C 新手，我尝试开发一个基本功能，但它给了我一个分段错误错误。请告诉我我错过了什么。

#include<string.h>
#include<stdio.h>


void parse(char *msg);
int main()
{
    char *msg =  "This is a message";
    parse(msg);
}

void parse(char *msg){
    char *mm;
    mm = msg;

    char *tok;
    tok = strtok(mm," ");
    while(tok == NULL){
        tok = strtok(NULL," ");
            printf("%s \n",tok);
    }
}

错误消息（运行时）

Segmentation fault (core dumped)

提前致谢

Answer 1

msg 指向一个字符串文字，并且您正在尝试修改它。在 C 中，修改字符串字面量是 undefined behaviour（实际上，编译器通常将它们放在只读内存中）。

要修复，请将msg 转换为数组：

int main()
{
    char msg[] =  "This is a message";
    parse(msg);
}

此外，您的 while 循环也存在一些问题：

1) 条件错误；
2) 第二个strtok() 调用应该出现在printf() 之后。

void parse(char *msg){
    char *mm = msg;
    char *tok = strtok(mm, " ");
    while (tok) {
        printf("%s \n",tok);
        tok = strtok(NULL," ");
    }
}

Answer 2

您不能可靠地修改字符串文字；它们通常是只读的（在您的情况下，显然是只读的）。尝试修改字符串文字会调用未定义的行为，这始终是一件坏事™！

用途：

int main(void)
{
    char msg[] = "This is a message";
    parse(msg);
}

Answer 3

你的定义：

char *msg =  "This is a message";

使msg 成为无法修改的常量字符字符串。但是strtok 修改了它。

你可能想把它改成

char *msg =  strdup("This is a message");

完成后别忘了释放指针。

Answer 4

也许不是

while(tok == NULL)

你是说

while(tok != NULL)

或者只是

while (tok) // <- because in C, conditions are always compared to 0

。但是，给您带来分段错误的是strtok 修改了传递的字符串，这就是为什么它需要一个指向char 的非const 指针（请参阅http://linux.die.net/man/3/strtok）。因此，因为您传递了一个指向不可修改的字符串文字的指针，所以您会收到分段错误（这是您的 luck；如果您运气不好，这个错误也可能会通过 QA 并投入生产)。

Answer 5

去吧…………

#include<string.h>
#include<stdio.h>


void parse(char *msg);
int main()
{
    char msg[] =  "This is a message";
    parse(msg);
}

void parse(char *msg){
    char *mm;
    mm = msg;

    char *tok;
    tok = strtok(mm," ");
    while(tok != NULL){
        printf("%s \n",tok);
    tok = strtok(NULL," ");
    }
}

Answer 6

如何将 tok==NULL 改为 tok!=NULL