On Mon, Jun 05, 2017 at 08:23:40PM +0800, Jia-Ju Bai wrote: > The driver may sleep under a spin lock, and the function call path is: > post_one_send (acquire the lock by spin_lock_irqsave) > init_send_wqe > copy_from_user --> may sleep > > There is no flow that makes "qp->is_user" true, and copy_from_user may > cause bug when a non-user pointer is used. So the lines of copy_from_user > and check of "qp->is_user" are removed. > > Signed-off-by: Jia-Ju Bai <[email protected]> > ---
Thanks, Reviewed-by: Leon Romanovsky <[email protected]>
signature.asc
Description: PGP signature
