From: Dirk-Jan C. Binnema <dirk-jan.binnema@nokia.com>
Date: Tue, 30 Oct 2007 11:49:38 +0000 (+0000)
Subject: * fix for the crash in NB#74605
X-Git-Tag: git_migration_finished~2199
X-Git-Url: http://git.maemo.org/git/?p=modest;a=commitdiff_plain;h=5d50cd32592c4b11955589126a2017e7048e508c

* fix for the crash in NB#74605
* code cleanup

pmo-trunk-r3601
---

diff --git a/src/modest-protocol-info.c b/src/modest-protocol-info.c
index 5a579aa..a771c01 100644
--- a/src/modest-protocol-info.c
+++ b/src/modest-protocol-info.c
@@ -32,7 +32,7 @@
 #include <modest-protocol-info.h>
 #include <modest-pair.h>
 #include <modest-defs.h>
-
+#include <modest-text-utils.h>
 
 typedef struct {
 	gint   proto;
@@ -94,35 +94,32 @@ get_protocol_by_name (const ProtocolInfo* map,
 	guint i;
 
 	g_return_val_if_fail (map, default_value);
-
+	g_return_val_if_fail (query_name, default_value);
+	
 	for(i = 0; i < size; ++i)
-	{
-		if((case_sensitive && strcmp(map[i].name, query_name) == 0) ||
-		   (!case_sensitive && g_ascii_strcasecmp(map[i].name, query_name) == 0))
-		{
+		if (modest_text_utils_utf8_strcmp (map[i].name, query_name,
+						   !case_sensitive) == 0)
 			return map[i].proto;
-		}
-	}
-
+	
 	return default_value;
 }
 
 ModestPairList*
-modest_protocol_info_get_transport_store_protocol_pair_list ()
+modest_protocol_info_get_transport_store_protocol_pair_list (void)
 {
 	return get_protocol_pair_list (TransportStoreProtocolMap,
 		G_N_ELEMENTS(TransportStoreProtocolMap));
 }
 
 ModestPairList*
-modest_protocol_info_get_auth_protocol_pair_list ()
+modest_protocol_info_get_auth_protocol_pair_list (void)
 {
 	return get_protocol_pair_list (AuthProtocolMap,
 		G_N_ELEMENTS(AuthProtocolMap));
 }
 
 ModestPairList*
-modest_protocol_info_get_connection_protocol_pair_list ()
+modest_protocol_info_get_connection_protocol_pair_list (void)
 {
 	return get_protocol_pair_list (ConnectionProtocolMap,
 		G_N_ELEMENTS(ConnectionProtocolMap));
@@ -131,6 +128,8 @@ modest_protocol_info_get_connection_protocol_pair_list ()
 ModestTransportStoreProtocol
 modest_protocol_info_get_transport_store_protocol (const gchar* name)
 {
+	g_return_val_if_fail (name, MODEST_PROTOCOL_TRANSPORT_STORE_UNKNOWN);
+	
 	return get_protocol_by_name(TransportStoreProtocolMap,
 	                            G_N_ELEMENTS(TransportStoreProtocolMap),
 	                            name,
@@ -141,6 +140,8 @@ modest_protocol_info_get_transport_store_protocol (const gchar* name)
 ModestAuthProtocol
 modest_protocol_info_get_auth_protocol (const gchar* name)
 {
+	g_return_val_if_fail (name, MODEST_PROTOCOL_AUTH_NONE);
+	
 	return get_protocol_by_name(AuthProtocolMap,
 	                            G_N_ELEMENTS(AuthProtocolMap),
 	                            name,
diff --git a/src/modest-protocol-info.h b/src/modest-protocol-info.h
index 77f16d9..9ddc12b 100644
--- a/src/modest-protocol-info.h
+++ b/src/modest-protocol-info.h
@@ -88,8 +88,7 @@ modest_protocol_info_get_transport_store_protocol_pair_list ();
  * Returns: a list of protocols. After use, it should be freed
  * with modest_pair_list_free
  */
-ModestPairList*
-modest_protocol_info_get_auth_protocol_pair_list ();
+ModestPairList* modest_protocol_info_get_auth_protocol_pair_list (void);
 
 
 /**
@@ -102,8 +101,7 @@ modest_protocol_info_get_auth_protocol_pair_list ();
  * Returns: a list of protocols. After use, it should be freed
  * with modest_pair_list_free
  */
-ModestPairList*
-modest_protocol_info_get_connection_protocol_pair_list ();
+ModestPairList* modest_protocol_info_get_connection_protocol_pair_list (void);
 	
 	
 /**
@@ -114,8 +112,7 @@ modest_protocol_info_get_connection_protocol_pair_list ();
  *  
  * Returns: the id of the protocol or MODEST_PROTOCOL_TRANSPORT_STORE_UNKNOWN
  */
-ModestTransportStoreProtocol
-modest_protocol_info_get_transport_store_protocol (const gchar* name);
+ModestTransportStoreProtocol modest_protocol_info_get_transport_store_protocol (const gchar* name);
 
 /**
  * modest_protocol_info_get_auth_protocol:
@@ -125,8 +122,7 @@ modest_protocol_info_get_transport_store_protocol (const gchar* name);
  *
  * Returns: The ID of the protocol or MODEST_PROTOCOL_AUTH_NONE
  */
-ModestAuthProtocol
-modest_protocol_info_get_auth_protocol (const gchar* name);
+ModestAuthProtocol modest_protocol_info_get_auth_protocol (const gchar* name);
 
 /**
  * modest_protocol_info_get_transport_store_protocol_name:
@@ -136,8 +132,7 @@ modest_protocol_info_get_auth_protocol (const gchar* name);
  *  
  * Returns: string id of the proto as a constant string, that should NOT be modified or freed
  */
-const gchar*
-modest_protocol_info_get_transport_store_protocol_name (ModestTransportStoreProtocol proto);
+const gchar* modest_protocol_info_get_transport_store_protocol_name (ModestTransportStoreProtocol proto);
 
 /**
  * modest_protocol_info_get_auth_protocol_name:
@@ -147,8 +142,7 @@ modest_protocol_info_get_transport_store_protocol_name (ModestTransportStoreProt
  *  
  * Returns: string id of the proto as a constant string, that should NOT be modified or freed
  */
-const gchar*
-modest_protocol_info_get_auth_protocol_name (ModestAuthProtocol proto);
+const gchar* modest_protocol_info_get_auth_protocol_name (ModestAuthProtocol proto);
 
 /*
  * modest_protocol_get_auth_protocol_pair_list:
@@ -158,9 +152,7 @@ modest_protocol_info_get_auth_protocol_name (ModestAuthProtocol proto);
  *
  * Returns: List of method/display name pairs
  */
-
-ModestPairList*
-modest_protocol_info_get_auth_protocol_pair_list ();
+ModestPairList* modest_protocol_info_get_auth_protocol_pair_list (void);
 
 /**
  * modest_protocol_info_get_auth_protocol_name: