Re: [PATCH] oledb32: Correct length calculation in DataConvert for DBTYPE_STR

[Huw Davies]

On Thu, Feb 23, 2017 at 06:49:58AM +0000, Alistair Leslie-Hughes wrote:
> Signed-off-by: Alistair Leslie-Hughes
>
> dlls/oledb32/convert.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/dlls/oledb32/convert.c b/dlls/oledb32/convert.c
> index ac462f2..7030eb5 100644
> --- a/dlls/oledb32/convert.c
> +++ b/dlls/oledb32/convert.c
> @@ -816,23 +816,23 @@ static HRESULT WINAPI convert_DataConvert(IDataConvert* iface,
>      case DBTYPE_STR:
>      {
>          BSTR b;
> -
>    DBLENGTH bstr_len;
> -
>    DBLENGTH length;
>    INT bytes_to_copy;
> -
>    hr = IDataConvert_DataConvert(iface, src_type, DBTYPE_BSTR, src_len, &bstr_len,
> -
>    hr = IDataConvert_DataConvert(iface, src_type, DBTYPE_BSTR, src_len, &length,
>                                  src, &b, sizeof(BSTR), src_status, dst_status,
>                                  precision, scale, flags);
>    if(hr != S_OK) return hr;
> -
>    bstr_len = SysStringLen(b);
> -
>    *dst_len = bstr_len * sizeof(char); /* Doesn't include size for '\0' */
> -
>    length = WideCharToMultiByte(CP_ACP, 0, b, -1, NULL, 0, NULL, NULL);
> -
>    *dst_len = length - 1; /* Doesn't include size for '\0' */
>    *dst_status = DBSTATUS_S_OK;
> -
>    bytes_to_copy = min(*dst_len + sizeof(char), dst_max_len);
> -
>    bytes_to_copy = min(length, dst_max_len);
>    if(dst)
>    {
>        if(bytes_to_copy >= sizeof(char))
>        {
> -
>            WideCharToMultiByte(CP_ACP, 0, b, bytes_to_copy - sizeof(char), dst, dst_max_len, NULL, NULL);
> -
>            WideCharToMultiByte(CP_ACP, 0, b, -1, dst, dst_max_len, NULL, NULL);
>            *((char *)dst + bytes_to_copy / sizeof(char) - 1) = 0;
> -
>            if(bytes_to_copy < *dst_len + sizeof(char))
> -
>            if(bytes_to_copy < length)
>                *dst_status = DBSTATUS_S_TRUNCATED;
>        }
>        else

This fails the tests:
convert.c:1926: Test failed: got 2

I suspect you want to use SysStringLen(b) as the wstr len in
the calls to WideCharToMultiByte.

It would also be worth adding a test for DBTYPE_STR | DBTYPE_BYREF in
this case too.

Huw.