使用std :: copy进行分段错误,memcpy有效

时间:2013-01-30 11:20:30

标签: c++ copy

我有一个奇怪的情况,使用std :: copy会导致seg故障,因为使用memcpy似乎可以正常工作。

我正在Google测试功能中创建一个PacketHeader对象。

我已经确认进入该对象的数据是有效的,所以我可能是愚蠢的,并且错过了使用std :: copy语法的明显内容。

GCC版本:4.4.6 20120305(Red Hat 4.4.6-4) GTest版本:1.6.0

还与boost :: options和libpcap

链接
TEST(SetIP, GoodVals)
{


    char errbuf[PCAP_ERRBUF_SIZE];
    pcap_t * handle = pcap_open_offline("./test_files/PCAP_Good.pcap", errbuf);

    pcap_pkthdr * header;
    const unsigned char * pkt_data;

    //Load one packet from the PCAP, doesn't matter what
    pcap_next_ex(handle, &header, &pkt_data);

    PacketHeader p(header);

}


PacketHeader::PacketHeader(const struct pcap_pkthdr * aPacketHeader) : prPacketHeader(NULL)
{
    prPacketHeader = new struct pcap_pkthdr;

    //This doesn't work
    //std::copy(aPacketHeader, aPacketHeader + sizeof(struct pcap_pkthdr), prPacketHeader);

    //This works
    memcpy(prPacketHeader, aPacketHeader, sizeof(struct pcap_pkthdr));

}

1 个答案:

答案 0 :(得分:10)

表达式

aPacketHeader + sizeof(struct pcap_pkthdr)

不符合您的预期。它实际上将sizeof(pcap_pkthdr) * sizeof(pcap_pkthdr)添加到指针基址。

请注意aPointer + somethingaPointer[something]相同。

只添加1

aPacketHeader + 1

或类型转换为char*

reinterpret_cast<char*>(aPacketHeader) + sizeof(struct pcap_pkthdr)

显而易见的解决方案当然应该使用简单的struct-to-struct复制:

*prPacketHeader = *aPacketHeader;