fixing overrelease and out-of-bounds write, fixes #13725

[wxWidgets.git] / src / common / url.cpp

diff --git a/src/common/url.cpp b/src/common/url.cpp
index 813ac542e57777c017262a3fd7f93eb886efd326..15b9311839e476a852c5975c9cf8670264912b7f 100644 (file)
--- a/src/common/url.cpp
+++ b/src/common/url.cpp
@@ -1,414 +1,496 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        url.cpp
+// Name:        src/common/url.cpp

 // Purpose:     URL parser
 // Author:      Guilhem Lavaux
 // Modified by:
 // Created:     20/07/1997
 // RCS-ID:      $Id$
 // Copyright:   (c) 1997, 1998 Guilhem Lavaux
 // Purpose:     URL parser
 // Author:      Guilhem Lavaux
 // Modified by:
 // Created:     20/07/1997
 // RCS-ID:      $Id$
 // Copyright:   (c) 1997, 1998 Guilhem Lavaux

-// Licence:     wxWindows license
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
-#pragma implementation "url.h"
-#endif
-

 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__

-#pragma hdrstop
+    #pragma hdrstop

 #endif
 
 #endif
 

-#include <string.h>
-#include <ctype.h>
+#if wxUSE_URL

 
 

-#include <wx/string.h>
-#include <wx/list.h>
-#include <wx/utils.h>
-#include <wx/url.h>
+#include "wx/url.h"

 
 

-#if !USE_SHARED_LIBRARY
-IMPLEMENT_CLASS(wxProtoInfo, wxObject)
-IMPLEMENT_CLASS(wxURL, wxObject)
+#ifndef WX_PRECOMP
+    #include "wx/list.h"
+    #include "wx/string.h"
+    #include "wx/utils.h"
+    #include "wx/module.h"

 #endif
 
 #endif
 

+#include <string.h>
+#include <ctype.h>
+
+IMPLEMENT_CLASS(wxURL, wxURI)
+

 // Protocols list
 // Protocols list

-wxProtoInfo *wxURL::g_protocols = NULL;
+wxProtoInfo *wxURL::ms_protocols = NULL;

 
 

-#if wxUSE_SOCKETS
-wxHTTP *wxURL::g_proxy = NULL;
+// Enforce linking of protocol classes:
+USE_PROTOCOL(wxFileProto)
+
+#if wxUSE_PROTOCOL_HTTP
+USE_PROTOCOL(wxHTTP)
+
+    wxHTTP *wxURL::ms_proxyDefault = NULL;
+    bool wxURL::ms_useDefaultProxy = false;
+#endif
+
+#if wxUSE_PROTOCOL_FTP
+USE_PROTOCOL(wxFTP)

 #endif
 
 // --------------------------------------------------------------
 #endif
 
 // --------------------------------------------------------------

-// wxURL
+//
+//                          wxURL
+//

 // --------------------------------------------------------------
 
 // --------------------------------------------------------------
 // --------------------------------------------------------------
 
 // --------------------------------------------------------------

-// --------- wxURL CONSTRUCTOR DESTRUCTOR -----------------------
+// Construction

 // --------------------------------------------------------------
 
 // --------------------------------------------------------------
 

-wxURL::wxURL(const wxString& url)
+wxURL::wxURL(const wxString& url) : wxURI(url)

 {
 {

-  m_protocol = NULL;
-  m_error = wxURL_NOERR;
-  m_url = url;
-#if wxUSE_SOCKETS
-  m_useProxy = (g_proxy != NULL);
-  m_proxy = g_proxy;
-#endif
-  ParseURL();
+    Init(url);
+    ParseURL();

 }
 
 }
 

