* Fixes NB#90984, clean a password before freeing memory
[modest] / src / modest-account-mgr-helpers.c
index d1c3626..989663d 100644 (file)
@@ -31,7 +31,9 @@
 #include <modest-account-mgr-priv.h>
 #include <tny-simple-list.h>
 #include <modest-runtime.h>
+#include <modest-defs.h>
 #include <string.h>
+#include <strings.h>
 
 static const gchar * null_means_empty (const gchar * str);
 
@@ -65,27 +67,27 @@ gboolean modest_account_mgr_set_signature (ModestAccountMgr *self, const gchar*
        return result;
 }
 
-gchar* modest_account_mgr_get_signature (ModestAccountMgr *self, const gchar* name, 
-       gboolean* use_signature)
+gchar* 
+modest_account_mgr_get_signature (ModestAccountMgr *self, 
+                                 const gchar* name, 
+                                 gboolean* use_signature)
 {
-       if (use_signature) {
-               *use_signature = 
-                       modest_account_mgr_get_bool (self, name, MODEST_ACCOUNT_USE_SIGNATURE, FALSE);
-       }
+       *use_signature = 
+               modest_account_mgr_get_bool (self, name, MODEST_ACCOUNT_USE_SIGNATURE, FALSE);
        
        return modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_SIGNATURE, FALSE);
 }
 
