Version set to 1.2-beta2

README.md

@@ -35,6 +35,9 @@ Suite of EAP supplicants for Microsoft Windows - IEEE 802.1X plug-ins for enterp
 - [CredWrite utility](https://github.com/Amebis/GEANTLink/tree/master/CredWrite) for automated user credential import to Credential Manager
 - [WLANManager utility](https://github.com/Amebis/GEANTLink/tree/master/WLANManager) to allow network profile configuration dialog shortcuts
 
+## Download
+Binaries are available for download [here](https://github.com/Amebis/GEANTLink/releases).
+
 ## Building
 
 ### Building Environment Requirements
@@ -117,3 +120,6 @@ Command            | Explanation
 `nmake SetupDebug` | Builds a debug version of project and debug MSI setup files. The resulting files can be found in `output\Setup` folder.
 
 The `/ls` flag can be appended to the commands above to reduce NMAKE’s verbosity. You can combine multiple targets (i.e. nmake Unregister Clean). Please, see NMAKE reference for further reading.
+
+### Translating into your language
+GÉANTLink is fully localizable. We kindly invite you to help [translating it on Transifex](https://www.transifex.com/eduroam_devel/geantlink/).

WLANManager/Main.cpp

@@ -90,7 +90,6 @@ static int WLANManager()
         }
 
         // Launch WLAN profile config dialog.
-        // Note: When a debugger is attached to this process the WlanUIEditProfile() will raise an exception and fail.
         WLAN_REASON_CODE wlrc;
         DWORD dwResult = WlanUIEditProfile(WLAN_UI_API_VERSION, pwcArglist[2], &(interfaces->InterfaceInfo[i].InterfaceGuid), NULL, WLSecurityPage, NULL, &wlrc);
         if (dwResult != ERROR_SUCCESS) {
@@ -119,6 +118,13 @@ int CALLBACK WinMain(_In_ HINSTANCE hInstance, _In_ HINSTANCE hPrevInstance, _In
     UNREFERENCED_PARAMETER(lpCmdLine);
     UNREFERENCED_PARAMETER(nCmdShow);
 
+    int res = 0;
+
+    {
+        // Note: When a debugger is attached to this process, the WlanUIEditProfile() will raise an exception and fail.
+        // It was accidentially discovered, that COM initialization resolves this issue.
+        com_initializer com_init(NULL);
+
         {
             // Initialize Windows XP visual styles
             INITCOMMONCONTROLSEX icc;
@@ -129,7 +135,8 @@ int CALLBACK WinMain(_In_ HINSTANCE hInstance, _In_ HINSTANCE hPrevInstance, _In
 
         pfnWlanReasonCodeToString = WlanReasonCodeToString;
 
-    int res = WLANManager();
+        res = WLANManager();
+    }
 
     assert(!_CrtDumpMemoryLeaks());
     return res;

include/Version.h

@@ -29,7 +29,7 @@
 // Product version as a single DWORD
 // Note: Used for version comparison within C/C++ code.
 //
-#define PRODUCT_VERSION          0x01020200
+#define PRODUCT_VERSION          0x01020300
 
 //
 // Product version by components
@@ -39,26 +39,26 @@
 //
 #define PRODUCT_VERSION_MAJ      1
 #define PRODUCT_VERSION_MIN      2
-#define PRODUCT_VERSION_REV      2
+#define PRODUCT_VERSION_REV      3
 #define PRODUCT_VERSION_BUILD    0
 
 //
 // Human readable product version and build year for UI
 //
-#define PRODUCT_VERSION_STR      "1.2-beta1"
+#define PRODUCT_VERSION_STR      "1.2-beta2"
 #define PRODUCT_BUILD_YEAR_STR   "2016"
 
 //
 // Numerical version presentation for ProductVersion propery in
 // MSI packages (syntax: N.N[.N[.N]])
 //
-#define PRODUCT_VERSION_INST     "1.2.2"
+#define PRODUCT_VERSION_INST     "1.2.3"
 
 //
 // The product code for ProductCode property in MSI packages
 // Replace with new on every version change, regardless how minor it is.
 //
-#define PRODUCT_VERSION_GUID     "{FF5DD25D-917C-4B33-81D3-C75B5E1C8CA8}"
+#define PRODUCT_VERSION_GUID     "{4C8DDB7D-3297-4B6A-ACF3-19702F85DE49}"
 
 //
 // Product vendor

lib/EAPBase_UI/include/EAP_UI.h

@@ -360,6 +360,14 @@ public:
     ///
     /// Constructs a credential dialog
     ///
+    /// \param[in] prov    Provider configuration data
+    /// \param[in] parent  Parent window
+    /// \param[in] id      An identifier for the dialog. A value of -1 is taken to mean a default.
+    /// \param[in] title   The title of the dialog
+    /// \param[in] pos     The dialog position. The value \c wxDefaultPosition indicates a default position, chosen by either the windowing system or wxWidgets, depending on platform.
+    /// \param[in] size    The dialog size. The value \c wxDefaultSize indicates a default size, chosen by either the windowing system or wxWidgets, depending on platform.
+    /// \param[in] style   The window style
+    ///
     wxEAPCredentialsDialog(const eap::config_provider &prov, wxWindow *parent, wxWindowID id = wxID_ANY, const wxString &title = _("EAP Credentials"), const wxPoint &pos = wxDefaultPosition, const wxSize &size = wxDefaultSize, long style = wxDEFAULT_DIALOG_STYLE);
 };
 

lib/GTC/src/Config.cpp

@@ -108,10 +108,10 @@ void eap::config_method_eapgtc::load(_In_ IXMLDOMNode *pConfigRoot)
     } else
         throw invalid_argument(string_printf(__FUNCTION__ " Unsupported authentication mode (%ls).", (BSTR)auth_mode));
 
+    m_module.log_config((xpath + L"/AuthMode").c_str(), (BSTR)auth_mode);
+
     // Load method configuration.
     config_method_with_cred::load(pConfigRoot);
-
-    m_module.log_config((xpath + L"/AuthMode").c_str(), auth_mode);
 }
 
 

lib/GTC_UI/include/GTC_UI.h

@@ -100,6 +100,14 @@ public:
     ///
     /// Constructs a credential dialog
     ///
+    /// \param[in] prov    Provider configuration data
+    /// \param[in] parent  Parent window
+    /// \param[in] id      An identifier for the dialog. A value of -1 is taken to mean a default.
+    /// \param[in] title   The title of the dialog
+    /// \param[in] pos     The dialog position. The value \c wxDefaultPosition indicates a default position, chosen by either the windowing system or wxWidgets, depending on platform.
+    /// \param[in] size    The dialog size. The value \c wxDefaultSize indicates a default size, chosen by either the windowing system or wxWidgets, depending on platform.
+    /// \param[in] style   The window style
+    ///
     wxGTCResponseDialog(const eap::config_provider &prov, wxWindow *parent, wxWindowID id = wxID_ANY, const wxString &title = _("GTC Challenge"), const wxPoint &pos = wxDefaultPosition, const wxSize &size = wxDefaultSize, long style = wxDEFAULT_DIALOG_STYLE);
 };
 

lib/TLS_UI/res/wxTLS_UI.cpp

@@ -55,7 +55,6 @@ wxTLSServerTrustPanelBase::wxTLSServerTrustPanelBase( wxWindow* parent, wxWindow
 	sb_root_ca_btn->Add( m_root_ca_add_file, 0, wxRIGHT|wxLEFT, 5 );
 	
 	m_root_ca_remove = new wxButton( sb_server_trust->GetStaticBox(), wxID_ANY, _("&Remove CA"), wxDefaultPosition, wxDefaultSize, 0 );
-	m_root_ca_remove->Enable( false );
 	m_root_ca_remove->SetToolTip( _("Removes selected certificate authorities from the list") );
 	
 	sb_root_ca_btn->Add( m_root_ca_remove, 0, wxLEFT, 5 );

lib/TLS_UI/res/wxTLS_UI.fbp

@@ -659,7 +659,7 @@
                                                         <property name="dock">Dock</property>
                                                         <property name="dock_fixed">0</property>
                                                         <property name="docking">Left</property>
-                                                        <property name="enabled">0</property>
+                                                        <property name="enabled">1</property>
                                                         <property name="fg"></property>
                                                         <property name="floatable">1</property>
                                                         <property name="font"></property>

lib/TTLS_UI/src/Module.cpp

@@ -43,7 +43,7 @@ public:
 protected:
     static wxCriticalSection s_lock;        ///< Initialization lock
     static unsigned long s_init_ref_count;  ///< Initialization reference counter
-    static wxLocale s_locale;               ///< Locale
+    static wxLocale *s_locale;              ///< Locale
 };
 
 
@@ -118,19 +118,27 @@ void eap::peer_ttls_ui::invoke_config_ui(
         // Initialize application.
         wxInitializerPeer init(m_instance);
 
-        {
+        wxWindow *parent;
+        if (hwndParent) {
             // Create wxWidget-approved parent window.
-            wxWindow parent;
+            parent = new wxWindow;
-            parent.SetHWND((WXHWND)(hwndParent ? hwndParent : GetForegroundWindow()));
+            parent->SetHWND((WXHWND)hwndParent);
-            parent.AdoptAttributesFromHWND();
+            parent->AdoptAttributesFromHWND();
-            wxTopLevelWindows.Append(&parent);
+            wxTopLevelWindows.Append(parent);
+        } else
+            parent = NULL;
 
         // Create and launch configuration dialog.
-            wxEAPConfigDialog<wxTTLSConfigWindow> dlg(cfg, &parent);
+        wxEAPConfigDialog<wxTTLSConfigWindow> dlg(cfg, parent);
+        if (!parent) {
+            FLASHWINFO fwi = { sizeof(FLASHWINFO), dlg.GetHWND(), FLASHW_ALL | FLASHW_TIMERNOFG };
+            ::FlashWindowEx(&fwi);
+        }
         result = dlg.ShowModal();
 
-            wxTopLevelWindows.DeleteObject(&parent);
+        if (parent) {
-            parent.SetHWND((WXHWND)NULL);
+            wxTopLevelWindows.DeleteObject(parent);
+            parent->SetHWND((WXHWND)NULL);
         }
     }
 
@@ -153,6 +161,9 @@ void eap::peer_ttls_ui::invoke_identity_ui(
     _Out_                                  DWORD  *pdwUserDataOutSize,
     _Out_                                  LPWSTR *ppwszIdentity)
 {
+#ifdef _DEBUG
+    //::Sleep(10000);
+#endif
     assert(ppwszIdentity);
 
     // Unpack configuration.
@@ -178,19 +189,26 @@ void eap::peer_ttls_ui::invoke_identity_ui(
         // Initialize application.
         wxInitializerPeer init(m_instance);
 
-        {
+        wxWindow *parent;
+        if (hwndParent) {
             // Create wxWidget-approved parent window.
-            wxWindow parent;
+            parent = new wxWindow;
-            parent.SetHWND((WXHWND)(hwndParent ? hwndParent : GetForegroundWindow()));
+            parent->SetHWND((WXHWND)hwndParent);
-            parent.AdoptAttributesFromHWND();
+            parent->AdoptAttributesFromHWND();
-            wxTopLevelWindows.Append(&parent);
+            wxTopLevelWindows.Append(parent);
+        } else
+            parent = NULL;
 
         if (cfg.m_providers.size() > 1) {
             // Multiple identity providers: User has to select one first.
-                wxEAPProviderSelectDialog dlg(cfg, &parent);
+            wxEAPProviderSelectDialog dlg(cfg, parent);
 
             // Centre and display dialog.
             dlg.Centre(wxBOTH);
+            if (!parent) {
+                FLASHWINFO fwi = { sizeof(FLASHWINFO), dlg.GetHWND(), FLASHW_ALL | FLASHW_TIMERNOFG };
+                ::FlashWindowEx(&fwi);
+            }
             if ((result = dlg.ShowModal()) == wxID_OK) {
                 cfg_prov = dlg.GetSelection();
                 assert(cfg_prov);
@@ -240,7 +258,7 @@ void eap::peer_ttls_ui::invoke_identity_ui(
                 src_outer != eap::credentials::source_config && eap::config_method::status_cred_begin <= cfg_method->m_last_status && cfg_method->m_last_status < eap::config_method::status_cred_end)
             {
                 // Build dialog to prompt for outer credentials.
-                    wxEAPCredentialsDialog dlg(*cfg_prov, &parent);
+                wxEAPCredentialsDialog dlg(*cfg_prov, parent);
                 if (eap::config_method::status_cred_begin <= cfg_method->m_last_status && cfg_method->m_last_status < eap::config_method::status_cred_end)
                     dlg.AddContent(new wxEAPCredentialWarningPanel(*cfg_prov, cfg_method->m_last_status, &dlg));
                 auto panel = new wxTLSCredentialsPanel(*cfg_prov, *cfg_method, *cred, &dlg, false);
@@ -253,6 +271,10 @@ void eap::peer_ttls_ui::invoke_identity_ui(
 
                 // Centre and display dialog.
                 dlg.Centre(wxBOTH);
+                if (!parent) {
+                    FLASHWINFO fwi = { sizeof(FLASHWINFO), dlg.GetHWND(), FLASHW_ALL | FLASHW_TIMERNOFG };
+                    ::FlashWindowEx(&fwi);
+                }
                 if ((result = dlg.ShowModal()) == wxID_OK) {
                     // Write credentials to credential manager.
                     if (panel->GetRemember()) {
@@ -290,7 +312,7 @@ void eap::peer_ttls_ui::invoke_identity_ui(
 #endif
                     {
                         // Native inner methods. Build dialog to prompt for inner credentials.
-                            wxEAPCredentialsDialog dlg(*cfg_prov, &parent);
+                        wxEAPCredentialsDialog dlg(*cfg_prov, parent);
                         if (eap::config_method::status_cred_begin <= cfg_method->m_inner->m_last_status && cfg_method->m_inner->m_last_status < eap::config_method::status_cred_end)
                             dlg.AddContent(new wxEAPCredentialWarningPanel(*cfg_prov, cfg_method->m_inner->m_last_status, &dlg));
                         wxEAPCredentialsPanelBase *panel = NULL;
@@ -321,6 +343,10 @@ void eap::peer_ttls_ui::invoke_identity_ui(
 
                         // Centre and display dialog.
                         dlg.Centre(wxBOTH);
+                        if (!parent) {
+                            FLASHWINFO fwi = { sizeof(FLASHWINFO), dlg.GetHWND(), FLASHW_ALL | FLASHW_TIMERNOFG };
+                            ::FlashWindowEx(&fwi);
+                        }
                         if ((result = dlg.ShowModal()) == wxID_OK) {
                             // Write credentials to credential manager.
                             if (panel->GetRemember()) {
@@ -374,8 +400,9 @@ void eap::peer_ttls_ui::invoke_identity_ui(
             }
         }
 
-            wxTopLevelWindows.DeleteObject(&parent);
+        if (parent) {
-            parent.SetHWND((WXHWND)NULL);
+            wxTopLevelWindows.DeleteObject(parent);
+            parent->SetHWND((WXHWND)NULL);
         }
     }
 
@@ -434,12 +461,15 @@ void eap::peer_ttls_ui::invoke_interactive_ui(
             // Initialize application.
             wxInitializerPeer init(m_instance);
 
-            {
+            wxWindow *parent;
+            if (hwndParent) {
                 // Create wxWidget-approved parent window.
-                wxWindow parent;
+                parent = new wxWindow;
-                parent.SetHWND((WXHWND)(hwndParent ? hwndParent : GetForegroundWindow()));
+                parent->SetHWND((WXHWND)hwndParent);
-                parent.AdoptAttributesFromHWND();
+                parent->AdoptAttributesFromHWND();
-                wxTopLevelWindows.Append(&parent);
+                wxTopLevelWindows.Append(parent);
+            } else
+                parent = NULL;
 
             {
                 sanitizing_wstring
@@ -447,7 +477,7 @@ void eap::peer_ttls_ui::invoke_interactive_ui(
                     response;
 
                 // Build dialog to prompt for response.
-                    wxGTCResponseDialog dlg(*cfg_prov, &parent);
+                wxGTCResponseDialog dlg(*cfg_prov, parent);
                 auto panel = new wxGTCResponsePanel(response, challenge.c_str(), &dlg);
                 dlg.AddContent(panel);
 
@@ -457,6 +487,10 @@ void eap::peer_ttls_ui::invoke_interactive_ui(
 
                 // Centre and display dialog.
                 dlg.Centre(wxBOTH);
+                if (!parent) {
+                    FLASHWINFO fwi = { sizeof(FLASHWINFO), dlg.GetHWND(), FLASHW_ALL | FLASHW_TIMERNOFG };
+                    ::FlashWindowEx(&fwi);
+                }
                 if ((result = dlg.ShowModal()) == wxID_OK) {
                     // Save response.
                     ctx.m_data.assign(
@@ -465,8 +499,9 @@ void eap::peer_ttls_ui::invoke_interactive_ui(
                 }
             }
 
-                wxTopLevelWindows.DeleteObject(&parent);
+            if (parent) {
-                parent.SetHWND((WXHWND)NULL);
+                wxTopLevelWindows.DeleteObject(parent);
+                parent->SetHWND((WXHWND)NULL);
             }
         }
 
@@ -523,9 +558,10 @@ wxInitializerPeer::wxInitializerPeer(_In_ HINSTANCE instance)
 
     // Do our wxWidgets configuration and localization initialization.
     wxInitializeConfig();
-    if (wxInitializeLocale(s_locale)) {
+    s_locale = new wxLocale;
-        s_locale.AddCatalog(wxT("wxExtend") wxT(wxExtendVersion));
+    if (wxInitializeLocale(*s_locale)) {
-        s_locale.AddCatalog(wxT("EAPTTLSUI"));
+        s_locale->AddCatalog(wxT("wxExtend") wxT(wxExtendVersion));
+        s_locale->AddCatalog(wxT("EAPTTLSUI"));
     }
 }
 
@@ -537,9 +573,14 @@ wxInitializerPeer::~wxInitializerPeer()
         return;
 
     wxEntryCleanup();
+
+    if (s_locale) {
+        delete s_locale;
+        s_locale = NULL;
+    }
 }
 
 
 wxCriticalSection wxInitializerPeer::s_lock;
 unsigned long wxInitializerPeer::s_init_ref_count = 0;
-wxLocale wxInitializerPeer::s_locale;
+wxLocale *wxInitializerPeer::s_locale = NULL;