-bool wxURL::ParseURL()
+wxURL::wxURL(const wxURI& uri) : wxURI(uri)

 {
 {

-  wxString last_url = m_url;
-
- // If the URL was already parsed (so m_protocol != NULL), we pass this section.
-  if (!m_protocol) {
-
-    // Clean up
-    CleanData();
+    Init(uri.BuildURI());
+    ParseURL();
+}

 
 

-    // Extract protocol name
-    if (!PrepProto(last_url)) {
-      m_error = wxURL_SNTXERR;
-      return FALSE;
-    }
+wxURL::wxURL(const wxURL& url) : wxURI(url)
+{
+    Init(url.GetURL());
+    ParseURL();
+}

 
 

-    // Find and create the protocol object
-    if (!FetchProtocol()) {
-      m_error = wxURL_NOPROTO;
-      return FALSE;
-    }
+void wxURL::Init(const wxString& url)
+{
+    m_protocol = NULL;
+    m_error = wxURL_NOERR;
+    m_url = url;
+#if wxUSE_URL_NATIVE
+    m_nativeImp = CreateNativeImpObject();
+#endif

 
 

-    // Do we need a host name ?
-    if (m_protoinfo->m_needhost) {
-      // Extract it
-      if (!PrepHost(last_url)) {
-        m_error = wxURL_SNTXERR;
-        return FALSE;
-      }
-    }
+#if wxUSE_PROTOCOL_HTTP
+    if ( ms_useDefaultProxy && !ms_proxyDefault )
+    {
+        SetDefaultProxy( wxGetenv(wxT("HTTP_PROXY")) );

 
 

-    // Extract full path
-    if (!PrepPath(last_url)) {
-      m_error = wxURL_NOPATH;
-      return FALSE;
+        if ( !ms_proxyDefault )
+        {
+            // don't try again
+            ms_useDefaultProxy = false;
+        }

     }
     }

-  }
-  // URL parse finished.

 
 

-#if wxUSE_SOCKETS
-  if (m_useProxy) {
-    // We destroy the newly created protocol.
-    CleanData();
+    m_useProxy = ms_proxyDefault != NULL;
+    m_proxy = ms_proxyDefault;
+#endif // wxUSE_PROTOCOL_HTTP

 
 

-    // Third, we rebuild the URL.
-    m_url = m_protoname + wxT(":");
-    if (m_protoinfo->m_needhost)
-      m_url = m_url + wxT("//") + m_hostname;
+}

 
 

-    m_url += m_path;
+// --------------------------------------------------------------
+// Assignment
+// --------------------------------------------------------------

 
 

-    // We initialize specific variables.
-    m_protocol = m_proxy; // FIXME: we should clone the protocol
-  }
-#endif
+wxURL& wxURL::operator = (const wxString& url)
+{
+    wxURI::operator = (url);
+    Free();
+    Init(url);
+    ParseURL();

 
 

-  m_error = wxURL_NOERR;
-  return TRUE;
+    return *this;

 }
 
 }
 

-void wxURL::CleanData()
+wxURL& wxURL::operator = (const wxURI& uri)

 {
 {

-#if wxUSE_SOCKETS
-  if (!m_useProxy)
-#endif
-    delete m_protocol;
+    if (&uri != this)
+    {
+        wxURI::operator = (uri);
+        Free();
+        Init(uri.BuildURI());
+        ParseURL();
+    }
+
+    return *this;

 }
 
 }
 

-wxURL::~wxURL()
+wxURL& wxURL::operator = (const wxURL& url)

 {
 {

-  CleanData();
-#if wxUSE_SOCKETS
-  if (m_proxy && m_proxy != g_proxy)
-    delete m_proxy;
-#endif
+    if (&url != this)
+    {
+        wxURI::operator = (url);
+        Free();
+        Init(url.GetURL());
+        ParseURL();
+    }
+
+    return *this;

 }
 
 // --------------------------------------------------------------
 }
 
 // --------------------------------------------------------------

-// --------- wxURL urls decoders --------------------------------
+// ParseURL
+//
+// Builds the URL and takes care of the old protocol stuff

 // --------------------------------------------------------------
 
 // --------------------------------------------------------------
 

