On Sun, Dec 27, 2020 at 10:26:00AM +0100, Heinrich Schuchardt wrote:

> Calling calloc() for 0 members does not make any sense.
> Setting ch_priv->busy_desc = NULL for ch_priv->desc_cnt > 0 is equally
> unreasonable.
> 
> The current code will lead to a NULL dereference in bcm6348_iudma_enable().
> 
> The assignments for ch_priv->busy_desc are obviously swapped.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>

Applied to u-boot/master, thanks!

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to