Re: [PATCH v2 0/5] MR2944: winewayland.drv: First window contents on screen

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winewayland.drv/window.c:
> -
> - TRACE("hwnd=%p\n", data->hwnd);
> -
> - return data;
>
> +}
> -
>
> +/***********************************************************************
> -
> -
>       wayland_win_data_destroy
> - */
>
> +static void wayland_win_data_destroy(struct wayland_win_data *data)
> +{
> - TRACE("hwnd=%p\n", data->hwnd);
> -
> - win_data_context[context_idx(data->hwnd)] = NULL;
> -
> - free(data);

You should probably free outside of the mutex.