Attachment 643647 Details for Bug 810040 – fprintd: request name unconditionally at startup

[patch] fprintd: request name unconditionally at startup

foo.patch (text/plain), 1.91 KB, created by Ray Strode [halfline] on 2012-11-12 17:45:46 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Ray Strode [halfline]

Created: 2012-11-12 17:45:46 UTC

Size: 1.91 KB

patch

obsolete

>From 60b009a25ea38309407b220f8106ca24648bc242 Mon Sep 17 00:00:00 2001
>From: Ray Strode <rstrode@redhat.com>
>Date: Mon, 12 Nov 2012 12:36:47 -0500
>Subject: [PATCH] main: request name unconditionally at startup
>
>the dbus activation machinery depends on daemons taking a name on
>the bus to complete activation without timeouts.
>
>fprintd fails prematurely if there is USB bus (as found in some
>virtual machine setups).
>
>This commit moves the bus-name-acquisition code to happen before
>the fail-from-no-usb-bus code to keep callers from timing out
>when activating fprintd.
>---
> src/main.c | 22 +++++++++++-----------
> 1 file changed, 11 insertions(+), 11 deletions(-)
>
>diff --git a/src/main.c b/src/main.c
>index 598a38f..f667239 100644
>--- a/src/main.c
>+++ b/src/main.c
>@@ -316,6 +316,17 @@ int main(int argc, char **argv)
> 		g_log_set_always_fatal (fatal_mask);
> 	}
> 
>+	if (!org_freedesktop_DBus_request_name(driver_proxy, FPRINT_SERVICE_NAME,
>+			0, &request_name_ret, &error)) {
>+		g_warning("Failed to get name: %s", error->message);
>+		return 1;
>+	}
>+
>+	if (request_name_ret != DBUS_REQUEST_NAME_REPLY_PRIMARY_OWNER) {
>+		g_warning ("Got result code %u from requesting name", request_name_ret);
>+		return 1;
>+	}
>+
> 	/* Load the configuration file,
> 	 * and the default storage plugin */
> 	if (!load_conf())
>@@ -352,17 +363,6 @@ int main(int argc, char **argv)
> 	driver_proxy = dbus_g_proxy_new_for_name(fprintd_dbus_conn,
> 		DBUS_SERVICE_DBUS, DBUS_PATH_DBUS, DBUS_INTERFACE_DBUS);
> 
>-	if (!org_freedesktop_DBus_request_name(driver_proxy, FPRINT_SERVICE_NAME,
>-			0, &request_name_ret, &error)) {
>-		g_warning("Failed to get name: %s", error->message);
>-		return 1;
>-	}
>-
>-	if (request_name_ret != DBUS_REQUEST_NAME_REPLY_PRIMARY_OWNER) {
>-		g_warning ("Got result code %u from requesting name", request_name_ret);
>-		return 1;
>-	}
>-
> 	g_message("D-Bus service launched with name: %s", FPRINT_SERVICE_NAME);
> 
> 	g_message("entering main loop");
>-- 
>1.8.0
>

Actions: View | Diff

Attachments on bug 810040: 633460 | 643647 | 643659