传递给方法后结构体值发生变化

Question

我正在使用 libpcap 编写数据包解析器，并在尝试从 pcap 标头读取数据包长度时遇到奇怪的行为。

下面的第一个 printf 语句打印出正确的数据包长度，而第二个语句则打印出一个数字，例如 362791。

struct pcap_pkthdr header;
pcap_t *handle;
const u_char *packet;

...

    while((packet = pcap_next(handle, &header)) != NULL) {
        printf("[%d]\n", header.len);   
        process_packet(&header, packet);
    }

...
}

void process_packet(struct pcap_pkthdr *header, const u_char *packet) {
    printf("[%d] %d bytes\n", header->ts, header->len);
}

struct pcap_pkthdr 的定义是：

struct pcap_pkthdr {
    struct timeval ts; /* time stamp */
    bpf_u_int32 caplen; /* length of portion present */
    bpf_u_int32 len; /* length this packet (off wire) */
};

的原型>pcap_next 是 u_char *pcap_next(pcap_t *p, struct pcap_pkthdr *h)

这有什么原因吗发生？

Answer 1

我认为根本问题在于代码

printf("[%d] %d bytes\n", header->ts, header->len);

中的ts< /code> 类型为 struct timeval 的数据段不仅仅是一个简单的整数数据，或者至少 sizeof(timeval) 与 sizeof(int) 不一样。这将导致 printf 在尝试获取 var 参数列表中的第二个参数时使用错误的地址偏移量。

所以我认为 printf("[%d] %d bytes\n", header->len, header->ts ); 应该正确工作，但你不能简单地交换它们。

Answer 2

你的 C 代码看起来有效，我看不出语法有任何问题。所以这可能与C本身无关。

目前唯一想到的是某些东西正在后台同时运行并撞击您的结构。我对 pcap 了解不多，所以我无法帮助您了解具体细节...

但是，当我有这样的奇怪之处时，我通常会创建一个统计函数，它打印出有关结构的所有可想象的事情...并且到处调用统计函数...这就像一个迷你的内联单元测试。

我希望你将你的代码翻新成这个并尝试一下：

void hdr_stats(struct pcap_pkthdr *header){
    unsigned int i;
    printf("address of header:  %p\n", header);

    printf("caplen:             %d\n", header->caplen);
    i = header->caplen;
    printf("caplen:             %d\n", i);

    printf("len:                %d\n", header->len);
    i = header->len;
    printf("len:                %d\n", i);

    //-------
    // add header->ts.xxxx debugging here, might give other clues...
    //-------
}

...

    while((packet = pcap_next(handle, &header)) != NULL) {
        printf("[%d]\n", header.len);   
        hdr_stats(&header);
        process_packet(&header, packet);
        hdr_stats(&header);
    }

...

void process_packet(struct pcap_pkthdr *header, const u_char *packet) {
    hdr_stats(header);
    printf("[%d]\n", header->len);
    hdr_stats(header);
}

这可能会帮助你解决你的错误。

Answer 3

...您在比较苹果和橙子吗？

/* header.len in brackets */
printf("[%d]\n", header.len); 

/* header->ts in brackets */
printf("[%d] %d bytes\n", header->ts, header->len);