-ModestTransportStoreProtocol modest_account_mgr_get_store_protocol (ModestAccountMgr *self, const gchar* name)
+ModestProtocolType modest_account_mgr_get_store_protocol (ModestAccountMgr *self, const gchar* name)
 {
-       ModestTransportStoreProtocol result = MODEST_PROTOCOL_STORE_POP; /* Arbitrary default */
+       ModestProtocolType result = MODEST_PROTOCOLS_STORE_POP; /* Arbitrary default */
        
        gchar *server_account_name = modest_account_mgr_get_string (self, name,
                                                                   MODEST_ACCOUNT_STORE_ACCOUNT,
                                                                   FALSE);
        if (server_account_name) {
               ModestServerAccountSettings* server_settings = 
-                       modest_account_mgr_load_server_settings (self, server_account_name);
+                       modest_account_mgr_load_server_settings (self, server_account_name, FALSE);
                result = modest_server_account_settings_get_protocol (server_settings);
               
                g_object_unref (server_settings);
@@ -98,9 +100,9 @@ ModestTransportStoreProtocol modest_account_mgr_get_store_protocol (ModestAccoun
 
 
 gboolean modest_account_mgr_set_connection_specific_smtp (ModestAccountMgr *self, 
-       const gchar* connection_name, const gchar* server_account_name)
+       const gchar* connection_id, const gchar* server_account_name)
 {
-       modest_account_mgr_remove_connection_specific_smtp (self, connection_name);
+       modest_account_mgr_remove_connection_specific_smtp (self, connection_id);
        
        ModestConf *conf = MODEST_ACCOUNT_MGR_GET_PRIVATE (self)->modest_conf;
 
@@ -115,7 +117,7 @@ gboolean modest_account_mgr_set_connection_specific_smtp (ModestAccountMgr *self
                result = FALSE;
        } else {        
                /* The server account is in the item after the connection name: */
-               list = g_slist_append (list, (gpointer)connection_name);
+               list = g_slist_append (list, (gpointer)connection_id);
                list = g_slist_append (list, (gpointer)server_account_name);
        
                /* Reset the changed list: */
@@ -138,22 +140,22 @@ gboolean modest_account_mgr_set_connection_specific_smtp (ModestAccountMgr *self
  * modest_account_mgr_remove_connection_specific_smtp
  * @self: a ModestAccountMgr instance
  * @name: the account name
- * @connection_name: A libconic IAP connection name
+ * @connection_id: A libconic IAP connection id
  * 
  * Disassacoiate a server account to use with the specific connection for this account.
  *
  * Returns: TRUE if it worked, FALSE otherwise
  */                             
 gboolean modest_account_mgr_remove_connection_specific_smtp (ModestAccountMgr *self, 
-       const gchar* connection_name)
+       const gchar* connection_id)
 {
        ModestAccountMgrPrivate *priv = MODEST_ACCOUNT_MGR_GET_PRIVATE (self);
        
        gboolean result = TRUE;
        GError *err = NULL;
        GSList *list = modest_conf_get_list (priv->modest_conf, 
-                                                       MODEST_CONF_CONNECTION_SPECIFIC_SMTP_LIST,
-                                                   MODEST_CONF_VALUE_STRING, &err);
+                                            MODEST_CONF_CONNECTION_SPECIFIC_SMTP_LIST,
+                                            MODEST_CONF_VALUE_STRING, &err);
        if (err) {
                g_printerr ("modest: %s: error getting list: %s.\n", __FUNCTION__, err->message);
                g_error_free (err);
@@ -165,7 +167,7 @@ gboolean modest_account_mgr_remove_connection_specific_smtp (ModestAccountMgr *s
                return FALSE;
                
        /* The server account is in the item after the connection name: */
-       GSList *list_connection = g_slist_find_custom (list, connection_name, (GCompareFunc)strcmp);
+       GSList *list_connection = g_slist_find_custom (list, connection_id, (GCompareFunc)strcmp);
        if (list_connection) {
                GSList *account_node = g_slist_next (list_connection);
                /* remove both items: */
@@ -205,13 +207,13 @@ gboolean modest_account_mgr_set_use_connection_specific_smtp (ModestAccountMgr *
 /**
  * modest_account_mgr_get_connection_specific_smtp
  * @self: a ModestAccountMgr instance
- * @connection_name: A libconic IAP connection name
+ * @connection_id: A libconic IAP connection id
  * 
  * Retrieve a server account to use with this specific connection for this account.
  *
  * Returns: a server account name to use for this connection, or NULL if none is specified.
  */                     
-gchar* modest_account_mgr_get_connection_specific_smtp (ModestAccountMgr *self,  const gchar* connection_name)
+gchar* modest_account_mgr_get_connection_specific_smtp (ModestAccountMgr *self,  const gchar* connection_id)
 {
        gchar *result = NULL;
        
@@ -232,8 +234,8 @@ gchar* modest_account_mgr_get_connection_specific_smtp (ModestAccountMgr *self,
        /* The server account is in the item after the connection name: */
        GSList *iter = list;
        while (iter) {
-               const gchar* this_connection_name = (const gchar*)(iter->data);
-               if (strcmp (this_connection_name, connection_name) == 0) {
+               const gchar* this_connection_id = (const gchar*)(iter->data);
+               if (strcmp (this_connection_id, connection_id) == 0) {
                        iter = g_slist_next (iter);
                        
                        if (iter) {
@@ -253,7 +255,7 @@ gchar* modest_account_mgr_get_connection_specific_smtp (ModestAccountMgr *self,
                
        /*
        if (!result) {
-               printf ("  debug: no server found for connection_name=%s.\n", connection_name); 
+               printf ("  debug: no server found for connection_id=%s.\n", connection_id);     
        }
        */
                                
@@ -284,8 +286,7 @@ modest_account_mgr_set_server_account_username (ModestAccountMgr *self, const gc
                
        /* We don't know anything about new usernames: */
        if (strcmp (existing_username, username) != 0)
-               modest_account_mgr_set_server_account_username_has_succeeded (self, account_name,
-                                                                             TRUE);
+               modest_account_mgr_set_server_account_username_has_succeeded (self, account_name, FALSE);
                
        g_free (existing_username);
 }
@@ -306,15 +307,6 @@ modest_account_mgr_set_server_account_username_has_succeeded (ModestAccountMgr *
                                     succeeded, TRUE /* server account */);
 }
 
-void
-modest_account_mgr_set_server_account_password (ModestAccountMgr *self, const gchar* account_name, 
-                                   const gchar* password)
-{
-       modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PASSWORD, 
-                                      password, TRUE /* server account */);
-}
-
-       
 gchar*
 modest_account_mgr_get_server_account_password (ModestAccountMgr *self, const gchar* account_name)
 {
@@ -330,8 +322,11 @@ modest_account_mgr_get_server_account_has_password (ModestAccountMgr *self, cons
                TRUE /* server account */);
        if (password && strlen (password)) {
                result = TRUE;
-       }
        
+               /* Clean password */
+               bzero (password, strlen (password));
+       }
+
        g_free (password);
        return result;
 }
@@ -361,17 +356,26 @@ modest_account_mgr_set_server_account_hostname (ModestAccountMgr *self,
 
 
 
-ModestAuthProtocol
+ModestProtocolType
 modest_account_mgr_get_server_account_secure_auth (ModestAccountMgr *self, 
        const gchar* account_name)
 {
-       ModestAuthProtocol result = MODEST_PROTOCOL_AUTH_NONE;
-       gchar* value = modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_AUTH_MECH, 
-               TRUE /* server account */);
+       ModestProtocolRegistry *protocol_registry;
+       ModestProtocolType result = MODEST_PROTOCOLS_AUTH_NONE;
+       gchar* value;
+
+       protocol_registry = modest_runtime_get_protocol_registry ();
+       value = modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_AUTH_MECH, 
+                                              TRUE /* server account */);
        if (value) {
-               result = modest_protocol_info_get_auth_protocol (value);
-                       
+               ModestProtocol *protocol;
+
+               protocol = modest_protocol_registry_get_protocol_by_name (protocol_registry, MODEST_PROTOCOL_REGISTRY_AUTH_PROTOCOLS, value);
                g_free (value);
+
+               if (protocol)
+                       result = modest_protocol_get_type_id (protocol);
+                       
        }
        
        return result;
@@ -380,28 +384,42 @@ modest_account_mgr_get_server_account_secure_auth (ModestAccountMgr *self,
 
 void
 modest_account_mgr_set_server_account_secure_auth (ModestAccountMgr *self, 
-       const gchar* account_name, ModestAuthProtocol secure_auth)
+       const gchar* account_name, ModestProtocolType secure_auth)
 {
-       /* Get the conf string for the enum value: */
-       const gchar* str_value = NULL;
-
-       str_value = modest_protocol_info_get_auth_protocol_name (secure_auth);
+       const gchar* str_value;
+       ModestProtocolRegistry *protocol_registry;
+       ModestProtocol *protocol;
+
+       /* Get the conf string for the protocol: */
+       protocol_registry = modest_runtime_get_protocol_registry ();
+       protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry, secure_auth);
+       str_value = modest_protocol_get_name (protocol);
        
        /* Set it in the configuration: */
        modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_AUTH_MECH, str_value, TRUE);
 }
 
-ModestConnectionProtocol
+ModestProtocolType
 modest_account_mgr_get_server_account_security (ModestAccountMgr *self, 
        const gchar* account_name)
 {
-       ModestConnectionProtocol result = MODEST_PROTOCOL_CONNECTION_NORMAL;
-       gchar* value = modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_SECURITY, 
-               TRUE /* server account */);
+       ModestProtocolType result = MODEST_PROTOCOLS_CONNECTION_NONE;
+       gchar* value;
+
+       value = modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_SECURITY, 
+                                              TRUE /* server account */);
        if (value) {
-               result = modest_protocol_info_get_connection_protocol (value);
-                       
+               ModestProtocolRegistry *protocol_registry;
+               ModestProtocol *protocol;
+
+               protocol_registry = modest_runtime_get_protocol_registry ();
+               protocol = modest_protocol_registry_get_protocol_by_name (protocol_registry,
+                                                                         MODEST_PROTOCOL_REGISTRY_CONNECTION_PROTOCOLS,
+                                                                         value);
                g_free (value);
+
+               if (protocol)
+                       result = modest_protocol_get_type_id (protocol);
        }
        
        return result;
@@ -409,23 +427,32 @@ modest_account_mgr_get_server_account_security (ModestAccountMgr *self,
 
 void
 modest_account_mgr_set_server_account_security (ModestAccountMgr *self, 
-       const gchar* account_name, ModestConnectionProtocol security)
+                                               const gchar* account_name, 
+                                               ModestProtocolType security)
 {
-       /* Get the conf string for the enum value: */
-       const gchar* str_value = NULL;
-       str_value = modest_protocol_info_get_connection_protocol_name (security);
+       const gchar* str_value;
+       ModestProtocolRegistry *protocol_registry;
+       ModestProtocol *protocol;
+
+       /* Get the conf string for the protocol type: */
+       protocol_registry = modest_runtime_get_protocol_registry ();
+       protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry, security);
+       str_value = modest_protocol_get_name (protocol);
        
        /* Set it in the configuration: */
        modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_SECURITY, str_value, TRUE);
 }
 
 ModestServerAccountSettings*
-modest_account_mgr_load_server_settings (ModestAccountMgr *self, const gchar* name)
+modest_account_mgr_load_server_settings (ModestAccountMgr *self, const gchar* name, gboolean is_transport_and_not_store)
 {
        ModestServerAccountSettings *settings;
+       ModestProtocol *protocol;
+       ModestProtocolRegistry *registry;
        gchar *string;
        
        g_return_val_if_fail (modest_account_mgr_account_exists (self, name, TRUE), NULL);
+       registry = modest_runtime_get_protocol_registry ();
        settings = modest_server_account_settings_new ();
 
        modest_server_account_settings_set_account_name (settings, name);
@@ -441,28 +468,49 @@ modest_account_mgr_load_server_settings (ModestAccountMgr *self, const gchar* na
        g_free (string);
 
        string = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_PROTO, TRUE);
+       if (is_transport_and_not_store) {
+               protocol = modest_protocol_registry_get_protocol_by_name (registry, MODEST_PROTOCOL_REGISTRY_TRANSPORT_PROTOCOLS, string);
+       } else {
+               protocol = modest_protocol_registry_get_protocol_by_name (registry, MODEST_PROTOCOL_REGISTRY_STORE_PROTOCOLS, string);
+       }
        modest_server_account_settings_set_protocol (settings,
-                                                    modest_protocol_info_get_transport_store_protocol (string));
+                                                    modest_protocol_get_type_id (protocol));
        g_free (string);
 
        modest_server_account_settings_set_port (settings,
                                                 modest_account_mgr_get_int (self, name, MODEST_ACCOUNT_PORT, TRUE));
 
        string = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_AUTH_MECH, TRUE);
-       modest_server_account_settings_set_auth_protocol (settings,
-                                                         modest_protocol_info_get_auth_protocol(string));
-       g_free (string);
-               
+       if (string) {
+               protocol = modest_protocol_registry_get_protocol_by_name (registry, MODEST_PROTOCOL_REGISTRY_AUTH_PROTOCOLS, string);
+               modest_server_account_settings_set_auth_protocol (settings,
+                                                                 modest_protocol_get_type_id (protocol));
+               g_free (string);
+       } else {
+               modest_server_account_settings_set_auth_protocol (settings, MODEST_PROTOCOLS_AUTH_NONE);
+       }
+       
        string = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_SECURITY, TRUE);
-       modest_server_account_settings_set_security (settings,
-                                                    modest_protocol_info_get_connection_protocol(string));
-       g_free (string);
+       if (string) {
+               protocol = modest_protocol_registry_get_protocol_by_name (registry, MODEST_PROTOCOL_REGISTRY_CONNECTION_PROTOCOLS, string);
+               modest_server_account_settings_set_security_protocol (settings,
+                                                                     modest_protocol_get_type_id (protocol));
+               g_free (string);
+       } else {
+               modest_server_account_settings_set_security_protocol (settings,
+                                                                     MODEST_PROTOCOLS_CONNECTION_NONE);
+       }
 
        string = modest_account_mgr_get_string (self, name, 
                                                MODEST_ACCOUNT_PASSWORD, TRUE);
        modest_server_account_settings_set_password (settings, string);
        g_free (string);
        
+       string = modest_account_mgr_get_string (self, name, 
+                                               MODEST_ACCOUNT_URI, TRUE);
+       modest_server_account_settings_set_uri (settings, string);
+       g_free (string);
+       
        return settings;
 }
 
@@ -472,50 +520,70 @@ modest_account_mgr_save_server_settings (ModestAccountMgr *self,
 {
        gboolean has_errors = FALSE;
        const gchar *account_name;
-       const gchar *protocol;
+       const gchar *protocol_name;
        const gchar *uri;
+       ModestProtocolRegistry *protocol_registry;
+       ModestProtocol *protocol;
        
        g_return_val_if_fail (MODEST_IS_SERVER_ACCOUNT_SETTINGS (settings), FALSE);
+       protocol_registry = modest_runtime_get_protocol_registry ();
        account_name = modest_server_account_settings_get_account_name (settings);
 
        /* if we don't have a valid account name we cannot save */
        g_return_val_if_fail (account_name, FALSE);
 
-       protocol = modest_protocol_info_get_transport_store_protocol_name (
-               modest_server_account_settings_get_protocol (settings));
+       protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry,
+                                                                 modest_server_account_settings_get_protocol (settings));
+       protocol_name = modest_protocol_get_name (protocol);
        uri = modest_server_account_settings_get_uri (settings);
        if (!uri) {
-               const gchar *hostname = null_means_empty (modest_server_account_settings_get_hostname (settings));
-               const gchar *username = null_means_empty (modest_server_account_settings_get_username (settings));
-               const gchar *password = null_means_empty (modest_server_account_settings_get_password (settings));
-               gint port = modest_server_account_settings_get_port (settings);
-               const gchar *auth_protocol = modest_protocol_info_get_auth_protocol_name (
-                       modest_server_account_settings_get_auth_protocol (settings));
-               const gchar *security = modest_protocol_info_get_connection_protocol_name (
-                       modest_server_account_settings_get_security (settings));
+               const gchar *hostname;
+               const gchar *username;
+               const gchar *password;
+               gint port;
+               const gchar *auth_protocol_name;
+               const gchar *security_name;
+
+               hostname = null_means_empty (modest_server_account_settings_get_hostname (settings));
+               username = null_means_empty (modest_server_account_settings_get_username (settings));
+               password = null_means_empty (modest_server_account_settings_get_password (settings));
+               port = modest_server_account_settings_get_port (settings);
+               protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry,
+                                                                         modest_server_account_settings_get_auth_protocol (settings));
+               auth_protocol_name = modest_protocol_get_name (protocol);
+               protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry,
+                                                                         modest_server_account_settings_get_security_protocol (settings));
+               security_name = modest_protocol_get_name (protocol);
 
                has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_HOSTNAME, 
                                                            hostname, TRUE);
-               has_errors || (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_USERNAME,
+               if (!has_errors)
+                       (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_USERNAME,
                                                                           username, TRUE));
-               has_errors || (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PASSWORD,
+               if (!has_errors)
+                       (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PASSWORD,
                                                                           password, TRUE));
-               has_errors || (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PROTO,
-                                                                          protocol, TRUE));
-               has_errors || (has_errors = !modest_account_mgr_set_int (self, account_name, MODEST_ACCOUNT_PORT,
+               if (!has_errors)
+                       (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PROTO,
+                                                                          protocol_name, TRUE));
+               if (!has_errors)
+                       (has_errors = !modest_account_mgr_set_int (self, account_name, MODEST_ACCOUNT_PORT,
                                                                        port, TRUE));
-               has_errors || (has_errors = !modest_account_mgr_set_string (self, account_name, 
+               if (!has_errors)
+                       (has_errors = !modest_account_mgr_set_string (self, account_name, 
                                                                           MODEST_ACCOUNT_AUTH_MECH,
-                                                                          auth_protocol, TRUE));               
-               has_errors || (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_SECURITY,
-                                                                          security,
+                                                                          auth_protocol_name, TRUE));          
+               if (!has_errors)
+                       (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_SECURITY,
+                                                                          security_name,
                                                                           TRUE));
        } else {
                const gchar *uri = modest_server_account_settings_get_uri (settings);
                has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_URI,
                                                            uri, TRUE);
-               has_errors || (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PROTO,
-                                                                          protocol, TRUE));
+               if (!has_errors)
+                       (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PROTO,
+                                                                          protocol_name, TRUE));
        }
 
        return !has_errors;
