Bug 157059 – [curl] ASSERTION FAILED: isLocalFile() || url == m_string in fast/dom/34176.html

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Fujii Hironori]

[curl] ASSERTION FAILED: isLocalFile() || url == m_string in fast/dom/34176.html

trunk@200119, WinCairo port, Debug build

> perl Tools\Scripts\build-webkit --release --wincairo --64-bit

Layout Test:

  fast/dom/34176.html

ASSERTION FAILED: isLocalFile() || url == m_string
...\Source\WebCore\platform\URL.cpp(438) : WebCore::URL::URL

The value of url is "HTTP://d:/tmp/p1.png/", but m_string is "http://d/tmp/p1.png/re".

> WTF.dll!WTFCrash() Line 322	C++
> WebKit.dll!WebCore::URL::URL(WebCore::ParsedURLStringTag __formal, const WTF::String & url) Line 438	C++
> WebKit.dll!WebCore::headerCallback(char * ptr, unsigned __int64 size, unsigned __int64 nmemb, void * data) Line 501	C++
> libcurl.dll!Curl_client_chop_write(connectdata * conn, int type, char * ptr, unsigned __int64 len) Line 456	C
> libcurl.dll!Curl_http_readwrite_headers(SessionHandle * data, connectdata * conn, __int64 * nread, bool * stop_reading) Line 3113	C
> libcurl.dll!readwrite_data(SessionHandle * data, connectdata * conn, SingleRequest * k, int * didwhat, bool * done) Line 481	C
> libcurl.dll!Curl_readwrite(connectdata * conn, SessionHandle * data, bool * done) Line 1064	C
> libcurl.dll!multi_runsingle(Curl_multi * multi, timeval now, SessionHandle * data) Line 1535	C
> libcurl.dll!curl_multi_perform(void * multi_handle, int * running_handles) Line 1811	C
> WebKit.dll!WebCore::ResourceHandleManager::downloadTimerCallback() Line 656	C++
> [External Code]	
> WebKit.dll!WebCore::Timer::fired() Line 142	C++
> WebKit.dll!WebCore::ThreadTimers::sharedTimerFiredInternal() Line 124	C++
> WebKit.dll!WebCore::ThreadTimers::setSharedTimer::__l8::<lambda>() Line 73	C++
> [External Code]	
> WebKit.dll!WebCore::MainThreadSharedTimer::fired() Line 53	C++
> WebKit.dll!WebCore::TimerWindowWndProc(HWND__ * hWnd, unsigned int message, unsigned __int64 wParam, __int64 lParam) Line 91	C++
> [External Code]	
> DumpRenderTreeLib.dll!runTest(const std::basic_string<char,std::char_traits<char>,std::allocator<char> > & inputLine) Line 1145	C++
> DumpRenderTreeLib.dll!main(int argc, const char * * argv) Line 1486	C++
> DumpRenderTreeLib.dll!dllLauncherEntryPoint(int argc, const char * * argv) Line 1517	C++
> DumpRenderTree.exe!main(int argc, const char * * argv) Line 260	C++
> [External Code]

[Fujii Hironori]

This issue is observed at Google Maps (Japan), too.

1) Launch MiniBroser
2) Go to http://maps.google.co.jp/maps?hl=ja&tab=wl
3) ASSERTION FAILED

url is "https://www.google.co.jp:443/maps/@?hl=ja"
m_string "https://www.google.co.jp/maps/@?hl=ja"

[Fujii Hironori]

Created attachment 278577 [details]
Patch

[Darin Adler]

Comment on attachment 278577 [details]
Patch

This patch is OK. It would also be OK to just replaced ParsedURLString with URL() and not make any other changes.

I normally don’t review [curl] patches, so someone else should do that, but the change looks fine to me.

[WebKit Commit Bot]

Comment on attachment 278577 [details]
Patch

Clearing flags on attachment: 278577

Committed r200776: <http://trac.webkit.org/changeset/200776>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.