Re: [PATCH 0/2] MR541: winedbg: Prefer thread names from GetThreadDescription().

[Rémi Bernon (＠rbernon)]

R��mi Bernon (@rbernon) commented about programs/winedbg/gdbproxy.c:
>  reply_buffer_append(reply, str, strlen(str));
>
> }
>
> +static inline void reply_buffer_append_wstr(struct reply_buffer* reply, const WCHAR* wstr)
> +{
> - char* str;
> - int len;
> -
> - len = WideCharToMultiByte(CP_ACP, 0, wstr, -1, NULL, 0, NULL, NULL);
> - str = malloc(len);
> - if (WideCharToMultiByte(CP_ACP, 0, wstr, -1, str, len, NULL, NULL))
> -
>    reply_buffer_append_str(reply, str);

This could maybe handle allocation failure more gracefully, skipping the append?