尝试从常量 char * 类型的指针复制数据时出现“超出内存访问”错误。为什么？

Question

以下语句引发超出内存错误的访问。我仍然不确定它为什么会那样做。谁能帮我在这里做错了什么？谢谢

    memcpy(mhp_ptr->dpair[0].dpairid, id0, 8);

这里 id0 的类型是 const char *

其中，mhp_ptr 是 mhp_t * 类型，其定义如下：-

typedef struct mhp_s 
{ 
    uint8_t  editor[16];
    uint64_t refcode;
    uint64_t error_id;
    union
    {
        uint8_t dpairtbl[16][6];
        dpairtbl_t dpair[6];
    };
}  mhp_t;

而dpairtbl_t定义如下：-

typedef struct dpairtbl_s
{ 
    char  dpairid[8];
    uint64_t dpairdata;
} dpairtbl_t;

错误：- 访问超出分配的内存 “memcpy”可以从缓冲区“id0”中读取 8 个字节

我的项目的编译器设置将警告视为错误。正常编译工作正常，但它发出一个警告，根据我的编译器中的设置被视为错误。 只是想知道为什么它对 strcpy 工作正常，但在 memcpy 的情况下发出警告。在字符串的情况下，对 memcpy 的使用是否有任何 C++ 限制？

Answer 1

将memcpy 视为内存中字节的复制：

memcpy(mhp_ptr->dpair[0].dpairid, id0, 8);

如果id0的长度小于8，memcpy也会复制不属于你的内存，如果是8个或更多字符，dpairid不会为null-正确终止。

如果您无法保证 id0 的大小正好是 8 个字节，则应将其视为字符串，以便根据终止字符确定长度：

strncpy (mhp_ptr->dpair[0].dpairid, id0, 7);
mhp_ptr->dpair[0].dpairid[7] = '\0';

Answer 2

一个潜在的问题是您正在复制到 dpairid（大小为 8），大小为 dpairtbl_t（可能为 16）

要么你的尺寸有误（使用 sizeof(dpairid)），要么

typedef struct dpairtbl_u
{ 
     char  dpairid[8];
     uint64_t dpairdata;
} dpairtbl_t;

应该是联合

typedef union dpairtbl_u
{ 
     char  dpairid[8];
     uint64_t dpairdata;
} dpairtbl_t;

Answer 3

除了其他答案之外，我没有收到任何错误并且它有效。这是我的代码：

const char* id0 = "stackoverflow question";
mhp_t* mhp_ptr = new mhp_t;

memcpy(mhp_ptr->dpair[0].dpairid, id0, sizeof(dpairtbl_t));

cout << mhp_ptr->dpair[0].dpairid << endl;

delete mhp_ptr;

输出是：

stackove

当我查看sizeof(dpairtbl_t) 值时，它是8。也许您忘记了指针mhp_ptr 的分配/初始化。以及如何初始化id0？