[U-Boot] [PATCH] fsl/sata: Replace sprintf() with snprintf()
Yuantian.Tang at freescale.com
Yuantian.Tang at freescale.com
Wed May 6 04:55:03 CEST 2015
From: Tang Yuantian <Yuantian.Tang at freescale.com>
Function 'sprintf' does not check buffer boundaries but outputs
to the buffer of fixed size which could cause buffer overflow.
Use a safer function to replace it.
Signed-off-by: Tang Yuantian <Yuantian.Tang at freescale.com>
---
drivers/block/fsl_sata.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/block/fsl_sata.c b/drivers/block/fsl_sata.c
index 71d7cec..e720c3e 100644
--- a/drivers/block/fsl_sata.c
+++ b/drivers/block/fsl_sata.c
@@ -113,7 +113,7 @@ int init_sata(int dev)
/* Save the private struct to block device struct */
sata_dev_desc[dev].priv = (void *)sata;
- sprintf(sata->name, "SATA%d", dev);
+ snprintf(sata->name, 7, "SATA%d", dev);
/* Set the controller register base address to device struct */
reg = (fsl_sata_reg_t *)(fsl_sata_info[dev].sata_reg_base);
--
2.1.0.27.g96db324
More information about the U-Boot
mailing list