From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rusty Trivial Russell Subject: [TRIVIAL] [Trivial Patch] scsi_register-004 Date: Mon, 16 Dec 2002 21:39:43 +1100 Sender: linux-scsi-owner@vger.kernel.org Message-ID: <20021216210038.43BA32C293@lists.samba.org> Return-path: List-Id: linux-scsi@vger.kernel.org To: Marcelo Tosatti Cc: Achim Leubner , linux-scsi@vger.kernel.org (Included in 2.5) From: Michael Still This patch has been generated as part of my scsi_register audit of the 2.5.45 kernel tree. scsi_register() can fail, returning a NULL pointer, and the failure case was not being handled here... --- trivial-2.4.21-pre1/drivers/scsi/gdth.c.orig 2002-12-16 17:22:28.000000000 +1100 +++ trivial-2.4.21-pre1/drivers/scsi/gdth.c 2002-12-16 17:22:28.000000000 +1100 @@ -4029,6 +4029,9 @@ break; if (gdth_search_isa(isa_bios)) { /* controller found */ shp = scsi_register(shtp,sizeof(gdth_ext_str)); + if(shp == NULL) + continue; + ha = HADATA(shp); if (!gdth_init_isa(isa_bios,ha)) { scsi_unregister(shp); -- Don't blame me: the Monkey is driving File: Michael Still : [Trivial Patch] scsi_register-004