]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commitdiff
rtl8188eu: Fix a possible sleep-in-atomic bug in rtw_createbss_cmd
authorJia-Ju Bai <baijiaju1990@163.com>
Sun, 8 Oct 2017 11:54:45 +0000 (19:54 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 18 Oct 2017 13:43:35 +0000 (15:43 +0200)
The driver may sleep under a spinlock, and the function call path is:
rtw_surveydone_event_callback(acquire the spinlock)
  rtw_createbss_cmd
    kzalloc(GFP_KERNEL) --> may sleep

To fix it, GFP_KERNEL is replaced with GFP_ATOMIC.
This bug is found by my static analysis tool and my code review.

Signed-off-by: Jia-Ju Bai <baijiaju1990@163.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/core/rtw_cmd.c

index 65083a72b4085ddc364c35d605ba4b63c9a8fb51..be8542676adf9e2c458110a12079b24fa0e10ee2 100644 (file)
@@ -333,7 +333,7 @@ u8 rtw_createbss_cmd(struct adapter  *padapter)
        else
                RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_, (" createbss for SSid:%s\n", pmlmepriv->assoc_ssid.Ssid));
 
-       pcmd = kzalloc(sizeof(struct cmd_obj), GFP_KERNEL);
+       pcmd = kzalloc(sizeof(struct cmd_obj), GFP_ATOMIC);
        if (!pcmd) {
                res = _FAIL;
                goto exit;