From ae94f92b5d5692dd4f4084ee9b60e3fe6e44fac5 Mon Sep 17 00:00:00 2001 From: Havoc Pennington Date: Sat, 5 Mar 2005 18:38:54 +0000 Subject: 2005-03-05 Havoc Pennington * dbus/dbus-sysdeps.c: (pseudorandom_generate_random_bytes_buffer): fix to have no return value (_dbus_generate_random_bytes_buffer): fix return value * dbus/dbus-sysdeps-util.c: s/GETPWNAME/GETPWNAM/ so configure checks actually work, from Tom Parker --- ChangeLog | 10 ++++++++++ dbus/dbus-sysdeps-util.c | 6 +++--- dbus/dbus-sysdeps.c | 18 +++++++++++------- 3 files changed, 24 insertions(+), 10 deletions(-) diff --git a/ChangeLog b/ChangeLog index 06a69d2c..e3392d07 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,13 @@ +2005-03-05 Havoc Pennington + + * dbus/dbus-sysdeps.c: + (pseudorandom_generate_random_bytes_buffer): fix to have no return + value + (_dbus_generate_random_bytes_buffer): fix return value + + * dbus/dbus-sysdeps-util.c: s/GETPWNAME/GETPWNAM/ so configure + checks actually work, from Tom Parker + 2005-03-01 Colin Walters * test/glib/test-dbus-glib.c (lose, lose_gerror): Utility diff --git a/dbus/dbus-sysdeps-util.c b/dbus/dbus-sysdeps-util.c index 52298f81..5c9921e0 100644 --- a/dbus/dbus-sysdeps-util.c +++ b/dbus/dbus-sysdeps-util.c @@ -590,7 +590,7 @@ fill_group_info (DBusGroupInfo *info, * to add more configure checks. */ -#if defined (HAVE_POSIX_GETPWNAME_R) || defined (HAVE_NONPOSIX_GETPWNAME_R) +#if defined (HAVE_POSIX_GETPWNAM_R) || defined (HAVE_NONPOSIX_GETPWNAM_R) { struct group *g; int result; @@ -598,7 +598,7 @@ fill_group_info (DBusGroupInfo *info, struct group g_str; g = NULL; -#ifdef HAVE_POSIX_GETPWNAME_R +#ifdef HAVE_POSIX_GETPWNAM_R if (group_c_str) result = getgrnam_r (group_c_str, &g_str, buf, sizeof (buf), @@ -609,7 +609,7 @@ fill_group_info (DBusGroupInfo *info, #else p = getgrnam_r (group_c_str, &g_str, buf, sizeof (buf)); result = 0; -#endif /* !HAVE_POSIX_GETPWNAME_R */ +#endif /* !HAVE_POSIX_GETPWNAM_R */ if (result == 0 && g == &g_str) { return fill_user_info_from_group (g, info, error); diff --git a/dbus/dbus-sysdeps.c b/dbus/dbus-sysdeps.c index 830e26f1..96d51bed 100644 --- a/dbus/dbus-sysdeps.c +++ b/dbus/dbus-sysdeps.c @@ -1434,7 +1434,7 @@ fill_user_info (DBusUserInfo *info, * checks */ -#if defined (HAVE_POSIX_GETPWNAME_R) || defined (HAVE_NONPOSIX_GETPWNAME_R) +#if defined (HAVE_POSIX_GETPWNAM_R) || defined (HAVE_NONPOSIX_GETPWNAM_R) { struct passwd *p; int result; @@ -1442,8 +1442,8 @@ fill_user_info (DBusUserInfo *info, struct passwd p_str; p = NULL; -#ifdef HAVE_POSIX_GETPWNAME_R - if (uid >= 0) +#ifdef HAVE_POSIX_GETPWNAM_R + if (uid != DBUS_UID_UNSET) result = getpwuid_r (uid, &p_str, buf, sizeof (buf), &p); else @@ -1455,7 +1455,7 @@ fill_user_info (DBusUserInfo *info, else p = getpwnam_r (username_c, &p_str, buf, sizeof (buf)); result = 0; -#endif /* !HAVE_POSIX_GETPWNAME_R */ +#endif /* !HAVE_POSIX_GETPWNAM_R */ if (result == 0 && p == &p_str) { if (!fill_user_info_from_passwd (p, info, error)) @@ -2327,7 +2327,7 @@ _dbus_concat_dir_and_file (DBusString *dir, _dbus_string_get_length (dir)); } -static dbus_bool_t +static void pseudorandom_generate_random_bytes_buffer (char *buffer, int n_bytes) { @@ -2388,12 +2388,16 @@ _dbus_generate_random_bytes_buffer (char *buffer, DBusString str; if (!_dbus_string_init (&str)) - return pseudorandom_generate_random_bytes_buffer (buffer, n_bytes); + { + pseudorandom_generate_random_bytes_buffer (buffer, n_bytes); + return; + } if (!_dbus_generate_random_bytes (&str, n_bytes)) { _dbus_string_free (&str); - return pseudorandom_generate_random_bytes_buffer (buffer, n_bytes); + pseudorandom_generate_random_bytes_buffer (buffer, n_bytes); + return; } _dbus_string_copy_to_buffer (&str, buffer, n_bytes); -- cgit