分段错误 - char 指针

Question

在下面的代码中，行：

*end = *front;

给出了分段错误。 我在此处提出了类似的问题，但我不确定这是否是因为我有两份 num 的副本。 请解释一下为什么会出现段错误。 谢谢。

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

char* getPalin(char* num);

int main()
{
    char* num = (char*)malloc(100);

    num = "123456";

    printf("%s\n", getPalin(num) );

    return 0;
}

char* getPalin(char* num)
{
    int length = strlen(num);

    if ( length % 2 == 0 )
    {
        char* front = num;
        char* end = num + strlen(num) - 1;  //pointer to end

        while( front != num + (length/2) )  //pointers not middle yet
        {
            *end = *front;

            printf("%c", *end);

            front++;
            end--;
        }
    }

    return num;
}

Answer 1

这两行：

char* num = (char*)malloc(100);
num = "123456";

具有以下效果。

第一个分配 100 个字节并将 num 设置为指向这些字节。

第二个将 num 更改为指向字符串“123456”，该字符串几乎肯定位于只读内存中。

任何更改只读内存内容的尝试都会导致分段违规。 在尝试更改字符串之前，您需要将字符串复制到 malloc 的 num 中，其中：

strcpy (num, "123456");

这是您当前应该拥有的行：

num = "123456";

Answer 2

使用

strncpy(num, "123456", 100);

而不是

num = "123456";

Answer 3

根据康斯坦丁的回答。

您已经使用 malloc 语句为 num 分配了内存。

如果没有，那么您可以逃脱：

char* num = "123456";

这将动态定义和分配内存，但它很可能被分配为常量，因此是只读的。

使用 strncpy 而不是 strcpy 复制“123456”将确保字符串 null 终止符末尾之外的任何额外空格也初始化为 null，只要您将 n 指定为 100（对于您的示例）。 否则，如果不将 malloc 分配的内存初始化为 null (memset(num, 0, 100))，则可以想象您可能会超出字符串末尾。

哦差点忘了。 建议使用 strcpy_s 或 strncpy_s 因为它们更安全，尽管对于您的代码来说这并不重要。

Answer 4

错误的原因是：

  char* num = (char*)malloc(100);

在这一行中，您已将 num 声明为指向数组的指针或指向其第一个元素的指针，而不是字符串。

 num = "123456";

这一行您使用了 num，因为您将其声明为字符串。 这是违反分段的，因此会出现分段错误。 您的代码的首选（正确）语法是：

   char num[100];
   strcpy(num,"123456"); //Even if you use num="123456"; here it would still be wrong

OR

  char* num = (char*)malloc(100);
  strcpy(num,"123456");

OR

  char num[100]={'1','2','3','4','5','6'};

其中任何一个都可以完成您的工作。