Fix memory leak

ri_t is a complex structure.
It should be freed by call iri_destroy.
Cast it to a scopedPtr is forbiden because that call a delete function not iri_destroy

diff --git a/packaging/wrt.changes b/packaging/wrt.changes
index 5ec5709..ad9db4a 100644 (file)
--- a/packaging/wrt.changes
+++ b/packaging/wrt.changes
@@ -1,3 +1,6 @@
+* Tue Jul 09 2013 Baptiste DURAND <baptiste.durand@eurogiciel.fr> accepted/tizen/20130708.234009@378666e
+-  Fix memory leak
+
 * Mon Jul 08 2013 Rusty Lynch <rusty.lynch@intel.com> submit/tizen/20130627.212436@1351791
 - Refactor spec handling of decrypt support
 - Return an error when we try to load an encrypted ressource if DECRYPT build Option is disabled.

diff --git a/src/view/webkit/injected-bundle/injected_bundle_uri_handling.cpp b/src/view/webkit/injected-bundle/injected_bundle_uri_handling.cpp
index 9d41b04..7fe9e65 100644 (file)
--- a/src/view/webkit/injected-bundle/injected_bundle_uri_handling.cpp
+++ b/src/view/webkit/injected-bundle/injected_bundle_uri_handling.cpp
@@ -117,12 +117,18 @@ bool checkWhitelist(const char *url)
         return true;
     }
 
-    DPL::ScopedPtr<iri_t> iri(iri_parse(url));
+    iri_t* iri = iri_parse(url);
+    if (!iri)
+        return false;
+
     if (!iri->scheme || !iri->host || strlen(iri->host) == 0) {
+        iri_destroy(iri);
         return true;
     }
+
     std::string scheme = iri->scheme;
     std::string host = iri->host;
+    iri_destroy(iri);
 
     if (scheme.find(SCHEME_TYPE_HTTP) == std::string::npos) {
         LogDebug("url doesn't need to check white list");