Attachment 82211 Details for Bug 66728 – [PATCH 1/2] launch-helper: fix error code parsing

[patch] [PATCH 1/2] launch-helper: fix error code parsing

0001-launch-helper-fix-error-code-parsing.patch (text/plain), 2.90 KB, created by Chengwei Yang on 2013-07-09 07:55:22 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chengwei Yang

Created: 2013-07-09 07:55:22 UTC

Size: 2.90 KB

patch

obsolete

>From ee241ae190ea1a1af70ea3f03f4d914d7e914f0d Mon Sep 17 00:00:00 2001
>From: Chengwei Yang <chengwei.yang@intel.com>
>Date: Tue, 9 Jul 2013 15:37:58 +0800
>Subject: [PATCH 1/2] launch-helper: fix error code parsing
>
>Bug: https://bugs.freedesktop.org/show_bug.cgi?id=66728
>Signed-off-by: Chengwei Yang <chengwei.yang@intel.com>
>---
> bus/activation-helper-bin.c |    5 ++++-
> bus/activation-helper.c     |    2 +-
> bus/activation.c            |    5 +++++
> 3 files changed, 10 insertions(+), 2 deletions(-)
>
>diff --git a/bus/activation-helper-bin.c b/bus/activation-helper-bin.c
>index a360acc..f5f16d2 100644
>--- a/bus/activation-helper-bin.c
>+++ b/bus/activation-helper-bin.c
>@@ -45,6 +45,9 @@ convert_error_to_exit_code (DBusError *error)
>     return BUS_SPAWN_EXIT_CODE_SETUP_FAILED;
> 
>   if (dbus_error_has_name (error, DBUS_ERROR_SPAWN_SERVICE_INVALID))
>+    return BUS_SPAWN_EXIT_CODE_NAME_INVALID;
>+
>+  if (dbus_error_has_name (error, DBUS_ERROR_SPAWN_SERVICE_NOT_FOUND))
>     return BUS_SPAWN_EXIT_CODE_SERVICE_NOT_FOUND;
> 
>   if (dbus_error_has_name (error, DBUS_ERROR_SPAWN_PERMISSIONS_INVALID))
>@@ -65,7 +68,7 @@ convert_error_to_exit_code (DBusError *error)
>   /* should we assert? */
>   fprintf(stderr, "%s: %s\n", error->name, error->message);
>   
>-  return BUS_SPAWN_EXIT_CODE_SETUP_FAILED;
>+  return BUS_SPAWN_EXIT_CODE_GENERIC_FAILURE;
> }
> 
> int
>diff --git a/bus/activation-helper.c b/bus/activation-helper.c
>index 8d7ae36..3eb9bfc 100644
>--- a/bus/activation-helper.c
>+++ b/bus/activation-helper.c
>@@ -371,7 +371,7 @@ check_bus_name (const char *bus_name,
>   _dbus_string_init_const (&str, bus_name);
>   if (!_dbus_validate_bus_name (&str, 0, _dbus_string_get_length (&str)))
>     {
>-      dbus_set_error (error, DBUS_ERROR_SPAWN_SERVICE_NOT_FOUND,
>+      dbus_set_error (error, DBUS_ERROR_SPAWN_SERVICE_INVALID,
>                       "bus name '%s' is not a valid bus name\n",
>                       bus_name);
>       return FALSE;
>diff --git a/bus/activation.c b/bus/activation.c
>index 4269440..531bb34 100644
>--- a/bus/activation.c
>+++ b/bus/activation.c
>@@ -1289,6 +1289,10 @@ handle_servicehelper_exit_error (int        exit_code,
> {
>   switch (exit_code)
>     {
>+    case BUS_SPAWN_EXIT_CODE_CONFIG_INVALID:
>+      dbus_set_error (error, DBUS_ERROR_SPAWN_CONFIG_INVALID,
>+		      "Could not get user from config file");
>+      break;
>     case BUS_SPAWN_EXIT_CODE_NO_MEMORY:
>       dbus_set_error (error, DBUS_ERROR_NO_MEMORY,
>                       "Launcher could not run (out of memory)");
>@@ -1325,6 +1329,7 @@ handle_servicehelper_exit_error (int        exit_code,
>       dbus_set_error (error, DBUS_ERROR_SPAWN_CHILD_SIGNALED,
>                       "Launched child was signaled, it probably crashed");
>       break;
>+    case BUS_SPAWN_EXIT_CODE_GENERIC_FAILURE:
>     default:
>       dbus_set_error (error, DBUS_ERROR_SPAWN_CHILD_EXITED,
>                       "Launch helper exited with unknown return code %i", exit_code);
>-- 
>1.7.9.5
>

Actions: View | Splinter Review

Attachments on bug 66728: 82211 | 82212 | 87280 | 87311 | 87589 | 88541

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.