drivers: dma: Zero-initialize memory allocated using kmalloc

mmahadevan108 · nashif · commit 91f172e4adaa · 2021-09-01T12:34:50.000-04:00
The code was wrongly calling DMA_Abort on a channel
that not initialized. This fixes Issue#38078

Signed-off-by: Mahesh Mahadevan &lt;mahesh.mahadevan@nxp.com&gt;
diff --git a/drivers/dma/dma_mcux_lpc.c b/drivers/dma/dma_mcux_lpc.c
@@ -485,9 +485,7 @@ static int dma_mcux_lpc_init(const struct device *dev)
 		return -ENOMEM;
 	}
 
-	for (int i = 0; i < DEV_CFG(dev)->num_of_channels; i++) {
-		data->data_cb[i].dma_descriptor_table = NULL;
-	}
+	memset(data->data_cb, 0, size_channel_data);
 
 #if defined FSL_FEATURE_DMA_NUMBER_OF_CHANNELS
 	total_dma_channels = FSL_FEATURE_DMA_NUMBER_OF_CHANNELS;

Original file line number	Diff line number	Diff line change
`@@ -485,9 +485,7 @@ static int dma_mcux_lpc_init(const struct device *dev)`
`485`	`485`	`return -ENOMEM;`
`486`	`486`	`}`
`487`	`487`
`488`		`- for (int i = 0; i < DEV_CFG(dev)->num_of_channels; i++) {`
`489`		`- data->data_cb[i].dma_descriptor_table = NULL;`
`490`		`- }`
	`488`	`+ memset(data->data_cb, 0, size_channel_data);`
`491`	`489`
`492`	`490`	`#if defined FSL_FEATURE_DMA_NUMBER_OF_CHANNELS`
`493`	`491`	`total_dma_channels = FSL_FEATURE_DMA_NUMBER_OF_CHANNELS;`