Messages in this thread Patch in this message | | | From | Rasmus Villemoes <> | Subject | [PATCH] scsi: megaraid_sas: Prevent future %p disaster | Date | Fri, 6 Feb 2015 15:33:41 +0100 |
| |
There is currently no %po format extension, so currently the letters "on" are simply skipped and the pointer is printed as expected (while missing the word on). However, it is probably only a matter of time before someone comes up with a %po extension, at which point this is likely to fail spectacularly.
Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk> --- drivers/scsi/megaraid/megaraid_sas_base.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/megaraid/megaraid_sas_base.c b/drivers/scsi/megaraid/megaraid_sas_base.c index ff283d23788a..e7c6b9c946d6 100644 --- a/drivers/scsi/megaraid/megaraid_sas_base.c +++ b/drivers/scsi/megaraid/megaraid_sas_base.c @@ -3105,7 +3105,7 @@ megasas_internal_reset_defer_cmds(struct megasas_instance *instance) for (i = 0; i < max_cmd; i++) { cmd = instance->cmd_list[i]; if (cmd->sync_cmd == 1 || cmd->scmd) { - printk(KERN_NOTICE "megasas: moving cmd[%d]:%p:%d:%p" + printk(KERN_NOTICE "megasas: moving cmd[%d]:%p:%d:%p " "on the defer queue as internal\n", defer_index, cmd, cmd->sync_cmd, cmd->scmd); -- 2.1.3
| |