]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
scsi: fnic: add a space after %p in printf format
authorNicolas Iooss <nicolas.iooss_linux@m4x.org>
Sun, 10 Dec 2017 19:23:11 +0000 (20:23 +0100)
committerJack Vogel <jack.vogel@oracle.com>
Wed, 7 Mar 2018 04:11:13 +0000 (20:11 -0800)
fnic_fcpio_icmnd_cmpl_handler() displays the value of sc with:

    FNIC_SCSI_DBG(KERN_INFO...
        "... sc = 0x%p"
        "scsi_status ..."
        ...

As the literal strings get merged, the function uses %ps instead of the
intended raw %p format. Fix this by inserting a space.

Signed-off-by: Nicolas Iooss <nicolas.iooss_linux@m4x.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
(cherry picked from commit f280c77dc9bb850bc49a126ef5a088e7340a61b6)

Orabug: 27587343
Signed-off-by: Jack Vogel <jack.vogel@oracle.com>
Reviewed-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/fnic/fnic_scsi.c

index cf8119bc672b831e76d76bea4a2ace393c8c052f..d036b758f8024044b4bfcbbde9f65af3e8df7896 100644 (file)
@@ -906,7 +906,7 @@ static void fnic_fcpio_icmnd_cmpl_handler(struct fnic *fnic,
 
                FNIC_SCSI_DBG(KERN_INFO, fnic->lport->host,
                        "icmnd_cmpl abts pending "
-                         "hdr status = %s tag = 0x%x sc = 0x%p"
+                         "hdr status = %s tag = 0x%x sc = 0x%p "
                          "scsi_status = %x residual = %d\n",
                          fnic_fcpio_status_to_str(hdr_status),
                          id, sc,