[PATCH] dfu: memory leak in dfu_free_entities()

[Heinrich Schuchardt]

On 18.03.21 12:18, Heinrich Schuchardt wrote:
> We must free all entries that we remove from the list not just one to avoid
> a memory leak.
>
> With list_for_each_entry_safe_reverse() it is safe to delete the current
> entry as it provides a second pointer for the next entry.

I got that wrong:

dfu_alt_init() reserves memory for dfu_alt_num entries at once.
So must call free() only once.

Sorry for the noise.

Best regards

Heinrich