fixed compiler warning about uid_t handling

This fixes the same problem with uid_t as we had with HAL some time ago
on 64bit architectures in PolicyKit. This time I removed the useless check:

   uid == ((unsigned long) -1)

because this is always false on 64bit (comparison is always false due to
limited range of data type) and because the DBusError from the
dbus_bus_get_unix_user() call is set if the function returns DBUS_UID_UNSET
so we need only to check if the error is set.

diff --git a/polkit-dbus/polkit-dbus.c b/polkit-dbus/polkit-dbus.c
index ce6573c..7d35a0d 100644 (file)
--- a/polkit-dbus/polkit-dbus.c
+++ b/polkit-dbus/polkit-dbus.c
@@ -389,7 +389,7 @@ polkit_caller_new_from_dbus_name (DBusConnection *con, const char *dbus_name, DB
         session = NULL;
 
        uid = dbus_bus_get_unix_user (con, dbus_name, error);
-       if (uid == ((unsigned long) -1) || dbus_error_is_set (error)) {
+       if (dbus_error_is_set (error)) {
                g_warning ("Could not get uid for connection: %s %s", error->name, error->message);
                goto out;
        }