diff options
author | 2010-10-29 16:50:44 +0200 | |
---|---|---|
committer | 2010-10-29 16:50:50 +0200 | |
commit | f2166bb117952404689f0dc42624ee0b54a66e5e (patch) | |
tree | d77eebb80e4f0206a07f574ecd2af87f745d4bc3 /drivers/s390/char/tape_core.c | |
parent | [S390] dasd: provide a Sense Path Group ID ioctl (diff) | |
download | linux-f2166bb117952404689f0dc42624ee0b54a66e5e.tar.gz linux-f2166bb117952404689f0dc42624ee0b54a66e5e.tar.bz2 linux-f2166bb117952404689f0dc42624ee0b54a66e5e.zip |
[S390] tape: fix dbf usage
Get rid of the format string "%s" usage with volatile strings
to prevent use after free errors in the s390dbf.
Signed-off-by: Sebastian Ott <sebott@linux.vnet.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'drivers/s390/char/tape_core.c')
-rw-r--r-- | drivers/s390/char/tape_core.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/s390/char/tape_core.c b/drivers/s390/char/tape_core.c index 29c2d73d719d..6c408670e08d 100644 --- a/drivers/s390/char/tape_core.c +++ b/drivers/s390/char/tape_core.c @@ -1077,15 +1077,14 @@ __tape_do_irq (struct ccw_device *cdev, unsigned long intparm, struct irb *irb) /* FIXME: What to do with the request? */ switch (PTR_ERR(irb)) { case -ETIMEDOUT: - DBF_LH(1, "(%s): Request timed out\n", - dev_name(&cdev->dev)); + DBF_LH(1, "(%08x): Request timed out\n", + device->cdev_id); case -EIO: __tape_end_request(device, request, -EIO); break; default: - DBF_LH(1, "(%s): Unexpected i/o error %li\n", - dev_name(&cdev->dev), - PTR_ERR(irb)); + DBF_LH(1, "(%08x): Unexpected i/o error %li\n", + device->cdev_id, PTR_ERR(irb)); } return; } |