2018-04-26 11:17 GMT+02:00 Linus Walleij <linus.walleij@xxxxxxxxxx>:
> On Thu, Mar 29, 2018 at 8:29 PM, Timur Tabi <timur@xxxxxxxxxxxxxx> wrote:
>
>> If the main loop in linehandle_create() encounters an error, it
>> unwinds completely by freeing all previously requested GPIO
>> descriptors. However, if the error occurs in the beginning of
>> the loop before that GPIO is requested, then the exit code
>> attempts to free a null descriptor. If extrachecks is enabled,
>> gpiod_free() triggers a WARN_ON.
>>
>> Instead, keep a separate count of legitimate GPIOs so that only
>> those are freed.
>>
>> Cc: stable@xxxxxxxxxxxxxxx
>> Fixes: d7c51b47ac11 ("gpio: userspace ABI for reading/writing GPIO lines")
>> Signed-off-by: Timur Tabi <timur@xxxxxxxxxxxxxx>
>
> Patch applied for fixes.
>
> Bartosz, can you have a quick look at this? Did you run into the
> problem at any point?
>
I have never seen this issue, but the patch looks correct to me.
Thanks,
Bartosz