aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBen Hutchings <ben@decadent.org.uk>2013-05-27 19:07:19 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2013-07-13 10:34:44 -0700
commit3bb0ab9e10eecc932df94807cacb90ca30010cb7 (patch)
tree03da8548124ada33b9fbf50362bb66340fbe093a
parent5583db3830ede897a8e4709452bcf0bc5266bdd9 (diff)
downloadkernel_samsung_aries-3bb0ab9e10eecc932df94807cacb90ca30010cb7.zip
kernel_samsung_aries-3bb0ab9e10eecc932df94807cacb90ca30010cb7.tar.gz
kernel_samsung_aries-3bb0ab9e10eecc932df94807cacb90ca30010cb7.tar.bz2
SCSI: sd: Fix parsing of 'temporary ' cache mode prefix
commit 2ee3e26c673e75c05ef8b914f54fadee3d7b9c88 upstream. Commit 39c60a0948cc '[SCSI] sd: fix array cache flushing bug causing performance problems' added temp as a pointer to "temporary " and used sizeof(temp) - 1 as its length. But sizeof(temp) is the size of the pointer, not the size of the string constant. Change temp to a static array so that sizeof() does what was intended. Signed-off-by: Ben Hutchings <ben@decadent.org.uk> Signed-off-by: James Bottomley <JBottomley@Parallels.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/scsi/sd.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c
index 7db7eb7..43c7a90 100644
--- a/drivers/scsi/sd.c
+++ b/drivers/scsi/sd.c
@@ -138,7 +138,7 @@ sd_store_cache_type(struct device *dev, struct device_attribute *attr,
char *buffer_data;
struct scsi_mode_data data;
struct scsi_sense_hdr sshdr;
- const char *temp = "temporary ";
+ static const char temp[] = "temporary ";
int len;
if (sdp->type != TYPE_DISK)