-bool wxURL::PrepProto(wxString& url)
+bool wxURL::ParseURL()

 {
 {

-  int pos;
+    // If the URL was already parsed (m_protocol != NULL), pass this section.
+    if (!m_protocol)
+    {
+        // Clean up
+        CleanData();
+
+        // Make sure we have a protocol/scheme
+        if (!HasScheme())
+        {
+            m_error = wxURL_SNTXERR;
+            return false;
+        }
+
+        // Find and create the protocol object
+        if (!FetchProtocol())
+        {
+            m_error = wxURL_NOPROTO;
+            return false;
+        }
+
+        // Do we need a host name ?
+        if (m_protoinfo->m_needhost)
+        {
+            //  Make sure we have one, then
+            if (!HasServer())
+            {
+                m_error = wxURL_SNTXERR;
+                return false;
+            }
+        }
+    }

 
 

-  // Find end
-  pos = url.Find(wxT(':'));
-  if (pos == -1)
-    return FALSE;
+#if wxUSE_PROTOCOL_HTTP
+    if (m_useProxy)
+    {
+        // Third, we rebuild the URL.
+        m_url = m_scheme + wxT(":");
+        if (m_protoinfo->m_needhost)
+            m_url = m_url + wxT("//") + m_server;
+
+        // We initialize specific variables.
+        if (m_protocol)
+            m_protocol->Destroy();
+        m_protocol = m_proxy; // FIXME: we should clone the protocol
+    }
+#endif // wxUSE_PROTOCOL_HTTP

 
 

-  m_protoname = url(0, pos);
+    m_error = wxURL_NOERR;
+    return true;
+}

 
 

-  url = url(pos+1, url.Length());
+// --------------------------------------------------------------
+// Destruction/Cleanup
+// --------------------------------------------------------------

 
 

-  return TRUE;
+void wxURL::CleanData()
+{
+#if wxUSE_PROTOCOL_HTTP
+    if (!m_useProxy)
+#endif // wxUSE_PROTOCOL_HTTP
+    {
+        if (m_protocol)
+        {
+            // Need to safely delete the socket (pending events)
+            m_protocol->Destroy();
+            m_protocol = NULL;
+        }
+    }

 }
 
 }
 

-bool wxURL::PrepHost(wxString& url)
+void wxURL::Free()

 {
 {

-  wxString temp_url;
-  int pos, pos2;
-
-  if ((url.GetChar(0) != '/') || (url.GetChar(1) != '/'))
-    return FALSE;
-
-  url = url(2, url.Length());
-
-  pos = url.Find(wxT('/'));
-  if (pos == -1)
-    pos = url.Length();
-
-  if (pos == 0)
-    return FALSE;
-
-  temp_url = url(0, pos);
-  url = url(url.Find(wxT('/')), url.Length());
-
-  // Retrieve service number
-  pos2 = temp_url.Find(wxT(':'), TRUE);
-  if (pos2 != -1 && pos2 < pos) {
-    m_servname = temp_url(pos2+1, pos);
-    if (!m_servname.IsNumber())
-      return FALSE;
-    temp_url = temp_url(0, pos2);
-  }
-
-  // Retrieve user and password.
-  pos2 = temp_url.Find(wxT('@'));
-  // Even if pos2 equals -1, this code is right.
-  m_hostname = temp_url(pos2+1, temp_url.Length());
-
-  m_user = wxT("");
-  m_password = wxT("");
-
-  if (pos2 == -1)
-    return TRUE;
-
-  temp_url = temp_url(0, pos2);
-  pos2 = temp_url.Find(wxT(':'));
-
-  if (pos2 == -1)
-    return FALSE;
-
-  m_user = temp_url(0, pos2);
-  m_password = temp_url(pos2+1, url.Length());
-
-  return TRUE;
+    CleanData();
+#if wxUSE_PROTOCOL_HTTP
+    if (m_proxy && m_proxy != ms_proxyDefault)
+        delete m_proxy;
+#endif // wxUSE_PROTOCOL_HTTP
+#if wxUSE_URL_NATIVE
+    delete m_nativeImp;
+#endif

 }
 
 }
 

-bool wxURL::PrepPath(wxString& url)
+wxURL::~wxURL()

 {
 {

-  if (url.Length() != 0)
-    m_path = ConvertToValidURI(url);
-  else
-    m_path = wxT("/");
-  return TRUE;
+    Free();

 }
 
 }
 

