1 /* -*- mode: C; c-file-style: "gnu" -*- */
2 /* dbus-keyring.c Store secret cookies in your homedir
4 * Copyright (C) 2003 Red Hat Inc.
6 * Licensed under the Academic Free License version 1.2
8 * This program is free software; you can redistribute it and/or modify
9 * it under the terms of the GNU General Public License as published by
10 * the Free Software Foundation; either version 2 of the License, or
11 * (at your option) any later version.
13 * This program is distributed in the hope that it will be useful,
14 * but WITHOUT ANY WARRANTY; without even the implied warranty of
15 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16 * GNU General Public License for more details.
18 * You should have received a copy of the GNU General Public License
19 * along with this program; if not, write to the Free Software
20 * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
24 #include "dbus-keyring.h"
25 #include "dbus-userdb.h"
26 #include <dbus/dbus-string.h>
27 #include <dbus/dbus-list.h>
28 #include <dbus/dbus-sysdeps.h>
31 * @defgroup DBusKeyring keyring class
32 * @ingroup DBusInternals
33 * @brief DBusKeyring data structure
35 * Types and functions related to DBusKeyring. DBusKeyring is intended
36 * to manage cookies used to authenticate clients to servers. This is
37 * essentially the "verify that client can read the user's homedir"
38 * authentication mechanism. Both client and server must have access
41 * The secret keys are not kept in locked memory, and are written to a
42 * file in the user's homedir. However they are transient (only used
43 * by a single server instance for a fixed period of time, then
44 * discarded). Also, the keys are not sent over the wire.
46 * @todo there's a memory leak on some codepath in here, I saw it once
47 * when running make check - probably some specific initial cookies
48 * present in the cookie file, then depending on what we do with them.
52 * @defgroup DBusKeyringInternals DBusKeyring implementation details
53 * @ingroup DBusInternals
54 * @brief DBusKeyring implementation details
56 * The guts of DBusKeyring.
61 /** The maximum age of a key before we create a new key to use in
62 * challenges. This isn't super-reliably enforced, since system
63 * clocks can change or be wrong, but we make a best effort to only
64 * use keys for a short time.
66 #define NEW_KEY_TIMEOUT_SECONDS (60*5)
68 * The time after which we drop a key from the secrets file.
69 * The EXPIRE_KEYS_TIMEOUT_SECONDS - NEW_KEY_TIMEOUT_SECONDS is the minimum
70 * time window a client has to complete authentication.
72 #define EXPIRE_KEYS_TIMEOUT_SECONDS (NEW_KEY_TIMEOUT_SECONDS + (60*2))
74 * The maximum amount of time a key can be in the future.
76 #define MAX_TIME_TRAVEL_SECONDS (60*5)
79 * Maximum number of keys in the keyring before
80 * we just ignore the rest
82 #ifdef DBUS_BUILD_TESTS
83 #define MAX_KEYS_IN_FILE 10
85 #define MAX_KEYS_IN_FILE 256
90 dbus_int32_t id; /**< identifier used to refer to the key */
92 long creation_time; /**< when the key was generated,
93 * as unix timestamp. signed long
94 * matches struct timeval.
97 DBusString secret; /**< the actual key */
102 * @brief Internals of DBusKeyring.
104 * DBusKeyring internals. DBusKeyring is an opaque object, it must be
105 * used via accessor functions.
109 int refcount; /**< Reference count */
110 DBusString username; /**< Username keyring is for */
111 DBusString directory; /**< Directory the below two items are inside */
112 DBusString filename; /**< Keyring filename */
113 DBusString filename_lock; /**< Name of lockfile */
114 DBusKey *keys; /**< Keys loaded from the file */
115 int n_keys; /**< Number of keys */
119 _dbus_keyring_new (void)
121 DBusKeyring *keyring;
123 keyring = dbus_new0 (DBusKeyring, 1);
127 if (!_dbus_string_init (&keyring->directory))
130 if (!_dbus_string_init (&keyring->filename))
133 if (!_dbus_string_init (&keyring->filename_lock))
136 if (!_dbus_string_init (&keyring->username))
139 keyring->refcount = 1;
140 keyring->keys = NULL;
146 _dbus_string_free (&keyring->filename_lock);
148 _dbus_string_free (&keyring->filename);
150 _dbus_string_free (&keyring->directory);
158 free_keys (DBusKey *keys,
163 /* should be safe for args NULL, 0 */
168 _dbus_string_free (&keys[i].secret);
175 /* Our locking scheme is highly unreliable. However, there is
176 * unfortunately no reliable locking scheme in user home directories;
177 * between bugs in Linux NFS, people using Tru64 or other total crap
178 * NFS, AFS, random-file-system-of-the-week, and so forth, fcntl() in
179 * homedirs simply generates tons of bug reports. This has been
180 * learned through hard experience with GConf, unfortunately.
182 * This bad hack might work better for the kind of lock we have here,
183 * which we don't expect to hold for any length of time. Crashing
184 * while we hold it should be unlikely, and timing out such that we
185 * delete a stale lock should also be unlikely except when the
186 * filesystem is running really slowly. Stuff might break in corner
187 * cases but as long as it's not a security-level breakage it should
191 /** Maximum number of timeouts waiting for lock before we decide it's stale */
192 #define MAX_LOCK_TIMEOUTS 32
193 /** Length of each timeout while waiting for a lock */
194 #define LOCK_TIMEOUT_MILLISECONDS 250
197 _dbus_keyring_lock (DBusKeyring *keyring)
202 while (n_timeouts < MAX_LOCK_TIMEOUTS)
206 dbus_error_init (&error);
207 if (_dbus_create_file_exclusively (&keyring->filename_lock,
211 _dbus_verbose ("Did not get lock file, sleeping %d milliseconds (%s)\n",
212 LOCK_TIMEOUT_MILLISECONDS, error.message);
213 dbus_error_free (&error);
215 _dbus_sleep_milliseconds (LOCK_TIMEOUT_MILLISECONDS);
220 if (n_timeouts == MAX_LOCK_TIMEOUTS)
224 _dbus_verbose ("Lock file timed out %d times, assuming stale\n",
227 dbus_error_init (&error);
229 if (!_dbus_delete_file (&keyring->filename_lock, &error))
231 _dbus_verbose ("Couldn't delete old lock file: %s\n",
233 dbus_error_free (&error);
237 if (!_dbus_create_file_exclusively (&keyring->filename_lock,
240 _dbus_verbose ("Couldn't create lock file after deleting stale one: %s\n",
242 dbus_error_free (&error);
251 _dbus_keyring_unlock (DBusKeyring *keyring)
254 dbus_error_init (&error);
255 if (!_dbus_delete_file (&keyring->filename_lock, &error))
257 _dbus_warn ("Failed to delete lock file: %s\n",
259 dbus_error_free (&error);
264 find_key_by_id (DBusKey *keys,
273 if (keys[i].id == id)
283 add_new_key (DBusKey **keys_p,
290 unsigned long timestamp;
291 const unsigned char *s;
296 _DBUS_ASSERT_ERROR_IS_CLEAR (error);
298 if (!_dbus_string_init (&bytes))
300 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
308 /* Generate an integer ID and then the actual key. */
311 if (!_dbus_generate_random_bytes (&bytes, 4))
313 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
317 s = (const unsigned char*) _dbus_string_get_const_data (&bytes);
319 id = s[0] | (s[1] << 8) | (s[2] << 16) | (s[3] << 24);
322 _dbus_assert (id >= 0);
324 if (find_key_by_id (keys, n_keys, id) != NULL)
326 _dbus_string_set_length (&bytes, 0);
327 _dbus_verbose ("Key ID %d already existed, trying another one\n",
332 _dbus_verbose ("Creating key with ID %d\n", id);
334 #define KEY_LENGTH_BYTES 24
335 _dbus_string_set_length (&bytes, 0);
336 if (!_dbus_generate_random_bytes (&bytes, KEY_LENGTH_BYTES))
338 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
342 new = dbus_realloc (keys, sizeof (DBusKey) * (n_keys + 1));
345 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
350 *keys_p = keys; /* otherwise *keys_p ends up invalid */
353 if (!_dbus_string_init (&keys[n_keys-1].secret))
355 n_keys -= 1; /* we don't want to free the one we didn't init */
356 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
360 _dbus_get_current_time (×tamp, NULL);
362 keys[n_keys-1].id = id;
363 keys[n_keys-1].creation_time = timestamp;
364 if (!_dbus_string_move (&bytes, 0,
365 &keys[n_keys-1].secret,
368 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
369 _dbus_string_free (&keys[n_keys-1].secret);
379 _dbus_string_free (&bytes);
384 * Reloads the keyring file, optionally adds one new key to the file,
385 * removes all expired keys from the file iff a key was added, then
386 * resaves the file. Stores the keys from the file in keyring->keys.
387 * Note that the file is only resaved (written to) if a key is added,
388 * this means that only servers ever write to the file and need to
389 * lock it, which avoids a lot of lock contention at login time and
392 * @param keyring the keyring
393 * @param add_new #TRUE to add a new key to the file, expire keys, and resave
394 * @param error return location for errors
395 * @returns #FALSE on failure
398 _dbus_keyring_reload (DBusKeyring *keyring,
405 dbus_bool_t have_lock;
412 _DBUS_ASSERT_ERROR_IS_CLEAR (error);
414 if (!_dbus_string_init (&contents))
416 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
420 if (!_dbus_string_init (&line))
422 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
423 _dbus_string_free (&contents);
432 _dbus_get_current_time (&now, NULL);
436 if (!_dbus_keyring_lock (keyring))
438 dbus_set_error (error, DBUS_ERROR_FAILED,
439 "Could not lock keyring file to add to it");
446 dbus_error_init (&tmp_error);
447 if (!_dbus_file_get_contents (&contents,
451 _dbus_verbose ("Failed to load keyring file: %s\n",
453 /* continue with empty keyring file, so we recreate it */
454 dbus_error_free (&tmp_error);
457 if (!_dbus_string_validate_ascii (&contents, 0,
458 _dbus_string_get_length (&contents)))
460 _dbus_warn ("Secret keyring file contains non-ASCII! Ignoring existing contents\n");
461 _dbus_string_set_length (&contents, 0);
464 /* FIXME this is badly inefficient for large keyring files
465 * (not that large keyring files exist outside of test suites)
467 while (_dbus_string_pop_line (&contents, &line))
476 /* Don't load more than the max. */
477 if (n_keys >= (add_new ? MAX_KEYS_IN_FILE : MAX_KEYS_IN_FILE - 1))
481 if (!_dbus_string_parse_int (&line, 0, &val, &next))
483 _dbus_verbose ("could not parse secret key ID at start of line\n");
487 if (val > _DBUS_INT_MAX || val < 0)
489 _dbus_verbose ("invalid secret key ID at start of line\n");
495 _dbus_string_skip_blank (&line, next, &next);
497 if (!_dbus_string_parse_int (&line, next, ×tamp, &next))
499 _dbus_verbose ("could not parse secret key timestamp\n");
504 (now + MAX_TIME_TRAVEL_SECONDS) < timestamp ||
505 (now - EXPIRE_KEYS_TIMEOUT_SECONDS) > timestamp)
507 _dbus_verbose ("dropping/ignoring %ld-seconds old key with timestamp %ld as current time is %ld\n",
508 now - timestamp, timestamp, now);
512 _dbus_string_skip_blank (&line, next, &next);
514 len = _dbus_string_get_length (&line);
516 if ((len - next) == 0)
518 _dbus_verbose ("no secret key after ID and timestamp\n");
522 /* We have all three parts */
523 new = dbus_realloc (keys, sizeof (DBusKey) * (n_keys + 1));
526 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
533 if (!_dbus_string_init (&keys[n_keys-1].secret))
535 n_keys -= 1; /* we don't want to free the one we didn't init */
536 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
540 keys[n_keys-1].id = id;
541 keys[n_keys-1].creation_time = timestamp;
542 if (!_dbus_string_hex_decode (&line, next,
543 &keys[n_keys-1].secret,
546 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
551 _dbus_verbose ("Successfully loaded %d existing keys\n",
556 if (!add_new_key (&keys, &n_keys, error))
558 _dbus_verbose ("Failed to generate new key: %s\n",
559 error ? "(unknown)" : error->message);
563 _dbus_string_set_length (&contents, 0);
568 if (!_dbus_string_append_int (&contents,
572 if (!_dbus_string_append_byte (&contents, ' '))
575 if (!_dbus_string_append_int (&contents,
576 keys[i].creation_time))
579 if (!_dbus_string_append_byte (&contents, ' '))
582 if (!_dbus_string_hex_encode (&keys[i].secret, 0,
584 _dbus_string_get_length (&contents)))
587 if (!_dbus_string_append_byte (&contents, '\n'))
594 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
598 if (!_dbus_string_save_to_file (&contents, &keyring->filename,
604 free_keys (keyring->keys, keyring->n_keys);
605 keyring->keys = keys;
606 keyring->n_keys = n_keys;
614 _dbus_keyring_unlock (keyring);
616 if (! ((retval == TRUE && (error == NULL || error->name == NULL)) ||
617 (retval == FALSE && (error == NULL || error->name != NULL))))
619 if (error && error->name)
620 _dbus_verbose ("error is %s: %s\n", error->name, error->message);
621 _dbus_warn ("returning %d but error pointer %p name %s\n",
622 retval, error, error->name ? error->name : "(none)");
623 _dbus_assert_not_reached ("didn't handle errors properly");
631 _dbus_string_zero (&keys[i].secret);
632 _dbus_string_free (&keys[i].secret);
639 _dbus_string_free (&contents);
640 _dbus_string_free (&line);
645 /** @} */ /* end of internals */
648 * @addtogroup DBusKeyring
654 * Increments reference count of the keyring
656 * @param keyring the keyring
659 _dbus_keyring_ref (DBusKeyring *keyring)
661 keyring->refcount += 1;
665 * Decrements refcount and finalizes if it reaches
668 * @param keyring the keyring
671 _dbus_keyring_unref (DBusKeyring *keyring)
673 keyring->refcount -= 1;
675 if (keyring->refcount == 0)
677 _dbus_string_free (&keyring->username);
678 _dbus_string_free (&keyring->filename);
679 _dbus_string_free (&keyring->filename_lock);
680 _dbus_string_free (&keyring->directory);
681 free_keys (keyring->keys, keyring->n_keys);
687 * Creates a new keyring that lives in the ~/.dbus-keyrings
688 * directory of the given user. If the username is #NULL,
689 * uses the user owning the current process.
691 * @param username username to get keyring for, or #NULL
692 * @param context which keyring to get
693 * @param error return location for errors
694 * @returns the keyring or #NULL on error
697 _dbus_keyring_new_homedir (const DBusString *username,
698 const DBusString *context,
702 DBusKeyring *keyring;
703 dbus_bool_t error_set;
707 _DBUS_ASSERT_ERROR_IS_CLEAR (error);
712 if (!_dbus_string_init (&homedir))
714 dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL);
718 _dbus_string_init_const (&dotdir, ".dbus-keyrings");
720 if (username == NULL)
722 const DBusString *const_homedir;
724 if (!_dbus_username_from_current_process (&username) ||
725 !_dbus_homedir_from_current_process (&const_homedir))
728 if (!_dbus_string_copy (const_homedir, 0,
734 if (!_dbus_homedir_from_username (username, &homedir))
738 #ifdef DBUS_BUILD_TESTS
740 const char *override;
742 override = _dbus_getenv ("DBUS_TEST_HOMEDIR");
743 if (override != NULL && *override != '\0')
745 _dbus_string_set_length (&homedir, 0);
746 if (!_dbus_string_append (&homedir, override))
747 _dbus_assert_not_reached ("no memory");
749 _dbus_verbose ("Using fake homedir for testing: %s\n",
750 _dbus_string_get_const_data (&homedir));
754 static dbus_bool_t already_warned = FALSE;
757 _dbus_warn ("Using your real home directory for testing, set DBUS_TEST_HOMEDIR to avoid\n");
758 already_warned = TRUE;
764 _dbus_assert (username != NULL);
766 keyring = _dbus_keyring_new ();
770 /* should have been validated already, but paranoia check here */
771 if (!_dbus_keyring_validate_context (context))
774 dbus_set_error_const (error,
776 "Invalid context in keyring creation");
780 if (!_dbus_string_copy (username, 0,
781 &keyring->username, 0))
784 if (!_dbus_string_copy (&homedir, 0,
785 &keyring->directory, 0))
788 if (!_dbus_concat_dir_and_file (&keyring->directory,
792 if (!_dbus_string_copy (&keyring->directory, 0,
793 &keyring->filename, 0))
796 if (!_dbus_concat_dir_and_file (&keyring->filename,
800 if (!_dbus_string_copy (&keyring->filename, 0,
801 &keyring->filename_lock, 0))
804 if (!_dbus_string_append (&keyring->filename_lock, ".lock"))
807 dbus_error_init (&tmp_error);
808 if (!_dbus_keyring_reload (keyring, FALSE, &tmp_error))
810 _dbus_verbose ("didn't load an existing keyring: %s\n",
812 dbus_error_free (&tmp_error);
815 /* We don't fail fatally if we can't create the directory,
816 * but the keyring will probably always be empty
817 * unless someone else manages to create it
819 dbus_error_init (&tmp_error);
820 if (!_dbus_create_directory (&keyring->directory,
823 _dbus_verbose ("Creating keyring directory: %s\n",
825 dbus_error_free (&tmp_error);
828 _dbus_string_free (&homedir);
834 dbus_set_error_const (error,
835 DBUS_ERROR_NO_MEMORY,
838 _dbus_keyring_unref (keyring);
839 _dbus_string_free (&homedir);
845 * Checks whether the context is a valid context.
846 * Contexts that might cause confusion when used
847 * in filenames are not allowed (contexts can't
848 * start with a dot or contain dir separators).
850 * @todo this is the most inefficient implementation
853 * @param context the context
854 * @returns #TRUE if valid
857 _dbus_keyring_validate_context (const DBusString *context)
859 if (_dbus_string_get_length (context) == 0)
861 _dbus_verbose ("context is zero-length\n");
865 if (!_dbus_string_validate_ascii (context, 0,
866 _dbus_string_get_length (context)))
868 _dbus_verbose ("context not valid ascii\n");
872 /* no directory separators */
873 if (_dbus_string_find (context, 0, "/", NULL))
875 _dbus_verbose ("context contains a slash\n");
879 if (_dbus_string_find (context, 0, "\\", NULL))
881 _dbus_verbose ("context contains a backslash\n");
885 /* prevent attempts to use dotfiles or ".." or ".lock"
886 * all of which might allow some kind of attack
888 if (_dbus_string_find (context, 0, ".", NULL))
890 _dbus_verbose ("context contains a dot\n");
894 /* no spaces/tabs, those are used for separators in the protocol */
895 if (_dbus_string_find_blank (context, 0, NULL))
897 _dbus_verbose ("context contains a blank\n");
901 if (_dbus_string_find (context, 0, "\n", NULL))
903 _dbus_verbose ("context contains a newline\n");
907 if (_dbus_string_find (context, 0, "\r", NULL))
909 _dbus_verbose ("context contains a carriage return\n");
917 find_recent_key (DBusKeyring *keyring)
920 long tv_sec, tv_usec;
922 _dbus_get_current_time (&tv_sec, &tv_usec);
925 while (i < keyring->n_keys)
927 DBusKey *key = &keyring->keys[i];
929 _dbus_verbose ("Key %d is %ld seconds old\n",
930 i, tv_sec - key->creation_time);
932 if ((tv_sec - NEW_KEY_TIMEOUT_SECONDS) < key->creation_time)
942 * Gets a recent key to use for authentication.
943 * If no recent key exists, creates one. Returns
944 * the key ID. If a key can't be written to the keyring
945 * file so no recent key can be created, returns -1.
946 * All valid keys are > 0.
948 * @param keyring the keyring
949 * @param error error on failure
950 * @returns key ID to use for auth, or -1 on failure
953 _dbus_keyring_get_best_key (DBusKeyring *keyring,
958 _DBUS_ASSERT_ERROR_IS_CLEAR (error);
960 key = find_recent_key (keyring);
964 /* All our keys are too old, or we've never loaded the
965 * keyring. Create a new one.
967 if (!_dbus_keyring_reload (keyring, TRUE,
971 key = find_recent_key (keyring);
976 dbus_set_error_const (error,
978 "No recent-enough key found in keyring, and unable to create a new key");
984 * Checks whether the keyring is for the given username.
986 * @param keyring the keyring
987 * @param username the username to check
989 * @returns #TRUE if the keyring belongs to the given user
992 _dbus_keyring_is_for_user (DBusKeyring *keyring,
993 const DBusString *username)
995 return _dbus_string_equal (&keyring->username,
1000 * Gets the hex-encoded secret key for the given ID.
1001 * Returns #FALSE if not enough memory. Returns #TRUE
1002 * but empty key on any other error such as unknown
1005 * @param keyring the keyring
1006 * @param key_id the key ID
1007 * @param hex_key string to append hex-encoded key to
1008 * @returns #TRUE if we had enough memory
1011 _dbus_keyring_get_hex_key (DBusKeyring *keyring,
1013 DBusString *hex_key)
1017 key = find_key_by_id (keyring->keys,
1021 return TRUE; /* had enough memory, so TRUE */
1023 return _dbus_string_hex_encode (&key->secret, 0,
1025 _dbus_string_get_length (hex_key));
1028 /** @} */ /* end of exposed API */
1030 #ifdef DBUS_BUILD_TESTS
1031 #include "dbus-test.h"
1035 _dbus_keyring_test (void)
1047 /* Context validation */
1049 _dbus_string_init_const (&context, "foo");
1050 _dbus_assert (_dbus_keyring_validate_context (&context));
1051 _dbus_string_init_const (&context, "org_freedesktop_blah");
1052 _dbus_assert (_dbus_keyring_validate_context (&context));
1054 _dbus_string_init_const (&context, "");
1055 _dbus_assert (!_dbus_keyring_validate_context (&context));
1056 _dbus_string_init_const (&context, ".foo");
1057 _dbus_assert (!_dbus_keyring_validate_context (&context));
1058 _dbus_string_init_const (&context, "bar.foo");
1059 _dbus_assert (!_dbus_keyring_validate_context (&context));
1060 _dbus_string_init_const (&context, "bar/foo");
1061 _dbus_assert (!_dbus_keyring_validate_context (&context));
1062 _dbus_string_init_const (&context, "bar\\foo");
1063 _dbus_assert (!_dbus_keyring_validate_context (&context));
1064 _dbus_string_init_const (&context, "foo\xfa\xf0");
1065 _dbus_assert (!_dbus_keyring_validate_context (&context));
1066 _dbus_string_init_const (&context, "foo\x80");
1067 _dbus_assert (!_dbus_keyring_validate_context (&context));
1068 _dbus_string_init_const (&context, "foo\x7f");
1069 _dbus_assert (_dbus_keyring_validate_context (&context));
1070 _dbus_string_init_const (&context, "foo bar");
1071 _dbus_assert (!_dbus_keyring_validate_context (&context));
1073 if (!_dbus_string_init (&context))
1074 _dbus_assert_not_reached ("no memory");
1075 if (!_dbus_string_append_byte (&context, '\0'))
1076 _dbus_assert_not_reached ("no memory");
1077 _dbus_assert (!_dbus_keyring_validate_context (&context));
1078 _dbus_string_free (&context);
1080 /* Now verify that if we create a key in keyring 1,
1081 * it is properly loaded in keyring 2
1084 _dbus_string_init_const (&context, "org_freedesktop_dbus_testsuite");
1085 dbus_error_init (&error);
1086 ring1 = _dbus_keyring_new_homedir (NULL, &context,
1088 _dbus_assert (ring1);
1089 _dbus_assert (error.name == NULL);
1091 id = _dbus_keyring_get_best_key (ring1, &error);
1094 fprintf (stderr, "Could not load keyring: %s\n", error.message);
1095 dbus_error_free (&error);
1099 ring2 = _dbus_keyring_new_homedir (NULL, &context, &error);
1100 _dbus_assert (ring2);
1101 _dbus_assert (error.name == NULL);
1103 if (ring1->n_keys != ring2->n_keys)
1105 fprintf (stderr, "Different number of keys in keyrings\n");
1109 /* We guarantee we load and save keeping keys in a fixed
1113 while (i < ring1->n_keys)
1115 if (ring1->keys[i].id != ring2->keys[i].id)
1117 fprintf (stderr, "Keyring 1 has first key ID %d and keyring 2 has %d\n",
1118 ring1->keys[i].id, ring2->keys[i].id);
1122 if (ring1->keys[i].creation_time != ring2->keys[i].creation_time)
1124 fprintf (stderr, "Keyring 1 has first key time %ld and keyring 2 has %ld\n",
1125 ring1->keys[i].creation_time, ring2->keys[i].creation_time);
1129 if (!_dbus_string_equal (&ring1->keys[i].secret,
1130 &ring2->keys[i].secret))
1132 fprintf (stderr, "Keyrings 1 and 2 have different secrets for same ID/timestamp\n");
1139 printf (" %d keys in test\n", ring1->n_keys);
1141 _dbus_keyring_unref (ring1);
1142 _dbus_keyring_unref (ring2);
1148 _dbus_keyring_unref (ring1);
1150 _dbus_keyring_unref (ring2);
1155 #endif /* DBUS_BUILD_TESTS */