Re: [PATCH v2 1/6] staging: rtl8188eu: use safe iterator in rtw_free_network_queue

Dan Carpenter
2021-05-18T07:45:43Z

On Mon, May 17, 2021 at 10:18:21PM +0200, Martin Kaiser wrote:
> rtw_free_network_queue iterates over the scanned wireless networks and
> calls _rtw_free_network for each of them. In some cases,
> _rtw_free_network removes a network from the list.
>
> We have to use list_for_each_entry_safe if we remove list entries while
> we iterate over a list.
>
> Fixes: 23017c8842d2 ("staging: rtl8188eu: Use list iterators and helpers")
> Signed-off-by: Martin Kaiser <martin@kaiser.cx>
> ---
> v2:
> - use list_for_each_entry_safe
>

I hadn't intended for you to redo the patch otherwise I would have
asked you to include endless loop script in all the commit messages
and rename "temp" (temperatures) to "tmp" (temporary).

You have to learn to let go, patches are never perfect. But let's say
that these v2 patch are perfect and add a:

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>

Don't resend! :)

regards,
dan carpenter