Commit dac85354 authored by Jia-Ju Bai's avatar Jia-Ju Bai Committed by Stefan Bader

mISDN: Fix a sleep-in-atomic bug

BugLink: http://bugs.launchpad.net/bugs/1765010

[ Upstream commit 93818da5 ]

The driver may sleep under a read spin lock, and the function call path is:
send_socklist (acquire the lock by read_lock)
  skb_copy(GFP_KERNEL) --> may sleep

To fix it, the "GFP_KERNEL" is replaced with "GFP_ATOMIC".
Signed-off-by: default avatarJia-Ju Bai <baijiaju1990@163.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
Signed-off-by: default avatarSasha Levin <alexander.levin@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarJuerg Haefliger <juergh@canonical.com>
Signed-off-by: default avatarStefan Bader <stefan.bader@canonical.com>
parent a8b2ff48
...@@ -72,7 +72,7 @@ send_socklist(struct mISDN_sock_list *sl, struct sk_buff *skb) ...@@ -72,7 +72,7 @@ send_socklist(struct mISDN_sock_list *sl, struct sk_buff *skb)
if (sk->sk_state != MISDN_BOUND) if (sk->sk_state != MISDN_BOUND)
continue; continue;
if (!cskb) if (!cskb)
cskb = skb_copy(skb, GFP_KERNEL); cskb = skb_copy(skb, GFP_ATOMIC);
if (!cskb) { if (!cskb) {
printk(KERN_WARNING "%s no skb\n", __func__); printk(KERN_WARNING "%s no skb\n", __func__);
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment