[U-Boot] [PATCH v2] mmc/dwmmc: Using calloc instead malloc

Chin Liang See clsee at altera.com
Wed Dec 18 18:16:01 CET 2013


To enhance the SDMMC DesignWare driver to use calloc instead of
malloc. This will avoid the incident that uninitialized members
of mmc structure are later used for NULL comparison.

Signed-off-by: Chin Liang See <clsee at altera.com>
Cc: Rajeshwari Shinde <rajeshwari.s at samsung.com>
Cc: Jaehoon Chung <jh80.chung at samsung.com>
Cc: Mischa Jonker <mjonker at synopsys.com>
Cc: Alexey Brodkin <abrodkin at synopsys.com>
Cc: Andy Fleming <afleming at freescale.com>
Cc: Pantelis Antoniou <panto at antoniou-consulting.com>
---
Changes for v2
- Adding u-boot-mmc maintainer
---
 drivers/mmc/dw_mmc.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 mode change 100644 => 100755 drivers/mmc/dw_mmc.c

diff --git a/drivers/mmc/dw_mmc.c b/drivers/mmc/dw_mmc.c
old mode 100644
new mode 100755
index 19d9b0b..82abe19
--- a/drivers/mmc/dw_mmc.c
+++ b/drivers/mmc/dw_mmc.c
@@ -336,9 +336,9 @@ int add_dwmci(struct dwmci_host *host, u32 max_clk, u32 min_clk)
 	struct mmc *mmc;
 	int err = 0;
 
-	mmc = malloc(sizeof(struct mmc));
+	mmc = calloc(sizeof(struct mmc), 1);
 	if (!mmc) {
-		printf("mmc malloc fail!\n");
+		printf("mmc calloc fail!\n");
 		return -1;
 	}
 
-- 
1.7.9.5




More information about the U-Boot mailing list