連接跟蹤模塊中update_alloc_size函數(shù)的問題【已解決】

jiufei19

在連接跟蹤代碼對擴展進行注冊時，有如下函數(shù)處理

static void update_alloc_size(struct nf_ct_ext_type *type)
{
        int i, j;
        struct nf_ct_ext_type *t1, *t2;
        enum nf_ct_ext_id min = 0, max = NF_CT_EXT_NUM - 1;

        /* unnecessary to update all types */
         if ((type->flags & NF_CT_EXT_F_PREALLOC) == 0) {
                min = type->id;
                max = type->id;
        }

        /* This assumes that extended areas in conntrack for the types
           whose NF_CT_EXT_F_PREALLOC bit set are allocated in order */
        for (i = min; i <= max; i++) {
                t1 = nf_ct_ext_types;
                if (!t1)
                        continue;               
                t1->alloc_size = sizeof(struct nf_ct_ext)
                                 + ALIGN(sizeof(struct nf_ct_ext), t1->align)
                                 + t1->len;

上面紅色字體就是我的問題，即這里為什么要多加個sizeof（struct nf_ct_ext）呢？在該函數(shù)的上級調(diào)用函數(shù)nf_ct_extend_register中，已經(jīng)明明有如下語句了：
          type->alloc_size = ALIGN(sizeof(struct nf_ct_ext),
                                      type->align) + type->len;

PS：

這個問題lunixwh在“論壇›程序設計›C/C++›”中也提出了這一問題，參見“netfilter擴展字段 注冊的時候alloc_size 字段問題”，但是沒有得到解答

Tinnal

回復 1# jiufei19


NND，被你害了。
我本身對這NF又不熟。為了回你這個問題，去翻代碼，翻得連自己都暈了，還是沒能解析。
回頭想想，可能真是一個BUG，確認了一下，還真是。F！U！C！K！

社區(qū)的提交記錄為：
https://git.kernel.org/cgit/linu ... cc0d6a7cfde64307655
10年就發(fā)現(xiàn)這個問題了。

diff --git a/net/netfilter/nf_conntrack_extend.c b/net/netfilter/nf_conntrack_extend.c
index bd82450..920f924 100644
--- a/net/netfilter/nf_conntrack_extend.c
+++ b/net/netfilter/nf_conntrack_extend.c
@@ -144,9 +144,8 @@ static void update_alloc_size(struct nf_ct_ext_type *type)
                if (!t1)
                        continue;

-                t1->alloc_size = sizeof(struct nf_ct_ext)
-                                 + ALIGN(sizeof(struct nf_ct_ext), t1->align)
-                                 + t1->len;
+                t1->alloc_size = ALIGN(sizeof(struct nf_ct_ext), t1->align) +
+                                 t1->len;
                for (j = 0; j < NF_CT_EXT_NUM; j++) {
                        t2 = nf_ct_ext_types[j];
                        if (t2 == NULL || t2 == t1 ||

總結(jié)：你們公司再這樣不跟進社區(qū)的補丁遲早把公司和你玩死。
友情提示：最近出了BASH的CVE漏洞，10分的，最高級別！趕緊合補丁，兄弟。


我覺得這個貼你真得感謝一下我，不然不能平復我心中的怨恨。論壇里怎么這么多人問NF的問題，論壇里就沒有對NF比較熟的牛ren?

jiufei19

Tinnal 發(fā)表于 2014-09-29 00:42
回復 1# jiufei19

真是非常感謝Tinnal的熱心，好人一個啊，

說來自己也慚愧，我其實有另外一個高版本的內(nèi)核源碼，居然自己沒有想到去對比下究竟怎么回事。

不過順便問下Tinnal，這種問題你是通過什么方法查到的呢，我的意思是如何在git中去搜到這樣的一個關于bug的說明，授我以魚的時候，能否再授我以漁？

再次萬分感謝！

PS：

我試了下，是否就是在該git網(wǎng)站上的“l(fā)og msg”的Search對話框中輸入自己要查找的關鍵字，比如我輸入"update_alloc_size"，于是就可以查到了這個bug的相關說明，Tinnal就是這樣操作的嗎？

Tinnal

嗯，這樣也可以。其實當時我是看了整個文件的changelog

jiufei19

回復 4# Tinnal


    再次感謝Tinnal！