* dbus/dbus-tranport.c (auth_via_default_rules): made the verbose spam cross-platform

diff --git a/ChangeLog b/ChangeLog
index 0fc6e9e4de016fd281fc67c77bea2d345b64a6be..58cb591f1e7124cf2e4e86a3bf63d244b6aac3e9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2007-06-21  Havoc Pennington  <hp@redhat.com>
+
+       * dbus/dbus-tranport.c (auth_via_default_rules): made the verbose spam
+       cross-platform
+
 2007-06-21  Havoc Pennington  <hp@redhat.com>
 
        * dbus/dbus-watch.c (dbus_watch_get_fd): 1) its behavior should

diff --git a/dbus/dbus-transport.c b/dbus/dbus-transport.c
index d78727cf8897f35e97028bd7158d3fe3ff6c8aa1..1d8a5e6c9cfbcbbc68f088732ad580596feb1f22 100644 (file)
--- a/dbus/dbus-transport.c
+++ b/dbus/dbus-transport.c
@@ -619,21 +619,31 @@ auth_via_default_rules (DBusTransport *transport)
       _dbus_credentials_same_user (our_identity,
                                    auth_identity))
     {
-      /* FIXME the verbose spam here is unix-specific */                  
-      _dbus_verbose ("Client authorized as UID "DBUS_UID_FORMAT
-                     " matching our UID "DBUS_UID_FORMAT"\n",
-                     _dbus_credentials_get_unix_uid(auth_identity),
-                     _dbus_credentials_get_unix_uid(our_identity));
+      if (_dbus_credentials_include(our_identity,DBUS_CREDENTIAL_WINDOWS_SID))
+          _dbus_verbose ("Client authorized as SID '%s'"
+                         "matching our SID '%s'\n",
+                         _dbus_credentials_get_windows_sid(auth_identity),
+                         _dbus_credentials_get_windows_sid(our_identity));
+      else
+          _dbus_verbose ("Client authorized as UID "DBUS_UID_FORMAT
+                         " matching our UID "DBUS_UID_FORMAT"\n",
+                         _dbus_credentials_get_unix_uid(auth_identity),
+                         _dbus_credentials_get_unix_uid(our_identity));
       /* We have authenticated! */
       allow = TRUE;
     }
   else
     {
-      /* FIXME the verbose spam here is unix-specific */
-      _dbus_verbose ("Client authorized as UID "DBUS_UID_FORMAT
-                     " but our UID is "DBUS_UID_FORMAT", disconnecting\n",
-                     _dbus_credentials_get_unix_uid(our_identity),
-                     _dbus_credentials_get_unix_uid(our_identity));
+      if (_dbus_credentials_include(our_identity,DBUS_CREDENTIAL_WINDOWS_SID))
+          _dbus_verbose ("Client authorized as SID '%s'"
+                         " but our SID is '%s', disconnecting\n",
+                         _dbus_credentials_get_windows_sid(our_identity),
+                         _dbus_credentials_get_windows_sid(our_identity));
+      else
+          _dbus_verbose ("Client authorized as UID "DBUS_UID_FORMAT
+                         " but our UID is "DBUS_UID_FORMAT", disconnecting\n",
+                         _dbus_credentials_get_unix_uid(our_identity),
+                         _dbus_credentials_get_unix_uid(our_identity));
       _dbus_transport_disconnect (transport);
       allow = FALSE;
     }