[Webkit-unassigned] [Bug 70229] New: ResourceHandleManager::startJob() for Data URLs is missing deref()

[bugzilla-daemon at webkit.org]

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

           Summary: ResourceHandleManager::startJob() for Data URLs is
                    missing deref()
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: UNCONFIRMED
          Severity: Minor
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: jens.rosenboom at web.de


Hello All,

IMHO a deref() is missing for asynchronous jobs with data:// URLs.

In function ResourceHandleManager.cpp:
- function add() calls ref() on a new job and adds it to a list
- function startScheduledJobs() takes it from that list and calls startJob()
- function startJob() calls handleDataUrl() (for data:// URLs)

Shouldn't startJob() call deref() after handleDataUrl() returns?

Best regards,
Jens

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