Re: [PATCH v2] kernel: escape non-ASCII and control characters inprintk()

From: Vasiliy Kulikov
Date: Sun Jul 03 2011 - 07:42:52 EST


On Sun, Jul 03, 2011 at 14:01 +0400, Vasiliy Kulikov wrote:
> diff --git a/drivers/scsi/cxgbi/cxgb3i/cxgb3i.c b/drivers/scsi/cxgbi/cxgb3i/cxgb3i.c
> index fc2cdb6..b2a659b 100644
> --- a/drivers/scsi/cxgbi/cxgb3i/cxgb3i.c
> +++ b/drivers/scsi/cxgbi/cxgb3i/cxgb3i.c
> @@ -38,7 +38,7 @@ static unsigned int dbg_level;
>
> static char version[] =
> DRV_MODULE_DESC " " DRV_MODULE_NAME
> - " v" DRV_MODULE_VERSION " (" DRV_MODULE_RELDATE ")\n";
> + " v" DRV_MODULE_VERSION " (" DRV_MODULE_RELDATE ")";
>
> MODULE_AUTHOR("Chelsio Communications, Inc.");
> MODULE_DESCRIPTION(DRV_MODULE_DESC);
> @@ -1402,7 +1402,7 @@ static int __init cxgb3i_init_module(void)
> {
> int rc;
>
> - printk(KERN_INFO "%s", version);
> + printk(KERN_INFO "%s\b", version);

Surely this and init/main.c should have \n, sorry.

--
Vasiliy Kulikov
http://www.openwall.com - bringing security into open computing environments
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/