diff options
author | Sebastian Ott <sebott@linux.vnet.ibm.com> | 2010-10-25 10:10:46 -0400 |
---|---|---|
committer | Martin Schwidefsky <sky@mschwide.boeblingen.de.ibm.com> | 2010-10-25 10:10:20 -0400 |
commit | 26cffecf84c8cb33787dd13a72bd2124d107d413 (patch) | |
tree | aa50aaad1c06c9a3adb5d5bc15066f6167c2851d /drivers/s390 | |
parent | e05ef9bdb899e2f3798be74691842fc597d8ce60 (diff) |
[S390] dasd: fix use after free in dbf
Writing to /proc/dasd/statistics while the debug level of the
generic dasd debug entry is set to DBF_DEBUG will lead to an
use after free when accessing the debug entry later.
Since for the format string "%s" in the s390 dbf only a pointer
to the string is stored in the debug feature and the buffer used
here is freed afterwards.
To fix this just remove the debug message.
Signed-off-by: Sebastian Ott <sebott@linux.vnet.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'drivers/s390')
-rw-r--r-- | drivers/s390/block/dasd_proc.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/s390/block/dasd_proc.c b/drivers/s390/block/dasd_proc.c index 2eb025592809..c4a6a31bd9cd 100644 --- a/drivers/s390/block/dasd_proc.c +++ b/drivers/s390/block/dasd_proc.c | |||
@@ -251,7 +251,6 @@ static ssize_t dasd_stats_proc_write(struct file *file, | |||
251 | buffer = dasd_get_user_string(user_buf, user_len); | 251 | buffer = dasd_get_user_string(user_buf, user_len); |
252 | if (IS_ERR(buffer)) | 252 | if (IS_ERR(buffer)) |
253 | return PTR_ERR(buffer); | 253 | return PTR_ERR(buffer); |
254 | DBF_EVENT(DBF_DEBUG, "/proc/dasd/statictics: '%s'\n", buffer); | ||
255 | 254 | ||
256 | /* check for valid verbs */ | 255 | /* check for valid verbs */ |
257 | str = skip_spaces(buffer); | 256 | str = skip_spaces(buffer); |