diff options
author | Morten Johan Sørvig <morten.sorvig@digia.com> | 2015-01-13 10:52:39 +0100 |
---|---|---|
committer | Jani Heikkinen <jani.heikkinen@theqtcompany.com> | 2015-01-19 08:20:24 +0100 |
commit | 49fd3511f5bcdd61c92894636fcf0b2be23d1264 (patch) | |
tree | f527f8938f294f7a73b4e2699f3d974640e053b3 /src/corelib | |
parent | 128cab4635c3ea42af5d3b9952f7b7e1d89284e7 (diff) |
Fix memory leak in Qurl::toCFURL()
Release the temp CFString.
Change-Id: I8a5b8f18a42a4a9b2c6671f0f5b32a3f0b14238d
Task-number: QTBUG-43710
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Diffstat (limited to 'src/corelib')
-rw-r--r-- | src/corelib/io/qurl_mac.mm | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/corelib/io/qurl_mac.mm b/src/corelib/io/qurl_mac.mm index c235365ad8..4468bf0f2b 100644 --- a/src/corelib/io/qurl_mac.mm +++ b/src/corelib/io/qurl_mac.mm @@ -54,7 +54,13 @@ QUrl QUrl::fromCFURL(CFURLRef url) CFURLRef QUrl::toCFURL() const { - return CFURLCreateWithString(0, toString(FullyEncoded).toCFString(), 0); + CFURLRef url = 0; + CFStringRef str = toString(FullyEncoded).toCFString(); + if (str) { + url = CFURLCreateWithString(0, str, 0); + CFRelease(str); + } + return url; } QUrl QUrl::fromNSURL(const NSURL *url) |