We should add a lock protection when we free the skb,
because it maybe used by ipt_ulog_packet right now.
Signed-off-by: Gao feng <gaofeng@xxxxxxxxxxxxxx>
---
net/ipv4/netfilter/ipt_ULOG.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/net/ipv4/netfilter/ipt_ULOG.c b/net/ipv4/netfilter/ipt_ULOG.c
index b5ef3cb..b390002 100644
--- a/net/ipv4/netfilter/ipt_ULOG.c
+++ b/net/ipv4/netfilter/ipt_ULOG.c
@@ -430,11 +430,12 @@ static void __exit ulog_tg_exit(void)
pr_debug("timer was pending, deleting\n");
del_timer(&ub->timer);
}
-
+ spin_lock_bh(&ulog_lock);
if (ub->skb) {
kfree_skb(ub->skb);
ub->skb = NULL;
}
+ spin_unlock_bh(&ulog_lock);
}
}
--
1.7.11.7
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
