Re: [PATCH v2 0/2] iio: buffer: allocation and freeing buffers fix and optimization

From: Jonathan Cameron
Date: Sun Oct 17 2021 - 10:22:21 EST


On Wed, 13 Oct 2021 12:49:21 +0300
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:

> Yang submitted a fix, but I think the code can be refactored a bit to be more
> robust against similar mistakes in the future, if any.
>
> In v2:
> - put SoB Yang's patch (it's good for backporting)
> - added refactoring patch on top of Yang's fix
>
> Andy Shevchenko (1):
> iio: buffer: Use dedicated variable in
> iio_buffers_alloc_sysfs_and_mask()
>
> Yang Yingliang (1):
> iio: buffer: Fix double-free in iio_buffers_alloc_sysfs_and_mask()
>
> drivers/iio/industrialio-buffer.c | 18 +++++++-----------
> 1 file changed, 7 insertions(+), 11 deletions(-)
>
1st patch applied to the fixes-togreg branch of iio.git. I may well end up
sending these in the merge window anyway in which case I'll probably stick patch 2
on top of it before sending. If not I'll pick that up next cycle now.

Thanks,

Jonathan