[PATCH 1/1] dma: bcm6348: incorrect buffer allocation

Tom Rini trini at konsulko.com
Tue Jan 19 14:05:24 CET 2021


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.glpk at gmx.de>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210119/58ee34af/attachment.sig>


More information about the U-Boot mailing list