net: decnet: Fix refcount warning for new dn_fib_info
authorYajun Deng <yajun.deng@linux.dev>
Tue, 3 Aug 2021 07:37:39 +0000 (15:37 +0800)
committerJakub Kicinski <kuba@kernel.org>
Tue, 3 Aug 2021 21:23:22 +0000 (14:23 -0700)
fib_treeref needs to be set after kzalloc. The old code had a ++ which
led to the confusion when the int was replaced by a refcount_t.

Fixes: 79976892f7ea ("net: convert fib_treeref from int to refcount_t")
Signed-off-by: Yajun Deng <yajun.deng@linux.dev>
Reviewed-by: David Ahern <dsahern@kernel.org>
Link: https://lore.kernel.org/r/20210803073739.22339-1-yajun.deng@linux.dev
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/decnet/dn_fib.c

index 387a7e8..153a5fc 100644 (file)
@@ -389,7 +389,7 @@ link_it:
                return ofi;
        }
 
-       refcount_inc(&fi->fib_treeref);
+       refcount_set(&fi->fib_treeref, 1);
        refcount_set(&fi->fib_clntref, 1);
        spin_lock(&dn_fib_info_lock);
        fi->fib_next = dn_fib_info_list;