Thread (10 messages) flat view 10 messages, 3 authors, 2018-07-06

Re: [PATCH] ipv4: fib: avoid NULL dereference

From: David Ahern <hidden>
Date: 2018-07-06 14:48:47
Also in: lkml

On 7/6/18 8:28 AM, Mark Rutland wrote:
quoted hunk ↗ jump to hunk
diff --git a/net/ipv4/fib_trie.c b/net/ipv4/fib_trie.c
index 5bc0c89e81e4..8d98c8162554 100644
--- a/net/ipv4/fib_trie.c
+++ b/net/ipv4/fib_trie.c
@@ -497,11 +497,11 @@ static void tnode_free(struct key_vector *tn)
 	struct callback_head *head = &tn_info(tn)->rcu;
 
 	while (head) {
+		tn = container_of(head, struct tnode, rcu)->kv;
+
 		head = head->next;
 		tnode_free_size += TNODE_SIZE(1ul << tn->bits);
 		node_free(tn);
you are changing tn before the above 2 calls are made on the tn passed in

-
-		tn = container_of(head, struct tnode, rcu)->kv;
 	}
 
 	if (tnode_free_size >= PAGE_SIZE * sync_pages) {
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help