calloc 并使用 c 将数据复制到内存区域

Question

我正在尝试分配一块内存，然后将数据复制到该空间中。我编写了这个简单的程序，但它没有达到我的预期。有人可以指出我的错误推理吗？

谢谢。

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

void main(void)
{
int t1 = 11;
int t2 = 22;
int *bufptr;

bufptr = calloc(2, sizeof(int));
if(bufptr == NULL)
{
    fprintf(stderr, "Out of memory, exiting\n");
    exit(1);
}

memcpy(bufptr, &t1, sizeof(int));
memcpy((bufptr+sizeof(int)), &t2, sizeof(int));

printf("bufptr11: %d\n", *bufptr);
printf("bufptr22: %d\n", *bufptr+sizeof(int));
}

它打印出来的内容如下：
bufptr11：11
bufptr22: 15（应该是 22 而不是 15）

感谢大家的帮助，但我刚刚遇到了下一个障碍！ 本练习的重点是通过 udp 向另一台主机发送一些数据。在调用 sendto() 之前，我查看了 bufptr 的内容，一切看起来都很好，并且发送似乎进行得很顺利。另一方面（我在 127.0.0.1 上运行客户端/服务器）我只收到“crap”。我调用recvfrom(s_fd、bufptr、buflen等)。我使用相同的 calloc 调用为 bufptr 分配内存。此调用返回了适量的数据，但其内容全都是垃圾！

bufptr = calloc(2, sizeof(int));
if(bufptr == NULL)
{
   fprintf(stderr, "Out of memory, exiting\n");
   exit(1);
}

buflen = 2*sizeof(int);

rc = recvfrom(sd, bufptr, buflen, 0, (struct sockaddr *)&serveraddr, &serveraddrlen);
printf("t2: %d\n", *bufptr);
printf("t3: %d\n", *(bufptr+1));

Answer 1

printf("bufptr22: %d\n", *(bufptr+sizeof(int)));
编辑

：比括号更阴险的是（我一开始显然错过了）您实际上过度执行了指针算术。编译器已将 bufptr + x 调整为 bufptr + (x * sizeof(int)) 字节。因此，当您执行 bufptr + sizeof(int) 时，您实际上超出了分配的内存。

您可以通过以下方式看到这一点：

printf("bufptr: %p\n", bufptr);
printf("bufptr + sizeof(int): %p\n", bufptr + sizeof(int));

例如，对于我（32 位机器），它输出：

bufptr: 0x876f008
bufptr + sizeof(int): 0x876f018

16 个字节分开，而我总共只分配了 8 个字节！比 bufptr[1] 更有用的提醒比看起来更有用。

这是一种可怕的错误，通常一开始不会显现出来，然后当您修改不相关的代码时就会开始导致崩溃。 valgrind 会捕获它。

Answer 2

printf("bufptr22: %d\n", *bufptr+sizeof(int));

这被解释为 (*bufptr) + sizeof(int)。这给你 11 + 4 等于你看到的 15。

printf("bufptr22: %d\n", *(bufptr+sizeof(int)));

如果你想找回 22，这就是你所追求的。

Answer 3

试试这些：

memcpy(bufptr, &t1, sizeof(int)); // copy 11 into the first element of the array.
memcpy(bufptr+1, &t2, sizeof(int)); // bufptr+1 will point to the next int of the array.

printf("bufptr11: %d\n", *bufptr);
printf("bufptr22: %d\n", *(bufptr+1)); // parenthesis needed as * has higher precedence than +

Answer 4

这

memcpy((bufptr+sizeof(int)), &t2, sizeof(int));

是错误的，因为当您添加到指针时，您添加的单位是指针所指向的类型的大小。在本例中为int。所以不需要涉及sizeof，直接使用“对象”（int）的数量即可：

memcpy(bufptr + 1, &t2, sizeof(int));

另外，最好使用指针而不是重复类型名称，如果它将来发生变化，这可以保护您：

memcpy(bufptr + 1, &t2, sizeof *bufptr);

在后一种情况下，使用目标指针的大小通常是一个好主意，因为这可以保护您（至少一点）不被覆盖源指针类型是否应更改。它们当然应该是相同的，但仍然如此。