Bug 175103 - NetworkRTCProvider::createResolver() leaks CFHost objects
Summary: NetworkRTCProvider::createResolver() leaks CFHost objects
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Andreas Kling
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-08-02 17:42 PDT by Andreas Kling
Modified: 2017-08-02 20:53 PDT (History)
6 users (show)

See Also:


Attachments
Patch (2.22 KB, patch)
2017-08-02 18:21 PDT, Andreas Kling
youennf: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff
Patch for landing (2.23 KB, patch)
2017-08-02 18:36 PDT, Andreas Kling
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Kling 2017-08-02 17:42:05 PDT
Looks like we're missing an adoptCF() here eh!

<rdar://problem/33464999>
Comment 1 Andreas Kling 2017-08-02 17:43:31 PDT
Actually, let me make a separate radar for this. Importer, take the wheel!
Comment 2 Radar WebKit Bug Importer 2017-08-02 17:43:46 PDT
<rdar://problem/33690347>
Comment 3 Andreas Kling 2017-08-02 18:21:22 PDT
Created attachment 317055 [details]
Patch
Comment 4 WebKit Commit Bot 2017-08-02 18:33:10 PDT
Comment on attachment 317055 [details]
Patch

Rejecting attachment 317055 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'validate-changelog', '--check-oops', '--non-interactive', 317055, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in Source/WebKit/ChangeLog contains OOPS!.

Full output: http://webkit-queues.webkit.org/results/4243265
Comment 5 Andreas Kling 2017-08-02 18:36:18 PDT
Created attachment 317059 [details]
Patch for landing
Comment 6 WebKit Commit Bot 2017-08-02 20:53:08 PDT
Comment on attachment 317059 [details]
Patch for landing

Clearing flags on attachment: 317059

Committed r220178: <http://trac.webkit.org/changeset/220178>
Comment 7 WebKit Commit Bot 2017-08-02 20:53:10 PDT
All reviewed patches have been landed.  Closing bug.