c - 使用自制的 strstr() 函数检查其他字符串中嵌入的字符串，然后将其替换为其他字符串(由用户输入)

Question

这个问题已经有答案了:

Crash or "segmentation fault" when data is copied/scanned/read to an uninitialized pointer (5 个回答)

已关闭 4 年前。

我正在编写一个程序，该程序使用指向字符串 str[] 的指针数组。它接收两个字符串 str1 和 str2 并检查 str1 是否嵌入到 str[] 中的任何字符串中。如果找到 str1，则替换与 str2 一起使用。例如:如果str1包含“mountain”并且str2包含“car”，那么str中的第二个字符串应该更改为“Move a car”。当我尝试执行这个程序时，在gcc中它崩溃了，当我尝试时在线编译它会出现段错误，我无法理解该程序出错的地方以及需要进行哪些更正。

#include<stdio.h>
#include<string.h>
char* xstrstr(char* ,char* );
int main()
{
char *str[]={
                "We will teach you how to...",
                "Move a mountain",
                "Level a building",
                "Erase the past",
                "Make a million",
                "...all through C!"
            };
char* p, *temp, *cpy;
int len,i,j;
char str1[28];
char str2[10];
printf("Enter string1\n");
scanf("%s",str1);
printf("Enter string 2\n");
scanf("%s",str2);
printf("After modifying the saved strings\n\n");
for(i=0;i<6;i++)
{
    p=xstrstr(str[i],str1);
    if(p)
    {
       /*Copy the remaining string*/
       temp=p+strlen(str1);
       strcpy(cpy,temp);
       /*Replace the old string*/
       strcpy(p,str2);
      /*Finally append the remaining part*/
       strcat(p,cpy);
       break;
    }
}
if(p==NULL)
{
    printf("No string match found\n");
    return 1;
}
for(i=0;i<6;i++)
{
    printf("%s\n",str[i]);
}
return 0;
}
char* xstrstr(char *string1,char* string2)
{
    while(*string1)
    {
        char* begin=string1;
        char* find=string2;

           while((*string1)&&(*find)&&(*string1==*find))
           {
               string1++;
               find++;
           }
           if(!(*find))
           {
               return begin;
           }
           string1=begin+1;
    }
return NULL;
}

Answer 1

两个主要问题cpy 未初始化

 strcpy(cpy,temp);   

p 指向只读字符串文字，您可以对其进行写入。

 strcpy(p,str2);