From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: patch staging-iio-lis3l02dq-incorrect-ws-used-in-container-of-call.patch added to gregkh-2.6 tree To: jic23@cam.ac.uk,gregkh@suse.de,greg@kroah.com,linux-iio@vger.kernel.org From: Date: Thu, 22 Apr 2010 17:13:20 -0700 In-Reply-To: <4B99288C.3080601@cam.ac.uk> Message-ID: <12719816002587@kroah.org> MIME-Version: 1.0 Content-Type: text/plain; charset=ANSI_X3.4-1968 List-ID: This is a note to let you know that I've just added the patch titled Subject: staging: iio: lis3l02dq - incorrect ws used in container of = call. to my gregkh-2.6 tree. Its filename is staging-iio-lis3l02dq-incorrect-ws-used-in-container-of-call.patch This tree can be found at=20 http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/patch= es/ >>>From jic23@cam.ac.uk Thu Apr 22 16:37:59 2010 From: Jonathan Cameron Date: Thu, 11 Mar 2010 17:29:48 +0000 Subject: staging: iio: lis3l02dq - incorrect ws used in container of call= . To: Greg KH , "linux-iio@vger.kernel.org" Message-ID: <4B99288C.3080601@cam.ac.uk> The word oops comes to mind. Original patch to merge the two work queues in here (prior to Greg taking them into staging) changed the top half to only use one of them and the bottom half to assume it was the other. Currently causes a NULL pointer dereference if you enable any of the even= ts on an lis3l02dq. Just goes to show I need a few more regression tests. Signed-of-by: Jonathan Cameron Signed-off-by: Greg Kroah-Hartman --- drivers/staging/iio/accel/lis3l02dq_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/staging/iio/accel/lis3l02dq_core.c +++ b/drivers/staging/iio/accel/lis3l02dq_core.c @@ -618,7 +618,7 @@ static int lis3l02dq_thresh_handler_th(s static void lis3l02dq_thresh_handler_bh_no_check(struct work_struct *wor= k_s) { struct iio_work_cont *wc - =3D container_of(work_s, struct iio_work_cont, ws_nocheck); + =3D container_of(work_s, struct iio_work_cont, ws); struct lis3l02dq_state *st =3D wc->st; u8 t; =20