Re: [PATCH] rsi: fix potential null dereference in rsi_probe()

2019-10-06 Thread Kalle Valo
Johan Hovold writes: > On Fri, Oct 04, 2019 at 01:47:36PM +, Kalle Valo wrote: >> Denis Efremov wrote: >> >> > The id pointer can be NULL in rsi_probe(). > > While the existing code in rsi_probe() may lead you to believe that, > this statement is false. > >> > It is checked everywhere

Re: [PATCH] rsi: fix potential null dereference in rsi_probe()

2019-10-04 Thread Johan Hovold
On Fri, Oct 04, 2019 at 01:47:36PM +, Kalle Valo wrote: > Denis Efremov wrote: > > > The id pointer can be NULL in rsi_probe(). While the existing code in rsi_probe() may lead you to believe that, this statement is false. > > It is checked everywhere except > > for the else branch in the

Re: [PATCH] rsi: fix potential null dereference in rsi_probe()

2019-10-04 Thread Kalle Valo
Denis Efremov wrote: > The id pointer can be NULL in rsi_probe(). It is checked everywhere except > for the else branch in the idProduct condition. The patch adds NULL check > before the id dereference in the rsi_dbg() call. > > Fixes: 54fdb318c111 ("rsi: add new device model for 9116") > Cc:

[PATCH] rsi: fix potential null dereference in rsi_probe()

2019-10-02 Thread Denis Efremov
The id pointer can be NULL in rsi_probe(). It is checked everywhere except for the else branch in the idProduct condition. The patch adds NULL check before the id dereference in the rsi_dbg() call. Fixes: 54fdb318c111 ("rsi: add new device model for 9116") Cc: Amitkumar Karwar Cc: Siva