@@ -566,6 +634,7 @@ modest_account_mgr_load_account_settings (ModestAccountMgr *self,
 
        modest_account_settings_set_enabled (settings, modest_account_mgr_get_enabled (self, name));
        modest_account_settings_set_retrieve_type (settings, modest_account_mgr_get_retrieve_type (self, name));
+       modest_account_settings_set_retrieve_limit (settings, modest_account_mgr_get_retrieve_limit (self, name));
 
        default_account    = modest_account_mgr_get_default_account (self);
        modest_account_settings_set_is_default (settings,
@@ -588,7 +657,7 @@ modest_account_mgr_load_account_settings (ModestAccountMgr *self,
                                                            FALSE);
        if (server_account) {
                ModestServerAccountSettings *store_settings;
-               store_settings = modest_account_mgr_load_server_settings (self, server_account);
+               store_settings = modest_account_mgr_load_server_settings (self, server_account, FALSE);
                modest_account_settings_set_store_settings (settings,
                                                            store_settings);
                g_object_unref (store_settings);
@@ -601,7 +670,7 @@ modest_account_mgr_load_account_settings (ModestAccountMgr *self,
                                                        FALSE);
        if (server_account) {
                ModestServerAccountSettings *transport_settings;
-               transport_settings = modest_account_mgr_load_server_settings (self, server_account);
+               transport_settings = modest_account_mgr_load_server_settings (self, server_account, TRUE);
                modest_account_settings_set_transport_settings (settings, transport_settings);
                g_object_unref (transport_settings);
                g_free (server_account);