我必须询问我从这个代码中获得的中止（核心倾倒）错误

Question

执行此代码时，我遇到了一个segfault错误。它打印了5个数字中最大的数字，并且这些数字存储在堆中。

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

int main() {
    int *ptr = (int *)malloc(5 * sizeof(int));
    *ptr = 5;
    *(ptr + 1) = 7;
    *(ptr + 2) = 2;
    *(ptr + 3) = 9;
    *(ptr + 4) = 8;

    int *ptr_max = (int *)malloc(sizeof(int));
    *ptr_max = 0;

    for (int i = 0; i < 5; i++) {
        if (*ptr_max < *ptr) {
            *ptr_max = *ptr;
            ptr++;
        } else
            ptr++;
    }
    printf("%d\n", *ptr_max);
    free(ptr);
    free(ptr_max);
    return 0;
}

我想知道为什么我从上面的代码中遇到了这个错误。请任何人向我解释吗？

Answer 1

当您是free（） -ing ptr时，真正的问题就在于。一旦，您将其地址增加到指针跳到由malloc（）分配的下一个地址。始终确保*ptr与ptr [0]相同。因此，要解决此问题，您可以将ptr降低5，或创建复制的指针。

给出free（）的地址的示例，它们没有指向相同的内存块：

Before decrementing 0x213f2b4
After decrementing 0x213f2a0

通过5降低的原因是这两个十六进制20的值，与sizeof（int） * 5。相同

ptr -= 5;

，或者

您可以创建指针的副本，然后对复制的操作执行操作：

int *my_copied_ptr = ptr; // you don't need to free this pointer

然后，free（）他们：

free(ptr);
free(ptr_max);

现在，为了在大型代码库中进一步避免这些错误，请尝试使用[]运算符，例如：

ptr[0] = 5;
ptr[1] = 7;
ptr[2] = 2;
ptr[3] = 9;
ptr[4] = 8;