arch: x86: platform: mrst: pr_err() need notice that 'sdev->modalias' may not a NUL terminated string.

According to the areas which use "%16.16s" within this file, the 'sdev->modalias' may not always be a NUL terminated string.

So need use "%.16s" instead of "%s" in pr_err(), or may cause issue.

---
 arch/x86/platform/mrst/mrst.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

index a0a0a43..e2e069c 100644
@@ -746,7 +746,7 @@ static void __init intel_scu_spi_device_register(struct spi_board_info *sdev)
 
 	new_dev = kzalloc(sizeof(*sdev), GFP_KERNEL);
 	if (!new_dev) {
-		pr_err("failed to alloc mem for delayed spi dev %s\n",
+		pr_err("failed to alloc mem for delayed spi dev %.16s\n",
 			sdev->modalias);
 		return;
 	}

1.7.7.6

No. spi_match_id() expects a NULL terminated string and I would be suprised if it would be otherwise. Therefore moorsetown or this files has to learn to NULL terminate its strings.

Sebastian

In spi_match_id(), it is 'spi_device', not 'spi_board_info'. ;-)

But all together, it is my fault, the original implementation has no
issue: "I treated SFI_NAME_LEN as SPI_NAME_LEN when strncpy()".

BTW: 'pentry->name' in sfi_parse_devs() may not always be a NUL terminated string, could you give a check, too ? ;-)

Thanks.

Chen Gang

Asianux Corporation