+// --------------------------------------------------------------
+// FetchProtocol
+// --------------------------------------------------------------
+

 bool wxURL::FetchProtocol()
 {
 bool wxURL::FetchProtocol()
 {

-  wxProtoInfo *info = g_protocols;
+    wxProtoInfo *info = ms_protocols;

 
 

-  while (info) {
-    if (m_protoname == info->m_protoname) {
-      if (m_servname.IsNull())
-        m_servname = info->m_servname;
-
-      m_protoinfo = info;
-      m_protocol = (wxProtocol *)m_protoinfo->m_cinfo->CreateObject();
-      return TRUE;
+    while (info)
+    {
+        if (m_scheme == info->m_protoname)
+        {
+            if ( m_port.empty() )
+                m_port = info->m_servname;
+            m_protoinfo = info;
+            m_protocol = (wxProtocol *)m_protoinfo->m_cinfo->CreateObject();
+            return true;
+        }
+        info = info->next;

     }
     }

-    info = info->next;
-  }
-  return FALSE;
+    return false;

 }
 
 // --------------------------------------------------------------
 }
 
 // --------------------------------------------------------------

-// --------- wxURL get ------------------------------------------
+// GetInputStream

 // --------------------------------------------------------------
 
 // --------------------------------------------------------------
 

-wxInputStream *wxURL::GetInputStream(void)
+wxInputStream *wxURL::GetInputStream()

 {
 {

-  wxInputStream *the_i_stream = NULL;
+    if (!m_protocol)
+    {
+        m_error = wxURL_NOPROTO;
+        return NULL;
+    }

 
 

-  if (!m_protocol) {
-    m_error = wxURL_NOPROTO;
-    return NULL;
-  }
+    m_error = wxURL_NOERR;
+    if (HasUserInfo())
+    {
+        size_t dwPasswordPos = m_userinfo.find(':');
+
+        if (dwPasswordPos == wxString::npos)
+            m_protocol->SetUser(Unescape(m_userinfo));
+        else
+        {
+            m_protocol->SetUser(Unescape(m_userinfo(0, dwPasswordPos)));
+            m_protocol->SetPassword(Unescape(m_userinfo(dwPasswordPos+1, m_userinfo.length() + 1)));
+        }
+    }

 
 

-  m_error = wxURL_NOERR;
-  if (m_user != wxT("")) {
-    m_protocol->SetUser(m_user);
-    m_protocol->SetPassword(m_password);
-  }
+#if wxUSE_URL_NATIVE
+    // give the native implementation to return a better stream
+    // such as the native WinINet functionality under MS-Windows
+    if (m_nativeImp)
+    {
+        wxInputStream *rc;
+        rc = m_nativeImp->GetInputStream(this);
+        if (rc != 0)
+            return rc;
+    }
+    // else use the standard behaviour
+#endif // wxUSE_URL_NATIVE

 
 #if wxUSE_SOCKETS
     wxIPV4address addr;
 
 
 #if wxUSE_SOCKETS
     wxIPV4address addr;
 

-  // m_protoinfo is NULL when we use a proxy
-  if (!m_useProxy && m_protoinfo->m_needhost) {
-    if (!addr.Hostname(m_hostname)) {
-      m_error = wxURL_NOHOST;
-      return NULL;
+    // m_protoinfo is NULL when we use a proxy
+    if (
+#if wxUSE_PROTOCOL_HTTP
+         !m_useProxy &&
+#endif // wxUSE_PROTOCOL_HTTP
+         m_protoinfo->m_needhost )
+    {
+        if (!addr.Hostname(m_server))
+        {
+            m_error = wxURL_NOHOST;
+            return NULL;
+        }
+
+        addr.Service(m_port);
+
+        if (!m_protocol->Connect(addr, true)) // Watcom needs the 2nd arg for some reason
+        {
+            m_error = wxURL_CONNERR;
+            return NULL;
+        }

     }
     }

+#endif // wxUSE_SOCKETS

 
 

-    addr.Service(m_servname);
+    wxString fullPath;

 
 

-    if (!m_protocol->Connect(addr, TRUE)) // Watcom needs the 2nd arg for some reason
-    {
-      m_error = wxURL_CONNERR;
-      return NULL;
-    }
-  }
-#endif
+#if wxUSE_PROTOCOL_HTTP
+    // When we use a proxy, we have to pass the whole URL to it.
+    if (m_useProxy)
+        fullPath += m_url;
+#endif // wxUSE_PROTOCOL_HTTP
+
+    if(m_path.empty())
+        fullPath += wxT("/");
+    else
+        fullPath += m_path;

 
 

-  // When we use a proxy, we have to pass the whole URL to it.
-  if (m_useProxy)
-    the_i_stream = m_protocol->GetInputStream(m_url);
-  else
-    the_i_stream = m_protocol->GetInputStream(m_path);
+    if (HasQuery())
+        fullPath += wxT("?") + m_query;

 
 

-  if (!the_i_stream) {
-    m_error = wxURL_PROTOERR;
-    return NULL;
-  }
+    if (HasFragment())
+        fullPath += wxT("#") + m_fragment;

 
 

-  return the_i_stream;
+    wxInputStream *the_i_stream = m_protocol->GetInputStream(fullPath);
+
+    if (!the_i_stream)
+    {
+        m_error = wxURL_PROTOERR;
+        return NULL;
+    }
+
+    return the_i_stream;

 }
 
 }
 

