diff options
| author | Ross Burton <ross.burton@intel.com> | 2013-02-26 12:37:10 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-02-28 23:15:48 +0000 |
| commit | 9f1c471ed47372dfff214fff7f73f861f9821491 (patch) | |
| tree | 7f2c1ba1b4d6e22989a292560dd132c8689338a7 | |
| parent | f436271a62687c73fed4d4f8437aa19b9f33b0e3 (diff) | |
| download | poky-9f1c471ed47372dfff214fff7f73f861f9821491.tar.gz | |
dbus: set correct address when using --address=systemd:
Patch taken from upstream 1.7 branch.
(From OE-Core rev: d7b8ae19464a3fe87822376b541b3d6865fe6115)
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | meta/recipes-core/dbus/dbus-1.6.8/systemd-address.patch | 187 | ||||
| -rw-r--r-- | meta/recipes-core/dbus/dbus.inc | 1 |
2 files changed, 188 insertions, 0 deletions
diff --git a/meta/recipes-core/dbus/dbus-1.6.8/systemd-address.patch b/meta/recipes-core/dbus/dbus-1.6.8/systemd-address.patch new file mode 100644 index 0000000000..ae1291c947 --- /dev/null +++ b/meta/recipes-core/dbus/dbus-1.6.8/systemd-address.patch | |||
| @@ -0,0 +1,187 @@ | |||
| 1 | Upstream-Status: Backport | ||
| 2 | Signed-off-by: Ross Burton <ross.burton@intel.com> | ||
| 3 | |||
| 4 | From d728fdc655f17031da3bb129ab2fd17dadf0fe3a Mon Sep 17 00:00:00 2001 | ||
| 5 | From: Simon Peeters <peeters.simon@gmail.com> | ||
| 6 | Date: Sun, 07 Oct 2012 14:59:30 +0000 | ||
| 7 | Subject: Set correct address when using --address=systemd: | ||
| 8 | |||
| 9 | When dbus gets launched through systemd, we need to create an address | ||
| 10 | string based on the sockets passed. | ||
| 11 | |||
| 12 | The _dbus_append_addres_from_socket() function is responsible for | ||
| 13 | extracting the address information from the file-descriptor and | ||
| 14 | formatting it in a dbus friendly way. | ||
| 15 | |||
| 16 | This fixes bus activation when running dbus under a systemd session. | ||
| 17 | |||
| 18 | https://bugs.freedesktop.org/show_bug.cgi?id=50962 | ||
| 19 | |||
| 20 | Signed-off-by: Simon Peeters <peeters.simon@gmail.com> | ||
| 21 | --- | ||
| 22 | diff --git a/dbus/dbus-server-unix.c b/dbus/dbus-server-unix.c | ||
| 23 | index 130f66e..d995240 100644 | ||
| 24 | --- a/dbus/dbus-server-unix.c | ||
| 25 | +++ b/dbus/dbus-server-unix.c | ||
| 26 | @@ -149,7 +149,7 @@ _dbus_server_listen_platform_specific (DBusAddressEntry *entry, | ||
| 27 | } | ||
| 28 | else if (strcmp (method, "systemd") == 0) | ||
| 29 | { | ||
| 30 | - int n, *fds; | ||
| 31 | + int i, n, *fds; | ||
| 32 | DBusString address; | ||
| 33 | |||
| 34 | n = _dbus_listen_systemd_sockets (&fds, error); | ||
| 35 | @@ -159,27 +159,39 @@ _dbus_server_listen_platform_specific (DBusAddressEntry *entry, | ||
| 36 | return DBUS_SERVER_LISTEN_DID_NOT_CONNECT; | ||
| 37 | } | ||
| 38 | |||
| 39 | - _dbus_string_init_const (&address, "systemd:"); | ||
| 40 | + if (!_dbus_string_init (&address)) | ||
| 41 | + goto systemd_oom; | ||
| 42 | |||
| 43 | - *server_p = _dbus_server_new_for_socket (fds, n, &address, NULL); | ||
| 44 | - if (*server_p == NULL) | ||
| 45 | + for (i = 0; i < n; i++) | ||
| 46 | { | ||
| 47 | - int i; | ||
| 48 | - | ||
| 49 | - for (i = 0; i < n; i++) | ||
| 50 | + if (i > 0) | ||
| 51 | { | ||
| 52 | - _dbus_close_socket (fds[i], NULL); | ||
| 53 | + if (!_dbus_string_append (&address, ";")) | ||
| 54 | + goto systemd_oom; | ||
| 55 | } | ||
| 56 | - dbus_free (fds); | ||
| 57 | - | ||
| 58 | - dbus_set_error (error, DBUS_ERROR_NO_MEMORY, NULL); | ||
| 59 | - return DBUS_SERVER_LISTEN_DID_NOT_CONNECT; | ||
| 60 | + if (!_dbus_append_address_from_socket (fds[i], &address, error)) | ||
| 61 | + goto systemd_err; | ||
| 62 | } | ||
| 63 | |||
| 64 | + *server_p = _dbus_server_new_for_socket (fds, n, &address, NULL); | ||
| 65 | + if (*server_p == NULL) | ||
| 66 | + goto systemd_oom; | ||
| 67 | + | ||
| 68 | dbus_free (fds); | ||
| 69 | |||
| 70 | return DBUS_SERVER_LISTEN_OK; | ||
| 71 | - } | ||
| 72 | + systemd_oom: | ||
| 73 | + _DBUS_SET_OOM (error); | ||
| 74 | + systemd_err: | ||
| 75 | + for (i = 0; i < n; i++) | ||
| 76 | + { | ||
| 77 | + _dbus_close_socket (fds[i], NULL); | ||
| 78 | + } | ||
| 79 | + dbus_free (fds); | ||
| 80 | + _dbus_string_free (&address); | ||
| 81 | + | ||
| 82 | + return DBUS_SERVER_LISTEN_DID_NOT_CONNECT; | ||
| 83 | + } | ||
| 84 | #ifdef DBUS_ENABLE_LAUNCHD | ||
| 85 | else if (strcmp (method, "launchd") == 0) | ||
| 86 | { | ||
| 87 | diff --git a/dbus/dbus-sysdeps-unix.c b/dbus/dbus-sysdeps-unix.c | ||
| 88 | index b4ecc96..55743b1 100644 | ||
| 89 | --- a/dbus/dbus-sysdeps-unix.c | ||
| 90 | +++ b/dbus/dbus-sysdeps-unix.c | ||
| 91 | @@ -55,6 +55,7 @@ | ||
| 92 | #include <netinet/in.h> | ||
| 93 | #include <netdb.h> | ||
| 94 | #include <grp.h> | ||
| 95 | +#include <arpa/inet.h> | ||
| 96 | |||
| 97 | #ifdef HAVE_ERRNO_H | ||
| 98 | #include <errno.h> | ||
| 99 | @@ -4160,4 +4161,71 @@ _dbus_check_setuid (void) | ||
| 100 | #endif | ||
| 101 | } | ||
| 102 | |||
| 103 | +/** | ||
| 104 | + * Read the address from the socket and append it to the string | ||
| 105 | + * | ||
| 106 | + * @param fd the socket | ||
| 107 | + * @param address | ||
| 108 | + * @param error return location for error code | ||
| 109 | + */ | ||
| 110 | +dbus_bool_t | ||
| 111 | +_dbus_append_address_from_socket (int fd, | ||
| 112 | + DBusString *address, | ||
| 113 | + DBusError *error) | ||
| 114 | +{ | ||
| 115 | + union { | ||
| 116 | + struct sockaddr sa; | ||
| 117 | + struct sockaddr_storage storage; | ||
| 118 | + struct sockaddr_un un; | ||
| 119 | + struct sockaddr_in ipv4; | ||
| 120 | + struct sockaddr_in6 ipv6; | ||
| 121 | + } socket; | ||
| 122 | + char hostip[INET6_ADDRSTRLEN]; | ||
| 123 | + int size = sizeof (socket); | ||
| 124 | + | ||
| 125 | + if (getsockname (fd, &socket.sa, &size)) | ||
| 126 | + goto err; | ||
| 127 | + | ||
| 128 | + switch (socket.sa.sa_family) | ||
| 129 | + { | ||
| 130 | + case AF_UNIX: | ||
| 131 | + if (socket.un.sun_path[0]=='\0') | ||
| 132 | + { | ||
| 133 | + if (_dbus_string_append_printf (address, "unix:abstract=%s", &(socket.un.sun_path[1]))) | ||
| 134 | + return TRUE; | ||
| 135 | + } | ||
| 136 | + else | ||
| 137 | + { | ||
| 138 | + if (_dbus_string_append_printf (address, "unix:path=%s", socket.un.sun_path)) | ||
| 139 | + return TRUE; | ||
| 140 | + } | ||
| 141 | + break; | ||
| 142 | + case AF_INET: | ||
| 143 | + if (inet_ntop (AF_INET, &socket.ipv4.sin_addr, hostip, sizeof (hostip))) | ||
| 144 | + if (_dbus_string_append_printf (address, "tcp:family=ipv4,host=%s,port=%u", | ||
| 145 | + hostip, ntohs (socket.ipv4.sin_port))) | ||
| 146 | + return TRUE; | ||
| 147 | + break; | ||
| 148 | +#ifdef AF_INET6 | ||
| 149 | + case AF_INET6: | ||
| 150 | + if (inet_ntop (AF_INET6, &socket.ipv6.sin6_addr, hostip, sizeof (hostip))) | ||
| 151 | + if (_dbus_string_append_printf (address, "tcp:family=ipv6,host=%s,port=%u", | ||
| 152 | + hostip, ntohs (socket.ipv6.sin6_port))) | ||
| 153 | + return TRUE; | ||
| 154 | + break; | ||
| 155 | +#endif | ||
| 156 | + default: | ||
| 157 | + dbus_set_error (error, | ||
| 158 | + _dbus_error_from_errno (EINVAL), | ||
| 159 | + "Failed to read address from socket: Unknown socket type."); | ||
| 160 | + return FALSE; | ||
| 161 | + } | ||
| 162 | + err: | ||
| 163 | + dbus_set_error (error, | ||
| 164 | + _dbus_error_from_errno (errno), | ||
| 165 | + "Failed to open socket: %s", | ||
| 166 | + _dbus_strerror (errno)); | ||
| 167 | + return FALSE; | ||
| 168 | +} | ||
| 169 | + | ||
| 170 | /* tests in dbus-sysdeps-util.c */ | ||
| 171 | diff --git a/dbus/dbus-sysdeps-unix.h b/dbus/dbus-sysdeps-unix.h | ||
| 172 | index 9b70896..a265b33 100644 | ||
| 173 | --- a/dbus/dbus-sysdeps-unix.h | ||
| 174 | +++ b/dbus/dbus-sysdeps-unix.h | ||
| 175 | @@ -138,6 +138,10 @@ dbus_bool_t _dbus_parse_uid (const DBusString *uid_str, | ||
| 176 | |||
| 177 | void _dbus_close_all (void); | ||
| 178 | |||
| 179 | +dbus_bool_t _dbus_append_address_from_socket (int fd, | ||
| 180 | + DBusString *address, | ||
| 181 | + DBusError *error); | ||
| 182 | + | ||
| 183 | /** @} */ | ||
| 184 | |||
| 185 | DBUS_END_DECLS | ||
| 186 | -- | ||
| 187 | cgit v0.9.0.2-2-gbebe | ||
diff --git a/meta/recipes-core/dbus/dbus.inc b/meta/recipes-core/dbus/dbus.inc index 4897fd3e57..affffbc187 100644 --- a/meta/recipes-core/dbus/dbus.inc +++ b/meta/recipes-core/dbus/dbus.inc | |||
| @@ -16,6 +16,7 @@ INC_PR = "r6" | |||
| 16 | 16 | ||
| 17 | SRC_URI = "http://dbus.freedesktop.org/releases/dbus/dbus-${PV}.tar.gz \ | 17 | SRC_URI = "http://dbus.freedesktop.org/releases/dbus/dbus-${PV}.tar.gz \ |
| 18 | file://tmpdir.patch; \ | 18 | file://tmpdir.patch; \ |
| 19 | file://systemd-address.patch \ | ||
| 19 | file://dbus-1.init" | 20 | file://dbus-1.init" |
| 20 | 21 | ||
| 21 | inherit useradd autotools pkgconfig gettext update-rc.d | 22 | inherit useradd autotools pkgconfig gettext update-rc.d |
