Browse Source

ieee802154: add locking for seq numbers

Signed-off-by: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
Dmitry Eremin-Solenikov 15 years ago
parent
commit
c4835d81ef
1 changed files with 4 additions and 0 deletions
  1. 4 0
      net/ieee802154/netlink.c

+ 4 - 0
net/ieee802154/netlink.c

@@ -35,6 +35,7 @@
 #include <net/ieee802154_netdev.h>
 #include <net/ieee802154_netdev.h>
 
 
 static unsigned int ieee802154_seq_num;
 static unsigned int ieee802154_seq_num;
+static DEFINE_SPINLOCK(ieee802154_seq_lock);
 
 
 static struct genl_family ieee802154_coordinator_family = {
 static struct genl_family ieee802154_coordinator_family = {
 	.id		= GENL_ID_GENERATE,
 	.id		= GENL_ID_GENERATE,
@@ -57,12 +58,15 @@ static struct sk_buff *ieee802154_nl_create(int flags, u8 req)
 {
 {
 	void *hdr;
 	void *hdr;
 	struct sk_buff *msg = nlmsg_new(NLMSG_GOODSIZE, GFP_ATOMIC);
 	struct sk_buff *msg = nlmsg_new(NLMSG_GOODSIZE, GFP_ATOMIC);
+	unsigned long f;
 
 
 	if (!msg)
 	if (!msg)
 		return NULL;
 		return NULL;
 
 
+	spin_lock_irqsave(&ieee802154_seq_lock, f);
 	hdr = genlmsg_put(msg, 0, ieee802154_seq_num++,
 	hdr = genlmsg_put(msg, 0, ieee802154_seq_num++,
 			&ieee802154_coordinator_family, flags, req);
 			&ieee802154_coordinator_family, flags, req);
+	spin_unlock_irqrestore(&ieee802154_seq_lock, f);
 	if (!hdr) {
 	if (!hdr) {
 		nlmsg_free(msg);
 		nlmsg_free(msg);
 		return NULL;
 		return NULL;