godbach兄弟及各位大牛帮忙看个netfilter加hook不能连接FTP服务器的问题。

Question

先放代码

1. int function1(struct sk_buff *pskb)
2. {
3.     int             max_headroom;
4.     struct iphdr   *iph;
5.     struct sk_buff *skb     = pskb;   
6.     struct tcphdr  *tcphdr  = NULL;
7.     int             iph_len = 0;
8.     unsigned short  wBakCheck = 0;
9.     u32             dwCSum = 0;
11.     max_headroom = sizeof(struct iphdr) + sizeof(struct udphdr) + sizeof(struct gtpuhdr);
13.     iph = ip_hdr(skb);      //2.6.18为 iph = skb->nh.iph;
15.     if (iph->protocol  == IPPROTO_TCP)
16.     {
17.         tcphdr = (struct tcphdr *)(skb->data + iph->ihl * 4);      
18.         
19.         iph_len = ip_hdrlen(skb);
20.         wBakCheck = tcphdr->check;
22.         tcphdr->check = 0;
23.         dwCSum = skb->csum;
24.         
25.         skb->csum = csum_partial((unsigned char *)tcphdr, ntohs(iph->tot_len) - iph_len, 0);
27.         tcphdr->check  = csum_tcpudp_magic(iph->saddr,
28.                         iph->daddr,
29.                         ntohs(iph->tot_len) - iph_len,
30.                         iph->protocol,
31.                         skb->csum);
34.         skb->csum = dwCSum;
35.     }
36. }

复制代码

//以下是加HOOK的代码

static struct nf_hook_ops   gglocal_ops =
{
        .hook = gg_localout,
        .pf = PF_INET,
        .hooknum = NF_IP_LOCAL_OUT,
        .priority = NF_IP_PRI_RAW,
};

上面这段代码是我写的模块中的一段代码。

现在发现加入模块后运行可以ping通，但不能连接FTP服务器，请问上面那段代码有什么问题。

备注：项目是将一个以前在2.6.18内核下跑的模块移植到2.6.28下，在18内核下代码运行没有问题，在28下会出现上述情况，请大家帮帮忙，多谢！

Answer 1

这段代码只是重新进行了 re-checksum
所以我怀疑还是这里出的问题

从你的代码看，没有找到针对 IP 层的 re-chksum
你不妨用 wireshark 抓包看一下，如果发现 chksum 有问题，wireshark 那个抓包软件会立即用红色标注的

Answer 2

2 楼兄弟厉害，抓包确实显示的是chksum 有问题，颜色是红色。

那请问怎么改呢？

还有为什么上面的代码在18内核下就能运行正确呢？

Answer 3

回复 3# 5311
如果你的修改确实涉及到了 IP 头部，那就需要重新计算 IP 的校验和。内核中是有 API 可以调用的，应该是 ip_fast_csum

Answer 4

回复 1# 5311

备注：项目是将一个以前在2.6.18内核下跑的模块移植到2.6.28下，在18内核下代码运行没有问题，在28下会出现上述情况，请大家帮帮忙，多谢！

2.6.18 和 2.6.28 这两个版本之间，hook 点的宏定义发生了变化，你检查一下你的使用是否正确

Answer 5

hook点的钩子函数原型是发生了变化，这个我已经改过了。

Answer 6

回复 6# 5311

hook 点的宏定义你没有修改
2.6.25 以后是这样的

enum nf_inet_hooks {
        NF_INET_PRE_ROUTING,
        NF_INET_LOCAL_IN,
        NF_INET_FORWARD,
        NF_INET_LOCAL_OUT,
        NF_INET_POST_ROUTING,
        NF_INET_NUMHOOKS
};

从效果上，不改可能也没关系，因为其对应的值没有变化。

但是从兼容性上来讲，还是应该使用新版本的