使用原始数据包数据和inet_ntoa()进行转换

时间:2012-02-20 04:48:13

标签: c pointers casting pcap

尝试为数据包嗅探器编写处理程序。我遇到了投射和inet_ntoa()的问题。代码如下:

uint32_t *iphdr_srcaddr = malloc(sizeof(uint32_t));
if (*packet_ethertype == ETHERTYPE_IP) { /* IPv4 */
    // copy packet data to vars
    memcpy(iphdr_srcaddr, packet+26, 4);

    // change to host-byte-order
    *iphdr_srcaddr = ntohl(*iphdr_srcaddr);

    struct in_addr *test;
    test = (struct in_addr*) iphdr_srcaddr;

    printf("uint32_t: %u\n", *iphdr_srcaddr); // Gives the correct long integer for the address
    printf("struct in_addr: %u\n", test->s_addr); // Gives the correct long integer through the cast

    char *test2;
    test2 = inet_ntoa(*test);
}

现在,如果我尝试printf("%s\n", test),我会得到SEGV。我敢肯定我正在混淆指针,价值观和做某种愚蠢的演员。运行期间收到错误:

Program received signal SIGSEGV, Segmentation fault.
0x00007ffff787ec61 in __strlen_sse2 () from /lib/libc.so.6

编译警告,我确信这指向了正确的方向,但我不确定它意味着什么以及如何解决它:

mypcap.c: In function ‘handle_sniffed’:
mypcap.c:61:15: warning: assignment makes pointer from integer without a cast [enabled by default]

这是指行test2 = inet_ntoa(*test);

1 个答案:

答案 0 :(得分:7)

警告可能表示您在inet_ntoa()的范围内没有正确的原型(因为您没有包含正确的标题)。这意味着编译器假定它的返回类型为int

当您通过test时,您也会将printf()传递给test2

另外:

  • 无需使用malloc()分配单个uint32_t;
  • 您不需要调用ntohl(),因为inet_ntoa()期望其输入按网络字节顺序排列;和
  • inet_ntoa()已过期 - inet_ntop()应在新代码中使用。

尝试:

#include <arpa/inet.h>

if (*packet_ethertype == ETHERTYPE_IP) { /* IPv4 */
    struct in_addr sin_addr;
    char straddr[INET_ADDRSTRLEN];

    memcpy(&sin_addr.s_addr, packet+26, 4);

    if (inet_ntop(AF_INET, &sin_addr, straddr, sizeof straddr))
        printf("%s\n", straddr);
    else
        perror("inet_ntop");
}