Re: [PATCH 0/1] MR10931: winebth: Fix a memory leak in bluez_gatt_characteristic_read.

[Vibhav Pant (＠vibhavp)]

Vibhav Pant (@vibhavp) commented about dlls/winebth.sys/dbus.c:
> IRP *irp )
>  {
>      DBusMessageIter args_iter, dict_iter = DBUS_MESSAGE_ITER_INIT_CLOSED;
> -    struct bluez_async_req_data *data = NULL;

I think it would be much simpler to let the old code stay, and just add `free(data)` under `failed`. `free(NULL)` is guaranteed to do nothing.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10931#note_141721