如何在C中的声明之外为字符串分配值？ 总线错误/段故障

Question

我尝试重新编码与strstr()函数等效的方法，经过几次尝试我都正确了，这里的代码是：

（我知道这样做的方法更简单，但是这次我想尝试使用第三个字符串来存储事件）

 char *my_strstr(char *s1, char *s2)
 {
   int i, j;
   char *tmp;
   i = (j = 0);
   if (s1 != '\0' && s2 != '\0')
     {
       while (s1[i] != '\0' && s2[j] != '\0')
         {
           if (s1[i] == s2[j])
             {
               tmp[j] = s1[i];
               j++;
             }
           i++;
         }
         printf("tmp = %s\n", tmp);
     }
   return (tmp);
 }


 int main()
 {
   char a[] = "test Point123";
   char b[] = "Point123";
   char *ret;

   ret = my_strstr(a, b);
   printf("ret = %s\n",ret);

   return (0);
 }

我得到了想要的输出：

tmp = Point123
ret = Point123

但是为了确保我尝试使用更长的字符串，这就是问题开始的地方。 我尝试过的字符串

char a[] = "test Point123456789";
char b[] = "Point123456789";

和我得到的输出：

tmp = Point123456?"1
ret = Point123456?"1
Abort trap: 6

对于较长的字符串，我有时会遇到段错误，有时会出现总线错误10。在其他帖子上，我发现总线错误10有时会替代macOS上的段错误（我第一次在上面进行编码，我习惯于在linux），我没有发现有关陷阱的任何信息。

无论如何，我认为它更多是我的编译器的代码问题，我想知道为什么我的代码对较小的字符串起作用而对较大的字符串不起作用，并且我读到这可能是我如何影响所使用的字符串的值，但我不明白我在哪里出错。

因此，如果有人可以告诉我我做错了什么，我将不胜感激:)

编辑

我遵循了注释中的建议，这是我修复它后的功能，现在可以正常运行了。

（再次，我知道我不需要tmp，并且还有许多其他更简单，更快速的方法来完成它-这意味着培训（显然我需要^^））

char *my_strstr(char *s1, char *s2)
{
  int i, j;
  char *tmp;
  i = (j = 0);
  tmp = malloc(sizeof(strlen(s2)));
  if (s1 != '\0' && s2 != '\0')
    {
      while (s1[i] != '\0' && s2[j] != '\0')
        {
          if (s1[i] == s2[j])
            {
              tmp[j] = s1[i];
              j++;
            }
          else
            j = 0;
          i++;
        }
    }
  return (tmp);
}

Answer 1

在您的代码中， tmp是函数my_strstr()的自动局部变量。 它没有显式初始化，因此包含不确定性值。

在代码的后面，您将直接编写

 tmp[j] = s1[i];

使用（取消引用）未初始化的tmp意味着您正在尝试访问无效的内存位置（ 还记得“不确定值”吗？ ），您可能无法从应用程序中访问该位置。 它调用未定义的行为 。

解决方案：您需要在访问（取消引用）之前为tmp分配内存。

---

编辑：

但是，按照弗拉德先生的以下评论 ，您对my_strstr()逻辑也有问题。 一旦增加了j （ 搜索字符串的索引 ）并发现两者之间不匹配，就无需将其重新设置为0 。 也许您想再看一遍并更正它。

Answer 2

您没有为tmp分配内存，因此使用tmp[j]可以写入未分配给您的某个随机内存位置。 然后您所做的事情可能会再次覆盖该位置。

字符串越长，您弄乱的存储位置就越多。

您很幸运，使用较短的字符串可以得到一些结果。