From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1160998AbWJKGXf (ORCPT ); Wed, 11 Oct 2006 02:23:35 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1161001AbWJKGXf (ORCPT ); Wed, 11 Oct 2006 02:23:35 -0400 Received: from py-out-1112.google.com ([64.233.166.179]:14468 "EHLO py-out-1112.google.com") by vger.kernel.org with ESMTP id S1160998AbWJKGXe (ORCPT ); Wed, 11 Oct 2006 02:23:34 -0400 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:date:from:to:subject:message-id:organization:x-mailer:mime-version:content-type:content-transfer-encoding; b=rXTdZ5kgLt62xbqzDhNkg1VUIwiWy8Zp81e/hsqv7bhBxJe+MQEnkB86Ibl2B9rNZRFSjWUoa/vennnB/TC/XIIBcF1QOfFj6MGf2/ISf8nvp4d6glo39PHJpnlGWdBtDdbGgajQbsFsrAprq3SwqHT6tlj1D4E0r1LXZ7FFZzE= Date: Tue, 10 Oct 2006 23:23:30 -0700 From: Amit Choudhary To: Linux Kernel Subject: [PATCH 2.6.19-rc1] drivers/media/video/se401.c: check kmalloc() return value. Message-Id: <20061010232330.51cf48dd.amit2030@gmail.com> Organization: X X-Mailer: Sylpheed version 2.2.9 (GTK+ 2.8.15; i686-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Description: Check the return value of kmalloc() in function se401_start_stream(), in file drivers/media/video/se401.c. Signed-off-by: Amit Choudhary diff --git a/drivers/media/video/se401.c b/drivers/media/video/se401.c index 7aeec57..20b91db 100644 --- a/drivers/media/video/se401.c +++ b/drivers/media/video/se401.c @@ -57,6 +57,7 @@ MODULE_PARM_DESC(flickerless, "Net frequ module_param(video_nr, int, 0); static struct usb_driver se401_driver; +static int se401_stop_stream(struct usb_se401 *se401); /********************************************************************** @@ -450,6 +451,11 @@ static int se401_start_stream(struct usb } for (i=0; isbuf[i].data=kmalloc(SE401_PACKETSIZE, GFP_KERNEL); + /* No memory, stop streaming */ + if (!se401->sbuf[i].data) { + se401_stop_stream(se401); + return -ENOMEM; + } } se401->bayeroffset=0; @@ -458,6 +464,11 @@ static int se401_start_stream(struct usb se401->scratch_overflow=0; for (i=0; iscratch[i].data=kmalloc(SE401_PACKETSIZE, GFP_KERNEL); + /* No memory, stop streaming */ + if (!se401->scratch[i].data) { + se401_stop_stream(se401); + return -ENOMEM; + } se401->scratch[i].state=BUFFER_UNUSED; }