Re: wshom: Basic support for REG_SZ values in RegRead()

[Jacek Caban]

Hi Nikolay,

On 12/03/12 12:29, Nikolay Sivov wrote:
> - /* skip '' after root name */
> - if (name[len] == '\') len++;

It looks like if name[len] != '\', then name should be considered invalid.
> -
> - val = strlenW(name);
> - while (name[--val] != '\' && val)
> -
>    ;
> - keyname = SysAllocStringLen(&name[len], val-len);

This won't work if name has no backslashes.
> - /* skip leading '' */
> - val++;
> -
> - type = REG_NONE;
> - datalen = 0;
> - ret = RegGetValueW(root, keyname, &name[val], RRF_RT_ANY, &type, NULL, &datalen);
> - if (ret == ERROR_SUCCESS)
> - {
> -
>    if (type == REG_SZ)
> -
>    {
> -
>        V_VT(value) = VT_BSTR;
> -
>        V_BSTR(value) = SysAllocStringLen(NULL, datalen-1);
> -
>        ret = RegGetValueW(root, keyname, &name[val], RRF_RT_ANY, &type, V_BSTR(value), &datalen);
> -
>    }
> -
>    else
> -
>        FIXME("type %d not handled\n", type);

This should return an error.

Jacek