Re: [PATCH v8 0/8] MR391: scrrun: Implement MoveFolder().

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/scrrun/filesystem.c:
> static HRESULT WINAPI filesys_MoveFolder(IFileSystem3 *iface, BSTR source, BSTR destination)
>  {
> - int len;
> - WCHAR src_path[MAX_PATH];
> -
>
> int len, dst_len;
> -
>
> WCHAR src_path[MAX_PATH], dst_path[MAX_PATH];
> -
>
> WCHAR *filename;
>
> TRACE("%p %s %s\n", iface, debugstr_w(source), debugstr_w(destination));
>
> if(!source || !source[0] || !destination || !destination[0])
>     return E_INVALIDARG;
> -
>
> if (!GetFullPathNameW(source, MAX_PATH, src_path, &filename))

Looking at Wine's code, I think this will set filename to NULL if source ends in a path separator.