-#if wxUSE_SOCKETS
+#if wxUSE_PROTOCOL_HTTP

 void wxURL::SetDefaultProxy(const wxString& url_proxy)
 {
 void wxURL::SetDefaultProxy(const wxString& url_proxy)
 {

-  if (url_proxy.IsNull()) {
-    g_proxy->Close();
-    delete g_proxy;
-    g_proxy = NULL;
-    return;
-  }
-
-  wxString tmp_str = url_proxy;
-  int pos = tmp_str.Find(wxT(':'));
-  if (pos == -1)
-    return;
-
-  wxString hostname = tmp_str(0, pos),
-           port = tmp_str(pos+1, tmp_str.Length()-pos);
-  wxIPV4address addr;
-
-  if (!addr.Hostname(hostname))
-    return;
-  if (!addr.Service(port))
-    return;
-
-  if (g_proxy)
-  // Finally, when all is right, we connect the new proxy.
-    g_proxy->Close();
-  else
-    g_proxy = new wxHTTP();
-  g_proxy->Connect(addr, TRUE); // Watcom needs the 2nd arg for some reason
+    if ( !url_proxy )
+    {
+        if ( ms_proxyDefault )
+        {
+            ms_proxyDefault->Close();
+            wxDELETE(ms_proxyDefault);
+        }
+    }
+    else
+    {
+        wxString tmp_str = url_proxy;
+        int pos = tmp_str.Find(wxT(':'));
+        if (pos == wxNOT_FOUND)
+            return;
+
+        wxString hostname = tmp_str(0, pos),
+        port = tmp_str(pos+1, tmp_str.length()-pos);
+        wxIPV4address addr;
+
+        if (!addr.Hostname(hostname))
+            return;
+        if (!addr.Service(port))
+            return;
+
+        if (ms_proxyDefault)
+            // Finally, when all is right, we connect the new proxy.
+            ms_proxyDefault->Close();
+        else
+            ms_proxyDefault = new wxHTTP();
+        ms_proxyDefault->Connect(addr, true); // Watcom needs the 2nd arg for some reason
+    }

 }
 
 void wxURL::SetProxy(const wxString& url_proxy)
 {
 }
 
 void wxURL::SetProxy(const wxString& url_proxy)
 {

-  if (url_proxy.IsNull()) {
-    if (m_proxy) {
-      m_proxy->Close();
-      delete m_proxy;
+    if ( !url_proxy )
+    {
+        if ( m_proxy && m_proxy != ms_proxyDefault )
+        {
+            m_proxy->Close();
+            delete m_proxy;
+        }
+
+        m_useProxy = false;
+    }
+    else
+    {
+        wxString tmp_str;
+        wxString hostname, port;
+        int pos;
+        wxIPV4address addr;
+
+        tmp_str = url_proxy;
+        pos = tmp_str.Find(wxT(':'));
+        // This is an invalid proxy name.
+        if (pos == wxNOT_FOUND)
+            return;
+
+        hostname = tmp_str(0, pos);
+        port = tmp_str(pos+1, tmp_str.length()-pos);
+
+        addr.Hostname(hostname);
+        addr.Service(port);
+
+        // Finally, create the whole stuff.
+        if (m_proxy && m_proxy != ms_proxyDefault)
+            delete m_proxy;
+        m_proxy = new wxHTTP();
+        m_proxy->Connect(addr, true); // Watcom needs the 2nd arg for some reason
+
+        CleanData();
+        // Reparse url.
+        m_useProxy = true;
+        ParseURL();

     }
     }

-    m_useProxy = FALSE;
-    return;
-  }
-
-  wxString tmp_str;
-  wxString hostname, port;
-  int pos;
-  wxIPV4address addr;
-
-  tmp_str = url_proxy;
-  pos = tmp_str.Find(wxT(':'));
-  // This is an invalid proxy name.
-  if (pos == -1)
-    return;
-
-  hostname = tmp_str(0, pos);
-  port = tmp_str(pos, tmp_str.Length()-pos);
-
-  addr.Hostname(hostname);
-  addr.Service(port);
-
-  // Finally, create the whole stuff.
-  if (m_proxy && m_proxy != g_proxy)
-    delete m_proxy;
-  m_proxy = new wxHTTP();
-  m_proxy->Connect(addr, TRUE); // Watcom needs the 2nd arg for some reason
-
-  CleanData();
-  // Reparse url.
-  m_useProxy = TRUE;
-  ParseURL();

 }
 }

