ANSI C strstr（）不适用于指针[重复]

Question

我正在尝试使用 strstr() 在字符串中查找子字符串。仅使用 char[] 有效，char* 无效，导致分段错误。

所以这个是有效的：

int main() {
    char str[] = "apple apple peach";
    char *p;

    p = strstr(str, "peach");
    if (p!= NULL) {
        strncpy(p, "apple", 5);
    }
    return 0;
}

但是这个不行：

int main() {
    char *str = "apple apple peach";
    char *p;

    p = strstr(str, "peach");
    if (p!= NULL) {
        strncpy(p, "apple", 5);
    }
    return 0;
}

这两个都不是：

int main() {
    char *str = "apple apple peach";
    char *peach = "peach";
    char *p;

    p = strstr(str, peach);
    if (p!= NULL) {
        strncpy(p, "apple", 5);
    }
    return 0;
}

这两个都不是：

int main() {
    char *str = "apple apple peach";
    char peach[] = "peach";
    char *p;

    p = strstr(str, peach);
    if (p!= NULL) {
        strncpy(p, "apple", 5);
    }
    return 0;
}

这是一个已知的错误或功能吗？

Answer 1

你的问题不是strstr，而是strncpy。在指针的情况下，您使用strncpy 尝试写入字符串字面量，它是常量。

Answer 2

string-literal 修改是不安全的！ char[] 的那个成功只是因为strncpy(p, "apple", 5); 修改了 string-literal 的复制版本，不是字符串文字本身。标准中的相关段落如下：

6.4.5，第 6 段

... 然后使用多字节字符序列初始化一个静态存储持续时间和长度刚好足以包含该序列的数组...

6.4.5，第 7 段

...如果程序尝试修改这样的数组，则行为未定义。

简而言之，string-literal 被（概念上） 替换为数组，修改它是不安全的。请注意，string-literal 不一定是 const 限定的，尽管 MSVC++ 似乎要求 string-literal 是 const char * 或 const 限定的。

Answer 3

它与strstr 无关：它按预期工作。当您尝试修改分配给字符串文字的内存时，崩溃发生在strcpy。这是未定义的行为。

解决这个问题的关键是将数据放入允许写入的内存中。您的第一个程序通过声明 str 一个数组来实现：

char str[] = "apple apple peach";

那不是UB，所以程序运行完成。另一种选择是在动态内存中分配str，如下所示：

char *str = malloc(20);
strcpy(str, "apple apple peach");
char *p;
p = strstr(str, "peach");
if (p!= NULL) {
    strncpy(p, "apple", 5);
}
free(str);