Assert that DBUS_SERVER_LISTEN_ADDRESS_ALREADY_USED does not set error

The only place this is set in practice is in dbus-server-win.c, which
does not set the error. If it did, dbus_server_listen() would leak it.

Signed-off-by: Simon McVittie <smcv@collabora.com>
Reviewed-by: Philip Withnall <withnall@endlessm.com>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=107194

diff --git a/dbus/dbus-server.c b/dbus/dbus-server.c
index d91df8325cd70ed9557d7f3dc90dc0804b773caa..f26a8f7c1548b1ae3b3a3847beffcf36cb3fee61 100644 (file)
--- a/dbus/dbus-server.c
+++ b/dbus/dbus-server.c
@@ -595,6 +595,7 @@ dbus_server_listen (const char     *address,
           else if (result == DBUS_SERVER_LISTEN_ADDRESS_ALREADY_USED)
             {
               _dbus_assert (server == NULL);
+              _DBUS_ASSERT_ERROR_IS_CLEAR (&tmp_error);
               dbus_set_error (error,
                        DBUS_ERROR_ADDRESS_IN_USE,
                        "Address '%s' already used",