将字符串转换为 unsigned char[]

Question

我目前有一个数据包，如下所示：

struct Packet {
    unsigned short sequenceNumber;
    unsigned short length;
    unsigned char control;
    unsigned char ack;
    unsigned short crc;
    unsigned char data[];
    Packet copy(const Packet& aPacket) {
        sequenceNumber = aPacket.sequenceNumber;
        length = aPacket.length;
        control= aPacket.control;
        ack = aPacket.ack;
        crc = aPacket.crc;
        memcpy (data, aPacket.data, aPacket.length);
    }
};

该数据包被转换为字符串进行加密，然后需要从其解密的字符串形式恢复为数据包。我可以对除 unsigned char data[] 之外的所有变量执行此操作。我尝试了以下操作，但没有成功：

string data = thePack.substr(pos, thePack.length()-pos);
    unsigned char * cData = new unsigned char[data.length()];
    strcpy((char *)cData, data.c_str());
    memcpy(p.data, cData, data.length());

其中 data 是要复制到 unsigned char [] 中的数据的字符串表示形式，p 是数据包。

这给出了 valgrind 的以下内容：

==16851== Invalid write of size 1

==16851==    at 0x4A082E7: strcpy (mc_replace_strmem.c:303)

尽管它引用 strcpy 作为源代码，但只需注释掉 memcpy 行即可编译并正常运行。

我还尝试用 strcpy 替换 memcpy ，结果相同。有什么想法吗？我觉得这可能是由于数据可能尚未初始化并且没有为其分配任何内存，但我认为 memcpy 会解决这个问题。

Answer 1

您尚未指定 data 数组的大小。

unsigned char data[];

这是合法的，但很难使用。 data 数组将遵循内存中 Packet 结构的其余部分，但编译器不知道要为其分配多少空间。因此，您必须自己分配额外的空间：

size_t datalen = thePack.length()-pos;
void* pbuffer = malloc( sizeof (Packet) + datalen + 1 );
Packet* p = new (pbuffer) Packet;
memcpy(p.data, &thePack[pos], datalen);
p.data[datelen] = 0;

让编译器决定 Packet 应该有多大是行不通的，可以使用 new Packet 或局部变量<代码>数据包p；。这最终将导致没有为数据保留空间。不，memcpy 不分配内存。

一个更简洁的解决方案是使用 std::vector 来存储可变大小的数据数组。

Answer 2

您分配的 char[] 太小了 - 您必须在末尾为 NULL 字节留出空间：

unsigned char * cData = new unsigned char[data.length() + 1];

使用 strcpy 版本复制字符串，所以 NULL 字节被正确复制。虽然没有 +1 也可能运行正常，但不能保证，有时可能会崩溃。