[PATCH v2 1/2] urlmon: Use unescaped Urls for FTP actions
Jacek Caban
jacek at codeweavers.com
Mon Mar 25 10:46:06 CDT 2019
Hi Alistair,
On 3/23/19 10:34 AM, Alistair Leslie-Hughes wrote:
> From: André Hentschel <nerv at dawncrow.de>
>
> Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=26445
> Signed-off-by: Alistair Leslie-Hughes <leslie_alistair at hotmail.com>
> ---
> dlls/urlmon/ftp.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/dlls/urlmon/ftp.c b/dlls/urlmon/ftp.c
> index aef4a6cf16..31bdc79852 100644
> --- a/dlls/urlmon/ftp.c
> +++ b/dlls/urlmon/ftp.c
> @@ -66,17 +66,24 @@ static HRESULT FtpProtocol_open_request(Protocol *prot, IUri *uri, DWORD request
> HINTERNET internet_session, IInternetBindInfo *bind_info)
> {
> FtpProtocol *This = impl_from_Protocol(prot);
> + DWORD path_size = 0;
> + LPWSTR path;
> BSTR url;
> HRESULT hres;
>
> hres = IUri_GetAbsoluteUri(uri, &url);
> if(FAILED(hres))
> return hres;
> + path = heap_strdupW((LPWSTR)url);
Why do you need that? You could just unescape in place url, there is no
need for a new copy.
Jacek
More information about the wine-devel
mailing list