[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Tue, 10 Oct 2006 09:31:16 -0700
From: Greg KH <greg@...ah.com>
To: Manu Abraham <abraham.manu@...il.com>
Cc: Amit Choudhary <amit2030@...il.com>,
v4l-dvb maintainer list <v4l-dvb-maintainer@...uxtv.org>,
Linux Kernel <linux-kernel@...r.kernel.org>, stable@...nel.org
Subject: Re: [stable] [PATCH 2.6.19-rc1] drivers/media/dvb/bt8xx/dvb-bt8xx.c: check kmalloc() return value.
On Tue, Oct 10, 2006 at 03:18:58PM +0400, Manu Abraham wrote:
> Greg KH wrote:
> > On Mon, Oct 09, 2006 at 12:34:41PM +0400, Manu Abraham wrote:
> >> Amit Choudhary wrote:
> >>> Description: Check the return value of kmalloc() in function frontend_init(), in file drivers/media/dvb/bt8xx/dvb-bt8xx.c.
> >>>
> >>> Signed-off-by: Amit Choudhary <amit2030@...il.com>
> >>>
> >>> diff --git a/drivers/media/dvb/bt8xx/dvb-bt8xx.c b/drivers/media/dvb/bt8xx/dvb-bt8xx.c
> >>> index fb6c4cc..14e69a7 100644
> >>> --- a/drivers/media/dvb/bt8xx/dvb-bt8xx.c
> >>> +++ b/drivers/media/dvb/bt8xx/dvb-bt8xx.c
> >>> @@ -665,6 +665,10 @@ static void frontend_init(struct dvb_bt8
> >>> case BTTV_BOARD_TWINHAN_DST:
> >>> /* DST is not a frontend driver !!! */
> >>> state = (struct dst_state *) kmalloc(sizeof (struct dst_state), GFP_KERNEL);
> >>> + if (!state) {
> >>> + printk("dvb_bt8xx: No memory\n");
> >>> + break;
> >>> + }
> >>> /* Setup the Card */
> >>> state->config = &dst_config;
> >>> state->i2c = card->i2c_adapter;
> >>> -
> >>
> >> Signed-off-by: Manu Abraham <manu@...uxtv.org>
> >
> > Care to send the full patch in a format that we can apply it to the
> > -stable tree?
> >
>
>
> dvb-bt8xx.c | 4 ++++
> 1 files changed, 4 insertions(+)
>
>
> Thanks,
Um, can you resend it with the proper description and signed-off-by:
lines so that it can be applied correctly?
And does this solve a real bug, or is it just added error condition
checks? If the latter, I don't think it's ok for -stable right now.
thanks,
greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists