Re: [PATCH 0/7] MR5992: windows.web: Implement IJsonValueStatics::CreateStringValue().

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.web/json_value.c:
> static HRESULT WINAPI json_value_statics_CreateStringValue( IJsonValueStatics *iface, HSTRING input, IJsonValue **value )
>  {
> - FIXME( "iface %p, input %s, value %p stub!\n", iface, debugstr_hstring( input ), value );
> - return E_NOTIMPL;
> - struct json_value *impl;
> -
> - TRACE( "iface %p, input %s, value %p\n", iface, debugstr_hstring( input ), value );
> -
> - if (!value) return E_POINTER;
> - if (!(impl = calloc( 1, sizeof(*impl) ))) return E_OUTOFMEMORY;
> -
> - impl->IJsonValue_iface.lpVtbl = &json_value_vtbl;
> - impl->ref = 1;
> - WindowsDuplicateString( input, &impl->string_value );

```suggestion:-0+0
    if (FAILED(hr = WindowsDuplicateString( input, &impl->string_value )))
    {
         free( impl );
         return hr;
    }
```