-#endif
+#endif // wxUSE_PROTOCOL_HTTP
+
+// ----------------------------------------------------------------------
+// wxURLModule
+//
+// A module which deletes the default proxy if we created it
+// ----------------------------------------------------------------------

 
 

-wxString wxURL::ConvertToValidURI(const wxString& uri)
+#if wxUSE_SOCKETS
+
+class wxURLModule : public wxModule

 {
 {

-  wxString out_str;
-  wxString hexa_code;
-  size_t i;
-
-  for (i=0;i<uri.Len();i++) {
-    wxChar c = uri.GetChar(i);
-
-    if (c == wxT(' '))
-      out_str += wxT('+');
-    else {
-      if (!isalpha(c) && c != wxT('.') && c != wxT('+') && c != wxT('/')) {
-        hexa_code.Printf(wxT("%%%02X"), c);
-        out_str += hexa_code;
-      } else
-        out_str += c;
-    }
-  }
+public:
+    wxURLModule();
+
+    virtual bool OnInit();
+    virtual void OnExit();

 
 

-  return out_str;
+private:
+    DECLARE_DYNAMIC_CLASS(wxURLModule)
+};
+
+IMPLEMENT_DYNAMIC_CLASS(wxURLModule, wxModule)
+
+wxURLModule::wxURLModule()
+{
+    // we must be cleaned up before wxSocketModule as otherwise deleting
+    // ms_proxyDefault from our OnExit() won't work (and can actually crash)
+    AddDependency(wxClassInfo::FindClass(wxT("wxSocketModule")));

 }
 
 }
 

-wxString wxURL::ConvertFromURI(const wxString& uri)
+bool wxURLModule::OnInit()

 {
 {

-  wxString new_uri;
-
-  size_t i = 0;
-  while (i<uri.Len()) {
-    int code;
-    if (uri[i] == wxT('%')) {
-      i++;
-      if (uri[i] >= wxT('A') && uri[i] <= wxT('F'))
-        code = (uri[i] - wxT('A') + 10) * 16;
-      else
-        code = (uri[i] - wxT('0')) * 16;
-      i++;
-      if (uri[i] >= wxT('A') && uri[i] <= wxT('F'))
-        code += (uri[i] - wxT('A')) + 10;
-      else
-        code += (uri[i] - wxT('0'));
-      i++;
-      new_uri += (wxChar)code;
-      continue;
+#if wxUSE_PROTOCOL_HTTP
+    // env var HTTP_PROXY contains the address of the default proxy to use if
+    // set, but don't try to create this proxy right now because it will slow
+    // down the program startup (especially if there is no DNS server
+    // available, in which case it may take up to 1 minute)
+
+    if ( wxGetenv(wxT("HTTP_PROXY")) )
+    {
+        wxURL::ms_useDefaultProxy = true;

     }
     }

-    new_uri += uri[i];
-    i++;
-  }
-  return new_uri;
+#endif // wxUSE_PROTOCOL_HTTP
+    return true;
+}
+
+void wxURLModule::OnExit()
+{
+#if wxUSE_PROTOCOL_HTTP
+    wxDELETE(wxURL::ms_proxyDefault);
+#endif // wxUSE_PROTOCOL_HTTP

 }
 }

+
+#endif // wxUSE_SOCKETS
+
+
+#endif // wxUSE_URL