Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-11 Thread Jörg Otte
2016-11-11 14:55 GMT+01:00 Mauro Carvalho Chehab : > Em Thu, 10 Nov 2016 12:15:39 +0100 > Jörg Otte escreveu: > >> 2016-11-10 9:40 GMT+01:00 Mauro Carvalho Chehab : >> > Em Wed, 9 Nov 2016 11:07:35 -0800 >> > Linus Torvalds

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-11 Thread Jörg Otte
2016-11-11 14:55 GMT+01:00 Mauro Carvalho Chehab : > Em Thu, 10 Nov 2016 12:15:39 +0100 > Jörg Otte escreveu: > >> 2016-11-10 9:40 GMT+01:00 Mauro Carvalho Chehab : >> > Em Wed, 9 Nov 2016 11:07:35 -0800 >> > Linus Torvalds escreveu: >> > >> >> On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote:

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-11 Thread Mauro Carvalho Chehab
Em Thu, 10 Nov 2016 12:15:39 +0100 Jörg Otte escreveu: > 2016-11-10 9:40 GMT+01:00 Mauro Carvalho Chehab : > > Em Wed, 9 Nov 2016 11:07:35 -0800 > > Linus Torvalds escreveu: > > > >> On Wed, Nov 9, 2016 at 3:09 AM,

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-11 Thread Mauro Carvalho Chehab
Em Thu, 10 Nov 2016 12:15:39 +0100 Jörg Otte escreveu: > 2016-11-10 9:40 GMT+01:00 Mauro Carvalho Chehab : > > Em Wed, 9 Nov 2016 11:07:35 -0800 > > Linus Torvalds escreveu: > > > >> On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote: > >> > > >> > Tried patch with no success. Again a NULL ptr

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-10 Thread Jörg Otte
2016-11-10 9:40 GMT+01:00 Mauro Carvalho Chehab : > Em Wed, 9 Nov 2016 11:07:35 -0800 > Linus Torvalds escreveu: > >> On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote: >> > >> > Tried patch with no success. Again a NULL

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-10 Thread Jörg Otte
2016-11-10 9:40 GMT+01:00 Mauro Carvalho Chehab : > Em Wed, 9 Nov 2016 11:07:35 -0800 > Linus Torvalds escreveu: > >> On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote: >> > >> > Tried patch with no success. Again a NULL ptr dereferece. >> >> That patch was pure garbage, I think. Pretty much all

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-10 Thread Mauro Carvalho Chehab
Em Wed, 9 Nov 2016 11:07:35 -0800 Linus Torvalds escreveu: > On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote: > > > > Tried patch with no success. Again a NULL ptr dereferece. > > That patch was pure garbage, I think. Pretty much all the

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-10 Thread Mauro Carvalho Chehab
Em Wed, 9 Nov 2016 11:07:35 -0800 Linus Torvalds escreveu: > On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote: > > > > Tried patch with no success. Again a NULL ptr dereferece. > > That patch was pure garbage, I think. Pretty much all the other > drivers that use the same approach will have

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-09 Thread Mauro Carvalho Chehab
Em Wed, 9 Nov 2016 19:57:58 + Malcolm Priestley escreveu: > > Yeah, I avoided serializing the logic that detects if the firmware is > > loaded, but forgot that the power control had the same issue. The > > newer dvb usb drivers use the dvb-usb-v2, so I didn't touch this >

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-09 Thread Mauro Carvalho Chehab
Em Wed, 9 Nov 2016 19:57:58 + Malcolm Priestley escreveu: > > Yeah, I avoided serializing the logic that detects if the firmware is > > loaded, but forgot that the power control had the same issue. The > > newer dvb usb drivers use the dvb-usb-v2, so I didn't touch this > > code for a while.

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-09 Thread Mauro Carvalho Chehab
Em Wed, 9 Nov 2016 11:07:35 -0800 Linus Torvalds escreveu: > On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote: > > > > Tried patch with no success. Again a NULL ptr dereferece. > > That patch was pure garbage, I think. Pretty much all the

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-09 Thread Mauro Carvalho Chehab
Em Wed, 9 Nov 2016 11:07:35 -0800 Linus Torvalds escreveu: > On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote: > > > > Tried patch with no success. Again a NULL ptr dereferece. > > That patch was pure garbage, I think. Pretty much all the other > drivers that use the same approach will have

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-09 Thread Malcolm Priestley
On 08/11/16 21:38, Mauro Carvalho Chehab wrote: Em Tue, 8 Nov 2016 22:15:24 +0100 Benjamin Larsson escreveu: On 11/08/2016 09:22 PM, Mauro Carvalho Chehab wrote: Em Tue, 8 Nov 2016 10:42:03 -0800 Linus Torvalds escreveu: On Sun, Nov 6,

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-09 Thread Malcolm Priestley
On 08/11/16 21:38, Mauro Carvalho Chehab wrote: Em Tue, 8 Nov 2016 22:15:24 +0100 Benjamin Larsson escreveu: On 11/08/2016 09:22 PM, Mauro Carvalho Chehab wrote: Em Tue, 8 Nov 2016 10:42:03 -0800 Linus Torvalds escreveu: On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote: Since v4.9-rc4 I

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-09 Thread Linus Torvalds
On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote: > > Tried patch with no success. Again a NULL ptr dereferece. That patch was pure garbage, I think. Pretty much all the other drivers that use the same approach will have the same issue. Adding that init function just for the

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-09 Thread Linus Torvalds
On Wed, Nov 9, 2016 at 3:09 AM, Jörg Otte wrote: > > Tried patch with no success. Again a NULL ptr dereferece. That patch was pure garbage, I think. Pretty much all the other drivers that use the same approach will have the same issue. Adding that init function just for the semaphore is crazy.

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-09 Thread Jörg Otte
2016-11-08 21:22 GMT+01:00 Mauro Carvalho Chehab : > Em Tue, 8 Nov 2016 10:42:03 -0800 > Linus Torvalds escreveu: > >> On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote: >> > Since v4.9-rc4 I get following crash in

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-09 Thread Jörg Otte
2016-11-08 21:22 GMT+01:00 Mauro Carvalho Chehab : > Em Tue, 8 Nov 2016 10:42:03 -0800 > Linus Torvalds escreveu: > >> On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote: >> > Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module. >> >> Looks like it's commit 5ef8ed0e5608f ("[media]

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-08 Thread Mauro Carvalho Chehab
Em Tue, 8 Nov 2016 22:15:24 +0100 Benjamin Larsson escreveu: > On 11/08/2016 09:22 PM, Mauro Carvalho Chehab wrote: > > Em Tue, 8 Nov 2016 10:42:03 -0800 > > Linus Torvalds escreveu: > > > >> On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-08 Thread Mauro Carvalho Chehab
Em Tue, 8 Nov 2016 22:15:24 +0100 Benjamin Larsson escreveu: > On 11/08/2016 09:22 PM, Mauro Carvalho Chehab wrote: > > Em Tue, 8 Nov 2016 10:42:03 -0800 > > Linus Torvalds escreveu: > > > >> On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote: > >>> Since v4.9-rc4 I get following crash in

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-08 Thread Benjamin Larsson
On 11/08/2016 09:22 PM, Mauro Carvalho Chehab wrote: Em Tue, 8 Nov 2016 10:42:03 -0800 Linus Torvalds escreveu: On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote: Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module. Looks like

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-08 Thread Benjamin Larsson
On 11/08/2016 09:22 PM, Mauro Carvalho Chehab wrote: Em Tue, 8 Nov 2016 10:42:03 -0800 Linus Torvalds escreveu: On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote: Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module. Looks like it's commit 5ef8ed0e5608f ("[media] cinergyT2-core:

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-08 Thread Mauro Carvalho Chehab
Em Tue, 8 Nov 2016 10:42:03 -0800 Linus Torvalds escreveu: > On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote: > > Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module. > > Looks like it's commit 5ef8ed0e5608f ("[media]

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-08 Thread Mauro Carvalho Chehab
Em Tue, 8 Nov 2016 10:42:03 -0800 Linus Torvalds escreveu: > On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote: > > Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module. > > Looks like it's commit 5ef8ed0e5608f ("[media] cinergyT2-core: don't > do DMA on stack"), which movced the

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-08 Thread Linus Torvalds
On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote: > Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module. Looks like it's commit 5ef8ed0e5608f ("[media] cinergyT2-core: don't do DMA on stack"), which movced the DMA data array from the stack to the "private"

Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

2016-11-08 Thread Linus Torvalds
On Sun, Nov 6, 2016 at 7:40 AM, Jörg Otte wrote: > Since v4.9-rc4 I get following crash in dvb-usb-cinergyT2 module. Looks like it's commit 5ef8ed0e5608f ("[media] cinergyT2-core: don't do DMA on stack"), which movced the DMA data array from the stack to the "private" pointer. In the process it