aboutsummaryrefslogtreecommitdiffstats
path: root/e-util/e-passwords.c
diff options
context:
space:
mode:
authorDan Winship <danw@src.gnome.org>2001-10-23 02:46:46 +0800
committerDan Winship <danw@src.gnome.org>2001-10-23 02:46:46 +0800
commit44faa44bffd7fae58fa58e9eebda4b3244d025f8 (patch)
treed54d2dfd8a2caa8616a35960af1d979008f8936d /e-util/e-passwords.c
parent708dc70ac5b5cf2711a1f43293a9e0a4467adcb7 (diff)
downloadgsoc2013-evolution-44faa44bffd7fae58fa58e9eebda4b3244d025f8.tar
gsoc2013-evolution-44faa44bffd7fae58fa58e9eebda4b3244d025f8.tar.gz
gsoc2013-evolution-44faa44bffd7fae58fa58e9eebda4b3244d025f8.tar.bz2
gsoc2013-evolution-44faa44bffd7fae58fa58e9eebda4b3244d025f8.tar.lz
gsoc2013-evolution-44faa44bffd7fae58fa58e9eebda4b3244d025f8.tar.xz
gsoc2013-evolution-44faa44bffd7fae58fa58e9eebda4b3244d025f8.tar.zst
gsoc2013-evolution-44faa44bffd7fae58fa58e9eebda4b3244d025f8.zip
Pass a CORBA_Environment to bonobo_config_get_string so it doesn't g_warn
* e-passwords.c (e_passwords_get_password): Pass a CORBA_Environment to bonobo_config_get_string so it doesn't g_warn on error. (Since the "error" is most likely just that the password isn't cached.) (e_passwords_remember_password, e_passwords_get_password, e_passwords_add_password): Change "if (foo) { entire function; }" to "if (!foo) return;" * e-html-utils.c (special_chars): Don't allow single quote or backtick in email addresses, or pipes following URLs. svn path=/trunk/; revision=13887
Diffstat (limited to 'e-util/e-passwords.c')
-rw-r--r--e-util/e-passwords.c116
1 files changed, 61 insertions, 55 deletions
diff --git a/e-util/e-passwords.c b/e-util/e-passwords.c
index 2222b6e211..363c5592c7 100644
--- a/e-util/e-passwords.c
+++ b/e-util/e-passwords.c
@@ -168,35 +168,34 @@ e_passwords_clear_component_passwords ()
void
e_passwords_remember_password (const char *key)
{
- char *okey, *value;
-
- if (g_hash_table_lookup_extended (passwords, key,
- (gpointer*)&okey, (gpointer*)&value)) {
- char *path, *key64, *pass64;
- int len, state, save;
-
- /* add it to the on-disk cache of passwords */
- len = strlen (okey);
- key64 = g_malloc0 ((len + 2) * 4 / 3 + 1);
- state = save = 0;
- base64_encode_close (okey, len, FALSE, key64, &state, &save);
- path = g_strdup_printf ("/Passwords/%s/%s", component_name, key64);
- g_free (key64);
-
- len = strlen (value);
- pass64 = g_malloc0 ((len + 2) * 4 / 3 + 1);
- state = save = 0;
- base64_encode_close (value, len, FALSE, pass64, &state, &save);
-
- bonobo_config_set_string (db, path, pass64, NULL);
- g_free (path);
- g_free (pass64);
+ gpointer okey, value;
+ char *path, *key64, *pass64;
+ int len, state, save;
- /* now remove it from our session hash */
- g_hash_table_remove (passwords, key);
- g_free (okey);
- g_free (value);
- }
+ if (!g_hash_table_lookup_extended (passwords, key, &okey, &value))
+ return;
+
+ /* add it to the on-disk cache of passwords */
+ len = strlen (okey);
+ key64 = g_malloc0 ((len + 2) * 4 / 3 + 1);
+ state = save = 0;
+ base64_encode_close (okey, len, FALSE, key64, &state, &save);
+ path = g_strdup_printf ("/Passwords/%s/%s", component_name, key64);
+ g_free (key64);
+
+ len = strlen (value);
+ pass64 = g_malloc0 ((len + 2) * 4 / 3 + 1);
+ state = save = 0;
+ base64_encode_close (value, len, FALSE, pass64, &state, &save);
+
+ bonobo_config_set_string (db, path, pass64, NULL);
+ g_free (path);
+ g_free (pass64);
+
+ /* now remove it from our session hash */
+ g_hash_table_remove (passwords, key);
+ g_free (okey);
+ g_free (value);
}
/**
@@ -230,29 +229,34 @@ char *
e_passwords_get_password (const char *key)
{
char *passwd = g_hash_table_lookup (passwords, key);
- if (!passwd) {
- char *path, *key64;
- int len, state, save;
+ char *path, *key64;
+ int len, state, save;
+ CORBA_Environment ev;
+
+ if (passwd)
+ return g_strdup (passwd);
- /* not part of the session hash, look it up in the on disk db */
- len = strlen (key);
- key64 = g_malloc0 ((len + 2) * 4 / 3 + 1);
- state = save = 0;
- base64_encode_close ((char*)key, len, FALSE, key64, &state, &save);
- path = g_strdup_printf ("/Passwords/%s/%s", component_name, key64);
- g_free (key64);
+ /* not part of the session hash, look it up in the on disk db */
+ len = strlen (key);
+ key64 = g_malloc0 ((len + 2) * 4 / 3 + 1);
+ state = save = 0;
+ base64_encode_close ((char*)key, len, FALSE, key64, &state, &save);
+ path = g_strdup_printf ("/Passwords/%s/%s", component_name, key64);
+ g_free (key64);
- passwd = bonobo_config_get_string (db, path, NULL);
+ /* We need to pass an ev to bonobo-conf, or it will emit a
+ * g_warning if the data isn't found.
+ */
+ CORBA_exception_init (&ev);
+ passwd = bonobo_config_get_string (db, path, &ev);
+ CORBA_exception_free (&ev);
- g_free (path);
+ g_free (path);
- if (passwd)
- return decode_base64 (passwd);
- else
- return NULL;
- }
+ if (passwd)
+ return decode_base64 (passwd);
else
- return g_strdup (passwd);
+ return NULL;
}
/**
@@ -266,17 +270,19 @@ e_passwords_get_password (const char *key)
void
e_passwords_add_password (const char *key, const char *passwd)
{
- if (key && passwd) {
- gpointer okey, value;
-
- if (g_hash_table_lookup_extended (passwords, key, &okey, &value)) {
- g_hash_table_remove (passwords, key);
- g_free (okey);
- g_free (value);
- }
+ gpointer okey, value;
- g_hash_table_insert (passwords, g_strdup (key), g_strdup (passwd));
+ /* FIXME: shouldn't this be g_return_if_fail? */
+ if (!key || !passwd)
+ return;
+
+ if (g_hash_table_lookup_extended (passwords, key, &okey, &value)) {
+ g_hash_table_remove (passwords, key);
+ g_free (okey);
+ g_free (value);
}
+
+ g_hash_table_insert (passwords, g_strdup (key), g_strdup (passwd));
}