lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Mon, 17 Aug 2020 22:06:07 +0900
From:   Tetsuo Handa <>
To:     Ganapathi Bhat <>
Cc:     Brian Norris <>,
        amit karwar <>,,
        "David S. Miller" <>,
        Dmitry Vyukov <>,
        Xinming Hu <>,
        Kalle Valo <>,
        Linux Kernel <>,
        Linux USB Mailing List <>,
        linux-wireless <>,, Nishant Sarmukadam <>,,,
        syzbot <>
Subject: Re: [PATCH] mwifiex: don't call del_timer_sync() on uninitialized

Ganapathi, how do you want to fix this bug?

On 2020/07/29 3:45, Brian Norris wrote:
>> syzbot is reporting that del_timer_sync() is called from
>> mwifiex_usb_cleanup_tx_aggr() from mwifiex_unregister_dev() without
>> checking timer_setup() from mwifiex_usb_tx_init() was called [1].
>> Since mwifiex_usb_prepare_tx_aggr_skb() is calling del_timer() if
>> is_hold_timer_set == true, use the same condition for del_timer_sync().
>> [1]
>> Reported-by: syzbot <>
>> Cc: Ganapathi Bhat <>
>> Signed-off-by: Tetsuo Handa <>
>> ---
>> A patch from Ganapathi Bhat ( ) is stalling
>> at .
>> syzbot by now got this report for 10000 times. Do we want to go with this simple patch?
> Sorry, that stall is partly my fault, and partly Ganapathi's. It
> doesn't help that it took him 4 months to reply to my questions, so I
> completely lost even the tiny bit of context I had managed to build up
> in my head at initial review time... and so it's still buried in the
> dark corners of my inbox. (I think I'll go archive that now, because
> it really deserves a better sell than it had initially, if Ganapathi
> really wants to land it.)

Powered by blists - more mailing lists