LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] firewire: fw-sbp2: fix NULL pointer deref in slave_alloc
@ 2008-02-17 13:56 Stefan Richter
  2008-02-17 13:57 ` [PATCH] ieee1394: sbp2: fix rescan-scsi-bus Stefan Richter
  2008-02-17 17:44 ` [PATCH] firewire: fw-sbp2: fix NULL pointer deref in slave_alloc Stefan Richter
  0 siblings, 2 replies; 8+ messages in thread
From: Stefan Richter @ 2008-02-17 13:56 UTC (permalink / raw)
  To: linux1394-devel; +Cc: linux-kernel, linux-scsi

Fix a kernel bug when running rescan-scsi-bus while a FireWire disk is
connected:  http://bugzilla.kernel.org/show_bug.cgi?id=10008

Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
---
 drivers/firewire/fw-sbp2.c |    4 ++++
 1 file changed, 4 insertions(+)

Index: linux/drivers/firewire/fw-sbp2.c
===================================================================
--- linux.orig/drivers/firewire/fw-sbp2.c
+++ linux/drivers/firewire/fw-sbp2.c
@@ -1473,6 +1473,10 @@ static int sbp2_scsi_slave_alloc(struct 
 {
 	struct sbp2_logical_unit *lu = sdev->hostdata;
 
+	/* (Re-)Adding logical units via the SCSI stack is not supported. */
+	if (!lu)
+		return -ENOSYS;
+
 	sdev->allow_restart = 1;
 
 	/*

-- 
Stefan Richter
-=====-==--- --=- =---=
http://arcgraph.de/sr/


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2008-02-19  8:06 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-02-17 13:56 [PATCH] firewire: fw-sbp2: fix NULL pointer deref in slave_alloc Stefan Richter
2008-02-17 13:57 ` [PATCH] ieee1394: sbp2: fix rescan-scsi-bus Stefan Richter
2008-02-17 16:03   ` James Bottomley
2008-02-17 17:58     ` Stefan Richter
2008-02-17 21:08       ` Stefan Richter
2008-02-17 17:44 ` [PATCH] firewire: fw-sbp2: fix NULL pointer deref in slave_alloc Stefan Richter
2008-02-17 18:17   ` [PATCH] firewire: fw-sbp2: fix NULL pointer deref in scsi_remove_device Stefan Richter
2008-02-19  8:05     ` [PATCH update] " Stefan Richter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).