hpsa: fix an sprintf() overflow in the reset handler
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 4 Jun 2015 14:47:56 +0000 (17:47 +0300)
committerJames Bottomley <JBottomley@Odin.com>
Thu, 30 Jul 2015 20:11:40 +0000 (13:11 -0700)
The string "cmd %d RESET FAILED, new lockup detected" is not quite
large enough so the sprintf() will overflow.  I have increased the size
of the buffer and also changed the sprintf calls to snprintf.

Fixes: 73153fe533bc ('hpsa: use block layer tag for command allocation')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Don Brace <don.brace@pmcs.com>
Signed-off-by: James Bottomley <JBottomley@Odin.com>
drivers/scsi/hpsa.c

index 1dafeb43333b873d4433e4a2955328a88d3a6448..cab4e98b2b0e81ee7f45a08c152fb3cd1d509ba2 100644 (file)
@@ -5104,7 +5104,7 @@ static int hpsa_eh_device_reset_handler(struct scsi_cmnd *scsicmd)
        int rc;
        struct ctlr_info *h;
        struct hpsa_scsi_dev_t *dev;
-       char msg[40];
+       char msg[48];
 
        /* find the controller to which the command to be aborted was sent */
        h = sdev_to_hba(scsicmd->device);
@@ -5122,16 +5122,18 @@ static int hpsa_eh_device_reset_handler(struct scsi_cmnd *scsicmd)
 
        /* if controller locked up, we can guarantee command won't complete */
        if (lockup_detected(h)) {
-               sprintf(msg, "cmd %d RESET FAILED, lockup detected",
-                               hpsa_get_cmd_index(scsicmd));
+               snprintf(msg, sizeof(msg),
+                        "cmd %d RESET FAILED, lockup detected",
+                        hpsa_get_cmd_index(scsicmd));
                hpsa_show_dev_msg(KERN_WARNING, h, dev, msg);
                return FAILED;
        }
 
        /* this reset request might be the result of a lockup; check */
        if (detect_controller_lockup(h)) {
-               sprintf(msg, "cmd %d RESET FAILED, new lockup detected",
-                               hpsa_get_cmd_index(scsicmd));
+               snprintf(msg, sizeof(msg),
+                        "cmd %d RESET FAILED, new lockup detected",
+                        hpsa_get_cmd_index(scsicmd));
                hpsa_show_dev_msg(KERN_WARNING, h, dev, msg);
                return FAILED;
        }
@@ -5145,7 +5147,8 @@ static int hpsa_eh_device_reset_handler(struct scsi_cmnd *scsicmd)
        /* send a reset to the SCSI LUN which the command was sent to */
        rc = hpsa_do_reset(h, dev, dev->scsi3addr, HPSA_RESET_TYPE_LUN,
                           DEFAULT_REPLY_QUEUE);
-       sprintf(msg, "reset %s", rc == 0 ? "completed successfully" : "failed");
+       snprintf(msg, sizeof(msg), "reset %s",
+                rc == 0 ? "completed successfully" : "failed");
        hpsa_show_dev_msg(KERN_WARNING, h, dev, msg);
        return rc == 0 ? SUCCESS : FAILED;
 }