[Webkit-unassigned] [Bug 104307] New: CURL: Incorrect stripping of query from file URL

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=104307

           Summary: CURL: Incorrect stripping of query from file URL
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Page Loading
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: menright at jetheaddev.com


CURL: Incorrect stripping of query from file URL:

A URL of 
file:///dir/foo3.html?3
Turns into 
file:///dir/foo

Before being passed to CURL.

The cause is in ResourceHandleManager.cpp ResourceHandleManager::initializeHandle. When kurl.isLocalFile() is true, the query string is obtained and then:

int queryIndex = url.find(query);
if (queryIndex != -1)
    url = url.left(queryIndex - 1);

The fix is to change the find to reverseFind.

I have seen that the CURL code is not popular, but it is still in SVN. 

I looked briefly for equivalent logic in the Soup code but I didn't find query strings being stripped, just fragment identifiers.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.