Skip to content

Fix a few leaks and an incomplete copy in CFURLComponents #939

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 10, 2017

Conversation

mark1smi
Copy link
Contributor

@mark1smi mark1smi commented Apr 7, 2017

Addresses a few leaks:

  • during deallocation of a CFURLComponentsRef, _portComponent was leaked
  • _CFURLComponentsCopyURLRelativeToURL failed to CFRelease a local
  • _CFURLComponentsCopyQueryItems failed to CFRelease a local

Incomplete copy in _CFURLComponentsCreateCopy:

  • failed to transfer _passwordComponentValid along with _passwordComponent
  • failed to transfer _parseInfo

Addresses a few leaks:
* during deallocation of a CFURLComponentsRef, _portComponent was leaked
* _CFURLComponentsCopyURLRelativeToURL failed to CFRelease a local
* _CFURLComponentsCopyQueryItems failed to CFRelease a local

Incomplete copy in _CFURLComponentsCreateCopy:
* failed to transfer _passwordComponentValid along with _passwordComponent
* failed to transfer _parseInfo
@mark1smi
Copy link
Contributor Author

mark1smi commented Apr 7, 2017

@parkera - a few more fixes to CoreFoundation you might be interested in. Not sure if any of these symptomize in iOS or MacOS like #936 did.

@parkera
Copy link
Contributor

parkera commented Apr 10, 2017

I think these may be Linux only but I'll double check. Thanks!

@parkera
Copy link
Contributor

parkera commented Apr 10, 2017

@swift-ci test and merge

@swift-ci swift-ci merged commit c08fe07 into swiftlang:master Apr 10, 2017
@mark1smi mark1smi deleted the CFURLComponents_leaks branch April 11, 2017 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants