iptables的问题

adidiaos丶丶

注册表的函数：
struct xt_table *xt_register_table(struct net *net,
                                   const struct xt_table *input_table,
                                   struct xt_table_info *bootstrap,
                                   struct xt_table_info *newinfo)
{
        int ret;
        struct xt_table_info *private;
        struct xt_table *t, *table;

        /* Don't add one object to multiple lists. */
        table = kmemdup(input_table, sizeof(struct xt_table), GFP_KERNEL);
        if (!table) {
                ret = -ENOMEM;
                goto out;
        }

        ret = mutex_lock_interruptible(&xt[table->af].mutex);
        if (ret != 0)
                goto out_free;

        /* Don't autoload: we'd eat our tail... */
        list_for_each_entry(t, &net->xt.tables[table->af], list) {
                if (strcmp(t->name, table->name) == 0) {
                        ret = -EEXIST;
                        goto unlock;
                }
        }

        /* Simplifies replace_table code. */
        table->private = bootstrap;

        if (!xt_replace_table(table, 0, newinfo, &ret))
                goto unlock;

        private = table->private;
        pr_debug("table->private->number = %u\n", private->number);

        /* save number of initial entries */
        private->initial_entries = private->number;

        list_add(&table->list, &net->xt.tables[table->af]);
        mutex_unlock(&xt[table->af].mutex);
        return table;

unlock:
        mutex_unlock(&xt[table->af].mutex);
out_free:
        kfree(table);
out:
        return ERR_PTR(ret);
}

红色部分为什么要lock，后面的代码是没有操作xt的。高手指点一下，没明白什么意思。

Godbach

回复 1# adidiaos丶丶

1.       list_for_each_entry(t, &net->xt.tables[table->af], list) {
   
2.                 if (strcmp(t->name, table->name) == 0) {
   
3.                         ret = -EEXIST;
   
4.                         goto unlock;
   
5.                 }
   
6.         }

复制代码

这段代码不是在做查找吗，找到之后要操作的。

   

adidiaos丶丶

标红的xt和net->xt.tables[table->af]不是同一个东西吧回复 2# Godbach


   

Godbach

回复 1# adidiaos丶丶
主要还是看语义。

这里要注册一个 table 的时候，实现互斥的逻辑。