summary refs log tree commit diff
path: root/net/tipc/name_table.c
diff options
context:
space:
mode:
authorYing Xue <ying.xue@windriver.com>2014-12-09 15:17:56 +0800
committerDavid S. Miller <davem@davemloft.net>2014-12-09 18:27:03 -0500
commit023160bc8f100ad949ebaee0d3a1b7398d938171 (patch)
tree23c6403901add558eb44802cd9e019434f9dbe39 /net/tipc/name_table.c
parentfb3b596d3c4a8b6e3867079794856637db4e5283 (diff)
downloadlinux-023160bc8f100ad949ebaee0d3a1b7398d938171.tar.gz
tipc: avoid double lock 'spin_lock:&seq->lock'
The commit fb9962f3cefe ("tipc: ensure all name sequences are properly
protected with its lock") involves below errors:

net/tipc/name_table.c:980 tipc_purge_publications() error: double lock 'spin_lock:&seq->lock'

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Ying Xue <ying.xue@windriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/name_table.c')
-rw-r--r--net/tipc/name_table.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/tipc/name_table.c b/net/tipc/name_table.c
index aafa684c4db9..c8df0223371a 100644
--- a/net/tipc/name_table.c
+++ b/net/tipc/name_table.c
@@ -979,7 +979,7 @@ static void tipc_purge_publications(struct name_seq *seq)
 	}
 	hlist_del_init_rcu(&seq->ns_list);
 	kfree(seq->sseqs);
-	spin_lock_bh(&seq->lock);
+	spin_unlock_bh(&seq->lock);
 
 	kfree_rcu(seq, rcu);
 }