* [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free
@ 2015-08-21 4:29 Dāvis Mosāns
2015-09-01 20:08 ` Dāvis Mosāns
` (2 more replies)
0 siblings, 3 replies; 11+ messages in thread
From: Dāvis Mosāns @ 2015-08-21 4:29 UTC (permalink / raw)
To: James E.J. Bottomley, linux-scsi; +Cc: Dāvis Mosāns
When pci_pool_alloc fails in mvs_task_prep then task->lldd_task stays
NULL but it's later used in mvs_abort_task as slot which is passed
to mvs_slot_task_free causing NULL pointer dereference.
Just return from mvs_slot_task_free when passed with NULL slot.
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=101891
Signed-off-by: Dāvis Mosāns <davispuh@gmail.com>
---
drivers/scsi/mvsas/mv_sas.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c
index 454536c..9c78074 100644
--- a/drivers/scsi/mvsas/mv_sas.c
+++ b/drivers/scsi/mvsas/mv_sas.c
@@ -887,6 +887,8 @@ static void mvs_slot_free(struct mvs_info *mvi, u32 rx_desc)
static void mvs_slot_task_free(struct mvs_info *mvi, struct sas_task *task,
struct mvs_slot_info *slot, u32 slot_idx)
{
+ if (!slot)
+ return;
if (!slot->task)
return;
if (!sas_protocol_ata(task->task_proto))
--
2.5.0
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free
2015-08-21 4:29 [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free Dāvis Mosāns
@ 2015-09-01 20:08 ` Dāvis Mosāns
2015-09-02 12:08 ` Tomas Henzl
2015-10-20 17:41 ` Dāvis Mosāns
2 siblings, 0 replies; 11+ messages in thread
From: Dāvis Mosāns @ 2015-09-01 20:08 UTC (permalink / raw)
To: James E.J. Bottomley, linux-scsi; +Cc: Dāvis Mosāns
2015-08-21 7:29 GMT+03:00 Dāvis Mosāns <davispuh@gmail.com>:
> When pci_pool_alloc fails in mvs_task_prep then task->lldd_task stays
> NULL but it's later used in mvs_abort_task as slot which is passed
> to mvs_slot_task_free causing NULL pointer dereference.
>
> Just return from mvs_slot_task_free when passed with NULL slot.
>
> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=101891
> Signed-off-by: Dāvis Mosāns <davispuh@gmail.com>
> ---
> drivers/scsi/mvsas/mv_sas.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c
> index 454536c..9c78074 100644
> --- a/drivers/scsi/mvsas/mv_sas.c
> +++ b/drivers/scsi/mvsas/mv_sas.c
> @@ -887,6 +887,8 @@ static void mvs_slot_free(struct mvs_info *mvi, u32 rx_desc)
> static void mvs_slot_task_free(struct mvs_info *mvi, struct sas_task *task,
> struct mvs_slot_info *slot, u32 slot_idx)
> {
> + if (!slot)
> + return;
> if (!slot->task)
> return;
> if (!sas_protocol_ata(task->task_proto))
> --
> 2.5.0
>
ping? do I need to CC someone else?
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free
2015-08-21 4:29 [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free Dāvis Mosāns
2015-09-01 20:08 ` Dāvis Mosāns
@ 2015-09-02 12:08 ` Tomas Henzl
2015-10-20 17:41 ` Dāvis Mosāns
2 siblings, 0 replies; 11+ messages in thread
From: Tomas Henzl @ 2015-09-02 12:08 UTC (permalink / raw)
To: Dāvis Mosāns, James E.J. Bottomley, linux-scsi
On 21.8.2015 06:29, Dāvis Mosāns wrote:
> When pci_pool_alloc fails in mvs_task_prep then task->lldd_task stays
> NULL but it's later used in mvs_abort_task as slot which is passed
> to mvs_slot_task_free causing NULL pointer dereference.
>
> Just return from mvs_slot_task_free when passed with NULL slot.
>
> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=101891
> Signed-off-by: Dāvis Mosāns <davispuh@gmail.com>
Reviewed-by: Tomas Henzl <thenzl@redhat.com>
Cheers,
Tomas
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free
2015-08-21 4:29 [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free Dāvis Mosāns
2015-09-01 20:08 ` Dāvis Mosāns
2015-09-02 12:08 ` Tomas Henzl
@ 2015-10-20 17:41 ` Dāvis Mosāns
2015-10-21 7:33 ` Johannes Thumshirn
2 siblings, 1 reply; 11+ messages in thread
From: Dāvis Mosāns @ 2015-10-20 17:41 UTC (permalink / raw)
To: James E.J. Bottomley, linux-scsi, linux-kernel
2015-08-21 7:29 GMT+03:00 Dāvis Mosāns <davispuh@gmail.com>:
> When pci_pool_alloc fails in mvs_task_prep then task->lldd_task stays
> NULL but it's later used in mvs_abort_task as slot which is passed
> to mvs_slot_task_free causing NULL pointer dereference.
>
> Just return from mvs_slot_task_free when passed with NULL slot.
>
> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=101891
> Signed-off-by: Dāvis Mosāns <davispuh@gmail.com>
> ---
> drivers/scsi/mvsas/mv_sas.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c
> index 454536c..9c78074 100644
> --- a/drivers/scsi/mvsas/mv_sas.c
> +++ b/drivers/scsi/mvsas/mv_sas.c
> @@ -887,6 +887,8 @@ static void mvs_slot_free(struct mvs_info *mvi, u32 rx_desc)
> static void mvs_slot_task_free(struct mvs_info *mvi, struct sas_task *task,
> struct mvs_slot_info *slot, u32 slot_idx)
> {
> + if (!slot)
> + return;
> if (!slot->task)
> return;
> if (!sas_protocol_ata(task->task_proto))
> --
> 2.5.0
>
Can this get merged?
So far since august it have saved me from several kernel crashes.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free
2015-10-20 17:41 ` Dāvis Mosāns
@ 2015-10-21 7:33 ` Johannes Thumshirn
2015-10-21 13:18 ` Dāvis Mosāns
0 siblings, 1 reply; 11+ messages in thread
From: Johannes Thumshirn @ 2015-10-21 7:33 UTC (permalink / raw)
To: Dāvis Mosāns, James E.J. Bottomley, linux-scsi, linux-kernel
On Tue, 2015-10-20 at 20:41 +0300, Dāvis Mosāns wrote:
> 2015-08-21 7:29 GMT+03:00 Dāvis Mosāns <davispuh@gmail.com>:
> > When pci_pool_alloc fails in mvs_task_prep then task->lldd_task
> > stays
> > NULL but it's later used in mvs_abort_task as slot which is passed
> > to mvs_slot_task_free causing NULL pointer dereference.
> >
> > Just return from mvs_slot_task_free when passed with NULL slot.
> >
> > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=101891
> > Signed-off-by: Dāvis Mosāns <davispuh@gmail.com>
> > ---
> > drivers/scsi/mvsas/mv_sas.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/scsi/mvsas/mv_sas.c
> > b/drivers/scsi/mvsas/mv_sas.c
> > index 454536c..9c78074 100644
> > --- a/drivers/scsi/mvsas/mv_sas.c
> > +++ b/drivers/scsi/mvsas/mv_sas.c
> > @@ -887,6 +887,8 @@ static void mvs_slot_free(struct mvs_info *mvi,
> > u32 rx_desc)
> > static void mvs_slot_task_free(struct mvs_info *mvi, struct
> > sas_task *task,
> > struct mvs_slot_info *slot, u32 slot_idx)
> > {
> > + if (!slot)
> > + return;
> > if (!slot->task)
> > return;
> > if (!sas_protocol_ata(task->task_proto))
> > --
> > 2.5.0
> >
>
> Can this get merged?
> So far since august it have saved me from several kernel crashes.
If it saved you from several crashes, it probably should be tagged for
stable, shouldn't it?
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi"
> in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free
2015-10-21 7:33 ` Johannes Thumshirn
@ 2015-10-21 13:18 ` Dāvis Mosāns
0 siblings, 0 replies; 11+ messages in thread
From: Dāvis Mosāns @ 2015-10-21 13:18 UTC (permalink / raw)
To: Johannes Thumshirn; +Cc: James E.J. Bottomley, linux-scsi, linux-kernel
2015-10-21 10:33 GMT+03:00 Johannes Thumshirn <jthumshirn@suse.de>:
> On Tue, 2015-10-20 at 20:41 +0300, Dāvis Mosāns wrote:
>> 2015-08-21 7:29 GMT+03:00 Dāvis Mosāns <davispuh@gmail.com>:
>> > When pci_pool_alloc fails in mvs_task_prep then task->lldd_task
>> > stays
>> > NULL but it's later used in mvs_abort_task as slot which is passed
>> > to mvs_slot_task_free causing NULL pointer dereference.
>> >
>> > Just return from mvs_slot_task_free when passed with NULL slot.
>> >
>> > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=101891
>> > Signed-off-by: Dāvis Mosāns <davispuh@gmail.com>
>> > ---
>> > drivers/scsi/mvsas/mv_sas.c | 2 ++
>> > 1 file changed, 2 insertions(+)
>> >
>> > diff --git a/drivers/scsi/mvsas/mv_sas.c
>> > b/drivers/scsi/mvsas/mv_sas.c
>> > index 454536c..9c78074 100644
>> > --- a/drivers/scsi/mvsas/mv_sas.c
>> > +++ b/drivers/scsi/mvsas/mv_sas.c
>> > @@ -887,6 +887,8 @@ static void mvs_slot_free(struct mvs_info *mvi,
>> > u32 rx_desc)
>> > static void mvs_slot_task_free(struct mvs_info *mvi, struct
>> > sas_task *task,
>> > struct mvs_slot_info *slot, u32 slot_idx)
>> > {
>> > + if (!slot)
>> > + return;
>> > if (!slot->task)
>> > return;
>> > if (!sas_protocol_ata(task->task_proto))
>> > --
>> > 2.5.0
>> >
>>
>> Can this get merged?
>> So far since august it have saved me from several kernel crashes.
>
> If it saved you from several crashes, it probably should be tagged for
> stable, shouldn't it?
>
> Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
>
>
I don't really know how that works... this is my first patch so I'm
not really concerned about in which version it gets in as long as it does.
I've been compiling kernel with this patch for these months so for me it
doesn't really make any difference.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free
@ 2015-10-21 13:18 ` Dāvis Mosāns
0 siblings, 0 replies; 11+ messages in thread
From: Dāvis Mosāns @ 2015-10-21 13:18 UTC (permalink / raw)
To: Johannes Thumshirn; +Cc: James E.J. Bottomley, linux-scsi, linux-kernel
2015-10-21 10:33 GMT+03:00 Johannes Thumshirn <jthumshirn@suse.de>:
> On Tue, 2015-10-20 at 20:41 +0300, Dāvis Mosāns wrote:
>> 2015-08-21 7:29 GMT+03:00 Dāvis Mosāns <davispuh@gmail.com>:
>> > When pci_pool_alloc fails in mvs_task_prep then task->lldd_task
>> > stays
>> > NULL but it's later used in mvs_abort_task as slot which is passed
>> > to mvs_slot_task_free causing NULL pointer dereference.
>> >
>> > Just return from mvs_slot_task_free when passed with NULL slot.
>> >
>> > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=101891
>> > Signed-off-by: Dāvis Mosāns <davispuh@gmail.com>
>> > ---
>> > drivers/scsi/mvsas/mv_sas.c | 2 ++
>> > 1 file changed, 2 insertions(+)
>> >
>> > diff --git a/drivers/scsi/mvsas/mv_sas.c
>> > b/drivers/scsi/mvsas/mv_sas.c
>> > index 454536c..9c78074 100644
>> > --- a/drivers/scsi/mvsas/mv_sas.c
>> > +++ b/drivers/scsi/mvsas/mv_sas.c
>> > @@ -887,6 +887,8 @@ static void mvs_slot_free(struct mvs_info *mvi,
>> > u32 rx_desc)
>> > static void mvs_slot_task_free(struct mvs_info *mvi, struct
>> > sas_task *task,
>> > struct mvs_slot_info *slot, u32 slot_idx)
>> > {
>> > + if (!slot)
>> > + return;
>> > if (!slot->task)
>> > return;
>> > if (!sas_protocol_ata(task->task_proto))
>> > --
>> > 2.5.0
>> >
>>
>> Can this get merged?
>> So far since august it have saved me from several kernel crashes.
>
> If it saved you from several crashes, it probably should be tagged for
> stable, shouldn't it?
>
> Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
>
>
I don't really know how that works... this is my first patch so I'm
not really concerned about in which version it gets in as long as it does.
I've been compiling kernel with this patch for these months so for me it
doesn't really make any difference.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free
2015-10-21 13:18 ` Dāvis Mosāns
@ 2015-10-21 13:47 ` Johannes Thumshirn
-1 siblings, 0 replies; 11+ messages in thread
From: Johannes Thumshirn @ 2015-10-21 13:47 UTC (permalink / raw)
To: Dāvis Mosāns; +Cc: James E.J. Bottomley, linux-scsi, linux-kernel
On Wed, 2015-10-21 at 16:18 +0300, Dāvis Mosāns wrote:
> 2015-10-21 10:33 GMT+03:00 Johannes Thumshirn <jthumshirn@suse.de>:
> > On Tue, 2015-10-20 at 20:41 +0300, Dāvis Mosāns wrote:
> > > 2015-08-21 7:29 GMT+03:00 Dāvis Mosāns <davispuh@gmail.com>:
> > > > When pci_pool_alloc fails in mvs_task_prep then task->lldd_task
> > > > stays
> > > > NULL but it's later used in mvs_abort_task as slot which is
> > > > passed
> > > > to mvs_slot_task_free causing NULL pointer dereference.
> > > >
> > > > Just return from mvs_slot_task_free when passed with NULL slot.
> > > >
> > > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=101891
> > > > Signed-off-by: Dāvis Mosāns <davispuh@gmail.com>
> > > > ---
> > > > drivers/scsi/mvsas/mv_sas.c | 2 ++
> > > > 1 file changed, 2 insertions(+)
> > > >
> > > > diff --git a/drivers/scsi/mvsas/mv_sas.c
> > > > b/drivers/scsi/mvsas/mv_sas.c
> > > > index 454536c..9c78074 100644
> > > > --- a/drivers/scsi/mvsas/mv_sas.c
> > > > +++ b/drivers/scsi/mvsas/mv_sas.c
> > > > @@ -887,6 +887,8 @@ static void mvs_slot_free(struct mvs_info
> > > > *mvi,
> > > > u32 rx_desc)
> > > > static void mvs_slot_task_free(struct mvs_info *mvi, struct
> > > > sas_task *task,
> > > > struct mvs_slot_info *slot, u32
> > > > slot_idx)
> > > > {
> > > > + if (!slot)
> > > > + return;
> > > > if (!slot->task)
> > > > return;
> > > > if (!sas_protocol_ata(task->task_proto))
> > > > --
> > > > 2.5.0
> > > >
> > >
> > > Can this get merged?
> > > So far since august it have saved me from several kernel crashes.
> >
> > If it saved you from several crashes, it probably should be tagged
> > for
> > stable, shouldn't it?
> >
> > Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
> >
> >
>
> I don't really know how that works... this is my first patch so I'm
> not really concerned about in which version it gets in as long as it
> does.
> I've been compiling kernel with this patch for these months so for me
> it
> doesn't really make any difference.
You can add
Cc: stable@vger.kernel.org
somewhere around your Signed-off-by
Documentation/stable_kernel_rules.txt has all the process
documentation.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free
@ 2015-10-21 13:47 ` Johannes Thumshirn
0 siblings, 0 replies; 11+ messages in thread
From: Johannes Thumshirn @ 2015-10-21 13:47 UTC (permalink / raw)
To: Dāvis Mosāns; +Cc: James E.J. Bottomley, linux-scsi, linux-kernel
On Wed, 2015-10-21 at 16:18 +0300, Dāvis Mosāns wrote:
> 2015-10-21 10:33 GMT+03:00 Johannes Thumshirn <jthumshirn@suse.de>:
> > On Tue, 2015-10-20 at 20:41 +0300, Dāvis Mosāns wrote:
> > > 2015-08-21 7:29 GMT+03:00 Dāvis Mosāns <davispuh@gmail.com>:
> > > > When pci_pool_alloc fails in mvs_task_prep then task->lldd_task
> > > > stays
> > > > NULL but it's later used in mvs_abort_task as slot which is
> > > > passed
> > > > to mvs_slot_task_free causing NULL pointer dereference.
> > > >
> > > > Just return from mvs_slot_task_free when passed with NULL slot.
> > > >
> > > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=101891
> > > > Signed-off-by: Dāvis Mosāns <davispuh@gmail.com>
> > > > ---
> > > > drivers/scsi/mvsas/mv_sas.c | 2 ++
> > > > 1 file changed, 2 insertions(+)
> > > >
> > > > diff --git a/drivers/scsi/mvsas/mv_sas.c
> > > > b/drivers/scsi/mvsas/mv_sas.c
> > > > index 454536c..9c78074 100644
> > > > --- a/drivers/scsi/mvsas/mv_sas.c
> > > > +++ b/drivers/scsi/mvsas/mv_sas.c
> > > > @@ -887,6 +887,8 @@ static void mvs_slot_free(struct mvs_info
> > > > *mvi,
> > > > u32 rx_desc)
> > > > static void mvs_slot_task_free(struct mvs_info *mvi, struct
> > > > sas_task *task,
> > > > struct mvs_slot_info *slot, u32
> > > > slot_idx)
> > > > {
> > > > + if (!slot)
> > > > + return;
> > > > if (!slot->task)
> > > > return;
> > > > if (!sas_protocol_ata(task->task_proto))
> > > > --
> > > > 2.5.0
> > > >
> > >
> > > Can this get merged?
> > > So far since august it have saved me from several kernel crashes.
> >
> > If it saved you from several crashes, it probably should be tagged
> > for
> > stable, shouldn't it?
> >
> > Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
> >
> >
>
> I don't really know how that works... this is my first patch so I'm
> not really concerned about in which version it gets in as long as it
> does.
> I've been compiling kernel with this patch for these months so for me
> it
> doesn't really make any difference.
You can add
Cc: stable@vger.kernel.org
somewhere around your Signed-off-by
Documentation/stable_kernel_rules.txt has all the process
documentation.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free
2015-10-21 13:47 ` Johannes Thumshirn
(?)
@ 2015-10-21 14:26 ` Dāvis Mosāns
2015-10-21 14:29 ` Johannes Thumshirn
-1 siblings, 1 reply; 11+ messages in thread
From: Dāvis Mosāns @ 2015-10-21 14:26 UTC (permalink / raw)
To: Johannes Thumshirn; +Cc: James E.J. Bottomley, linux-scsi, linux-kernel
2015-10-21 16:47 GMT+03:00 Johannes Thumshirn <jthumshirn@suse.de>:
> On Wed, 2015-10-21 at 16:18 +0300, Dāvis Mosāns wrote:
>> 2015-10-21 10:33 GMT+03:00 Johannes Thumshirn <jthumshirn@suse.de>:
>> > On Tue, 2015-10-20 at 20:41 +0300, Dāvis Mosāns wrote:
>> > > 2015-08-21 7:29 GMT+03:00 Dāvis Mosāns <davispuh@gmail.com>:
>> > > > When pci_pool_alloc fails in mvs_task_prep then task->lldd_task
>> > > > stays
>> > > > NULL but it's later used in mvs_abort_task as slot which is
>> > > > passed
>> > > > to mvs_slot_task_free causing NULL pointer dereference.
>> > > >
>> > > > Just return from mvs_slot_task_free when passed with NULL slot.
>> > > >
>> > > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=101891
>> > > > Signed-off-by: Dāvis Mosāns <davispuh@gmail.com>
>> > > > ---
>> > > > drivers/scsi/mvsas/mv_sas.c | 2 ++
>> > > > 1 file changed, 2 insertions(+)
>> > > >
>> > > > diff --git a/drivers/scsi/mvsas/mv_sas.c
>> > > > b/drivers/scsi/mvsas/mv_sas.c
>> > > > index 454536c..9c78074 100644
>> > > > --- a/drivers/scsi/mvsas/mv_sas.c
>> > > > +++ b/drivers/scsi/mvsas/mv_sas.c
>> > > > @@ -887,6 +887,8 @@ static void mvs_slot_free(struct mvs_info
>> > > > *mvi,
>> > > > u32 rx_desc)
>> > > > static void mvs_slot_task_free(struct mvs_info *mvi, struct
>> > > > sas_task *task,
>> > > > struct mvs_slot_info *slot, u32
>> > > > slot_idx)
>> > > > {
>> > > > + if (!slot)
>> > > > + return;
>> > > > if (!slot->task)
>> > > > return;
>> > > > if (!sas_protocol_ata(task->task_proto))
>> > > > --
>> > > > 2.5.0
>> > > >
>> > >
>> > > Can this get merged?
>> > > So far since august it have saved me from several kernel crashes.
>> >
>> > If it saved you from several crashes, it probably should be tagged
>> > for
>> > stable, shouldn't it?
>> >
>> > Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
>> >
>> >
>>
>> I don't really know how that works... this is my first patch so I'm
>> not really concerned about in which version it gets in as long as it
>> does.
>> I've been compiling kernel with this patch for these months so for me
>> it
>> doesn't really make any difference.
>
> You can add
> Cc: stable@vger.kernel.org
> somewhere around your Signed-off-by
>
> Documentation/stable_kernel_rules.txt has all the process
> documentation.
Should I add it together with review tags too and resend patch or reply to this
thread with it edited or just leave it like it is now and whoever will see it
will add it himself?
also for stable requirements this line is a bit confusing
"It or an equivalent fix must already exist in Linus' tree (upstream)."
but then later seems it's not requirement for Option 1
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free
2015-10-21 14:26 ` Dāvis Mosāns
@ 2015-10-21 14:29 ` Johannes Thumshirn
0 siblings, 0 replies; 11+ messages in thread
From: Johannes Thumshirn @ 2015-10-21 14:29 UTC (permalink / raw)
To: Dāvis Mosāns; +Cc: James E.J. Bottomley, linux-scsi, linux-kernel
On Wed, 2015-10-21 at 17:26 +0300, Dāvis Mosāns wrote:
> 2015-10-21 16:47 GMT+03:00 Johannes Thumshirn <jthumshirn@suse.de>:
> > On Wed, 2015-10-21 at 16:18 +0300, Dāvis Mosāns wrote:
> > > 2015-10-21 10:33 GMT+03:00 Johannes Thumshirn <jthumshirn@suse.de
> > > >:
> > > > On Tue, 2015-10-20 at 20:41 +0300, Dāvis Mosāns wrote:
> > > > > 2015-08-21 7:29 GMT+03:00 Dāvis Mosāns <davispuh@gmail.com>:
> > > > > > When pci_pool_alloc fails in mvs_task_prep then task-
> > > > > > >lldd_task
> > > > > > stays
> > > > > > NULL but it's later used in mvs_abort_task as slot which is
> > > > > > passed
> > > > > > to mvs_slot_task_free causing NULL pointer dereference.
> > > > > >
> > > > > > Just return from mvs_slot_task_free when passed with NULL
> > > > > > slot.
> > > > > >
> > > > > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=10189
> > > > > > 1
> > > > > > Signed-off-by: Dāvis Mosāns <davispuh@gmail.com>
> > > > > > ---
> > > > > > drivers/scsi/mvsas/mv_sas.c | 2 ++
> > > > > > 1 file changed, 2 insertions(+)
> > > > > >
> > > > > > diff --git a/drivers/scsi/mvsas/mv_sas.c
> > > > > > b/drivers/scsi/mvsas/mv_sas.c
> > > > > > index 454536c..9c78074 100644
> > > > > > --- a/drivers/scsi/mvsas/mv_sas.c
> > > > > > +++ b/drivers/scsi/mvsas/mv_sas.c
> > > > > > @@ -887,6 +887,8 @@ static void mvs_slot_free(struct
> > > > > > mvs_info
> > > > > > *mvi,
> > > > > > u32 rx_desc)
> > > > > > static void mvs_slot_task_free(struct mvs_info *mvi,
> > > > > > struct
> > > > > > sas_task *task,
> > > > > > struct mvs_slot_info *slot, u32
> > > > > > slot_idx)
> > > > > > {
> > > > > > + if (!slot)
> > > > > > + return;
> > > > > > if (!slot->task)
> > > > > > return;
> > > > > > if (!sas_protocol_ata(task->task_proto))
> > > > > > --
> > > > > > 2.5.0
> > > > > >
> > > > >
> > > > > Can this get merged?
> > > > > So far since august it have saved me from several kernel
> > > > > crashes.
> > > >
> > > > If it saved you from several crashes, it probably should be
> > > > tagged
> > > > for
> > > > stable, shouldn't it?
> > > >
> > > > Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
> > > >
> > > >
> > >
> > > I don't really know how that works... this is my first patch so
> > > I'm
> > > not really concerned about in which version it gets in as long as
> > > it
> > > does.
> > > I've been compiling kernel with this patch for these months so
> > > for me
> > > it
> > > doesn't really make any difference.
> >
> > You can add
> > Cc: stable@vger.kernel.org
> > somewhere around your Signed-off-by
> >
> > Documentation/stable_kernel_rules.txt has all the process
> > documentation.
>
> Should I add it together with review tags too and resend patch or
> reply to this
> thread with it edited or just leave it like it is now and whoever
> will see it
> will add it himself?
good question, but I think James can help here.
>
>
> also for stable requirements this line is a bit confusing
> "It or an equivalent fix must already exist in Linus' tree
> (upstream)."
>
> but then later seems it's not requirement for Option 1
yes, if you tag it with the Cc, it will get to stable review _after_ it
is applied to Linus' tree
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2015-10-21 14:29 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-08-21 4:29 [PATCH] SCSI: mvsas: Fix NULL pointer dereference in mvs_slot_task_free Dāvis Mosāns
2015-09-01 20:08 ` Dāvis Mosāns
2015-09-02 12:08 ` Tomas Henzl
2015-10-20 17:41 ` Dāvis Mosāns
2015-10-21 7:33 ` Johannes Thumshirn
2015-10-21 13:18 ` Dāvis Mosāns
2015-10-21 13:18 ` Dāvis Mosāns
2015-10-21 13:47 ` Johannes Thumshirn
2015-10-21 13:47 ` Johannes Thumshirn
2015-10-21 14:26 ` Dāvis Mosāns
2015-10-21 14:29 ` Johannes Thumshirn
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.