rpms/bluez
7
0
Fork 0
Code Issues Pull Requests Releases Activity

Update patches to apply correctly

Browse Source 
First compilable version with hostnamed support
This commit is contained in:
Bastien Nocera 2011-06-23 14:52:45 +01:00
parent f18de429c1
commit 978b40871b
3 changed files with 329 additions and 185 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

50
0001-Add-sixaxis-cable-pairing-plugin.patch
View File

@ -1,4 +1,4 @@
From 61443ce94d503c359a99e7885fe489cfcff380fa Mon Sep 17 00:00:00 2001 From b9fd541d4ed3320ea9d1796166956a9d03d15e10 Mon Sep 17 00:00:00 2001
From: Bastien Nocera <hadess@hadess.net> From: Bastien Nocera <hadess@hadess.net>
Date: Tue, 1 Sep 2009 17:32:48 +0100 Date: Tue, 1 Sep 2009 17:32:48 +0100
Subject: [PATCH] Add sixaxis cable-pairing plugin Subject: [PATCH] Add sixaxis cable-pairing plugin
@ -15,13 +15,11 @@ address, and added to the database of the current default adapter.
plugins/cable.c | 382 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ plugins/cable.c | 382 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
src/adapter.c | 19 +++ src/adapter.c | 19 +++
src/adapter.h | 3 + src/adapter.h | 3 +
src/manager.c | 5 + 6 files changed, 428 insertions(+), 2 deletions(-)
src/manager.h | 1 +
8 files changed, 434 insertions(+), 2 deletions(-)
create mode 100644 plugins/cable.c create mode 100644 plugins/cable.c
diff --git a/Makefile.am b/Makefile.am diff --git a/Makefile.am b/Makefile.am
index 2f3051c..145b26d 100644 index ba9a89d..b5851e0 100644
--- a/Makefile.am --- a/Makefile.am
+++ b/Makefile.am +++ b/Makefile.am
@@ -202,6 +202,11 @@ builtin_sources += health/hdp_main.c health/hdp_types.h \ @@ -202,6 +202,11 @@ builtin_sources += health/hdp_main.c health/hdp_types.h \
@ -45,7 +43,7 @@ index 2f3051c..145b26d 100644
src_bluetoothd_LDFLAGS = -Wl,--export-dynamic \ src_bluetoothd_LDFLAGS = -Wl,--export-dynamic \
-Wl,--version-script=$(srcdir)/src/bluetooth.ver -Wl,--version-script=$(srcdir)/src/bluetooth.ver
@@ -373,7 +378,7 @@ EXTRA_DIST += doc/manager-api.txt \ @@ -388,7 +393,7 @@ EXTRA_DIST += doc/manager-api.txt \
AM_YFLAGS = -d AM_YFLAGS = -d
@ -108,7 +106,7 @@ index d77937b..7e6422e 100644
+ AM_CONDITIONAL(CABLE, test "${cable_enable}" = "yes" && test "${cable_found}" = "yes") + AM_CONDITIONAL(CABLE, test "${cable_enable}" = "yes" && test "${cable_found}" = "yes")
]) ])
diff --git a/configure.ac b/configure.ac diff --git a/configure.ac b/configure.ac
index 987b7e1..aece18b 100644 index 084ece5..0ef0225 100644
--- a/configure.ac --- a/configure.ac
+++ b/configure.ac +++ b/configure.ac
@@ -46,6 +46,7 @@ AC_PATH_GSTREAMER @@ -46,6 +46,7 @@ AC_PATH_GSTREAMER
@ -508,10 +506,10 @@ index 0000000..e8cff76
+BLUETOOTH_PLUGIN_DEFINE(cable, VERSION, +BLUETOOTH_PLUGIN_DEFINE(cable, VERSION,
+ BLUETOOTH_PLUGIN_PRIORITY_DEFAULT, cable_init, cable_exit) + BLUETOOTH_PLUGIN_PRIORITY_DEFAULT, cable_init, cable_exit)
diff --git a/src/adapter.c b/src/adapter.c diff --git a/src/adapter.c b/src/adapter.c
index 5598d17..86dfccc 100644 index 156d534..51e3b4c 100644
--- a/src/adapter.c --- a/src/adapter.c
+++ b/src/adapter.c +++ b/src/adapter.c
@@ -1081,6 +1081,25 @@ static struct btd_device *adapter_create_device(DBusConnection *conn, @@ -1088,6 +1088,25 @@ static struct btd_device *adapter_create_device(DBusConnection *conn,
return device; return device;
} }
@ -538,10 +536,10 @@ index 5598d17..86dfccc 100644
struct btd_device *device, struct btd_device *device,
gboolean remove_storage) gboolean remove_storage)
diff --git a/src/adapter.h b/src/adapter.h diff --git a/src/adapter.h b/src/adapter.h
index 13971bf..c85fa66 100644 index 38ea3ca..55c77aa 100644
--- a/src/adapter.h --- a/src/adapter.h
+++ b/src/adapter.h +++ b/src/adapter.h
@@ -114,6 +114,9 @@ void adapter_update_found_devices(struct btd_adapter *adapter, bdaddr_t *bdaddr, @@ -116,6 +116,9 @@ void adapter_update_found_devices(struct btd_adapter *adapter, bdaddr_t *bdaddr,
int adapter_remove_found_device(struct btd_adapter *adapter, bdaddr_t *bdaddr); int adapter_remove_found_device(struct btd_adapter *adapter, bdaddr_t *bdaddr);
void adapter_emit_device_found(struct btd_adapter *adapter, void adapter_emit_device_found(struct btd_adapter *adapter,
struct remote_dev_info *dev); struct remote_dev_info *dev);
@ -551,34 +549,6 @@ index 13971bf..c85fa66 100644
void adapter_mode_changed(struct btd_adapter *adapter, uint8_t scan_mode); void adapter_mode_changed(struct btd_adapter *adapter, uint8_t scan_mode);
int adapter_update_local_name(struct btd_adapter *adapter, const char *name); int adapter_update_local_name(struct btd_adapter *adapter, const char *name);
void adapter_service_insert(struct btd_adapter *adapter, void *rec); void adapter_service_insert(struct btd_adapter *adapter, void *rec);
diff --git a/src/manager.c b/src/manager.c
index dedec8b..d18d588 100644
--- a/src/manager.c
+++ b/src/manager.c
@@ -262,6 +262,11 @@ static void manager_set_default_adapter(int id)
DBUS_TYPE_INVALID);
}
+struct btd_adapter *manager_get_default_adapter(void)
+{
+ return manager_find_adapter_by_id(default_adapter_id);
+}
+
static void manager_remove_adapter(struct btd_adapter *adapter)
{
uint16_t dev_id = adapter_get_dev_id(adapter);
diff --git a/src/manager.h b/src/manager.h
index 90d3690..f5f9757 100644
--- a/src/manager.h
+++ b/src/manager.h
@@ -35,6 +35,7 @@ void manager_cleanup(DBusConnection *conn, const char *path);
const char *manager_get_base_path(void);
struct btd_adapter *manager_find_adapter(const bdaddr_t *sba);
struct btd_adapter *manager_find_adapter_by_id(int id);
+struct btd_adapter *manager_get_default_adapter(void);
void manager_foreach_adapter(adapter_cb func, gpointer user_data);
GSList *manager_get_adapters(void);
struct btd_adapter *btd_manager_register_adapter(int id);
-- --
1.7.5.1 1.7.5.4

9
bluez.spec
View File

@ -1,7 +1,7 @@
Summary: Bluetooth utilities Summary: Bluetooth utilities
Name: bluez Name: bluez
Version: 4.94 Version: 4.94
Release: 2%{?dist} Release: 3%{?dist}
License: GPLv2+ License: GPLv2+
Group: Applications/System Group: Applications/System
Source: http://www.kernel.org/pub/linux/bluetooth/%{name}-%{version}.tar.gz Source: http://www.kernel.org/pub/linux/bluetooth/%{name}-%{version}.tar.gz
@ -13,7 +13,7 @@ Source7: rfcomm.init
Source8: bluez-uinput.modules Source8: bluez-uinput.modules
# http://thread.gmane.org/gmane.linux.bluez.kernel/13255 # http://thread.gmane.org/gmane.linux.bluez.kernel/13255
Patch1: 0001-adaptername-Move-adapter-naming-into-a-plugin.patch Patch1: pretty-hostname-support.patch
# https://bugzilla.redhat.com/show_bug.cgi?id=498756 # https://bugzilla.redhat.com/show_bug.cgi?id=498756
Patch4: bluez-socket-mobile-cf-connection-kit.patch Patch4: bluez-socket-mobile-cf-connection-kit.patch
# http://thread.gmane.org/gmane.linux.bluez.kernel/2396 # http://thread.gmane.org/gmane.linux.bluez.kernel/2396
@ -236,6 +236,7 @@ fi
%{_bindir}/l2ping %{_bindir}/l2ping
%{_bindir}/rfcomm %{_bindir}/rfcomm
%{_bindir}/sdptool %{_bindir}/sdptool
%{_bindir}/gattool
%{_sbindir}/* %{_sbindir}/*
%{_mandir}/man1/ciptool.1.gz %{_mandir}/man1/ciptool.1.gz
%{_mandir}/man1/dfutool.1.gz %{_mandir}/man1/dfutool.1.gz
@ -295,6 +296,10 @@ fi
%{_mandir}/man1/pand.1.gz %{_mandir}/man1/pand.1.gz
%changelog %changelog
* Thu Jun 23 2011 Bastien Nocera <bnocera@redhat.com> 4.94-3
- Update patches to apply correctly
- First compilable version with hostnamed support
* Mon Jun 20 2011 Lennart Poettering <lpoetter@redhat.com> - 4.94-2 * Mon Jun 20 2011 Lennart Poettering <lpoetter@redhat.com> - 4.94-2
- Enable bluetoothd by default - Enable bluetoothd by default
- Follow-up on https://bugzilla.redhat.com/show_bug.cgi?id=694519 also fixing upgrades - Follow-up on https://bugzilla.redhat.com/show_bug.cgi?id=694519 also fixing upgrades

411
0001-adaptername-Move-adapter-naming-into-a-plugin.patch → pretty-hostname-support.patch
View File

@ -1,7 +1,219 @@
From b228f205853a122327a9ce93a75fb88848c02312 Mon Sep 17 00:00:00 2001 From fedee1259e50a5312720adc895bc2fd0270395aa Mon Sep 17 00:00:00 2001
From: Bastien Nocera <hadess@hadess.net> From: Bastien Nocera <hadess@hadess.net>
Date: Tue, 10 May 2011 18:02:14 +0100 Date: Thu, 9 Jun 2011 14:41:56 +0100
Subject: [PATCH] adaptername: Move adapter naming into a plugin Subject: [PATCH 1/3] Simplify "SetName" D-Bus call
By merging most of the functionality into adapter_update_local_name()
This also allows us to drop the adapter->name_stored struct member
as in any calls to adapter_update_local_name(), we would want to
write out the adapter name.
---
src/adapter.c | 61 ++++++++++++++++++++++++--------------------------------
src/adapter.h | 2 +-
2 files changed, 27 insertions(+), 36 deletions(-)
diff --git a/src/adapter.c b/src/adapter.c
index 2650b56..f8c0370 100644
--- a/src/adapter.c
+++ b/src/adapter.c
@@ -146,8 +146,6 @@ struct btd_adapter {
GSList *powered_callbacks;
- gboolean name_stored;
-
GSList *loaded_drivers;
};
@@ -914,10 +912,17 @@ void btd_adapter_class_changed(struct btd_adapter *adapter, uint32_t new_class)
DBUS_TYPE_UINT32, &new_class);
}
-void adapter_update_local_name(struct btd_adapter *adapter, const char *name)
+int adapter_update_local_name(struct btd_adapter *adapter, const char *name)
{
+ char *name_ptr;
+
if (strncmp(name, adapter->name, MAX_NAME_LENGTH) == 0)
- return;
+ return 0;
+
+ if (!g_utf8_validate(name, -1, NULL)) {
+ error("Name change failed: supplied name isn't valid UTF-8");
+ return -EINVAL;
+ }
strncpy(adapter->name, name, MAX_NAME_LENGTH);
@@ -925,49 +930,36 @@ void adapter_update_local_name(struct btd_adapter *adapter, const char *name)
attrib_gap_set(GATT_CHARAC_DEVICE_NAME,
(const uint8_t *) adapter->name, strlen(adapter->name));
- if (!adapter->name_stored) {
- char *name_ptr = adapter->name;
+ name_ptr = adapter->name;
- write_local_name(&adapter->bdaddr, adapter->name);
+ write_local_name(&adapter->bdaddr, adapter->name);
- if (connection)
- emit_property_changed(connection, adapter->path,
- ADAPTER_INTERFACE, "Name",
- DBUS_TYPE_STRING, &name_ptr);
+ if (connection)
+ emit_property_changed(connection, adapter->path,
+ ADAPTER_INTERFACE, "Name",
+ DBUS_TYPE_STRING, &name_ptr);
+
+ if (adapter->up) {
+ int err = adapter_ops->set_name(adapter->dev_id, name);
+ if (err < 0)
+ return err;
}
- adapter->name_stored = FALSE;
+ return 0;
}
static DBusMessage *set_name(DBusConnection *conn, DBusMessage *msg,
const char *name, void *data)
{
struct btd_adapter *adapter = data;
- char *name_ptr = adapter->name;
+ int ret;
- if (!g_utf8_validate(name, -1, NULL)) {
- error("Name change failed: supplied name isn't valid UTF-8");
+ ret = adapter_update_local_name(adapter, name);
+ if (ret == -EINVAL)
return btd_error_invalid_args(msg);
- }
-
- if (strncmp(name, adapter->name, MAX_NAME_LENGTH) == 0)
- goto done;
-
- strncpy(adapter->name, name, MAX_NAME_LENGTH);
- write_local_name(&adapter->bdaddr, name);
- emit_property_changed(connection, adapter->path,
- ADAPTER_INTERFACE, "Name",
- DBUS_TYPE_STRING, &name_ptr);
+ else if (ret < 0)
+ return btd_error_failed(msg, strerror(-ret));
- if (adapter->up) {
- int err = adapter_ops->set_name(adapter->dev_id, name);
- if (err < 0)
- return btd_error_failed(msg, strerror(-err));
-
- adapter->name_stored = TRUE;
- }
-
-done:
return dbus_message_new_method_return(msg);
}
@@ -2496,7 +2488,6 @@ int btd_adapter_stop(struct btd_adapter *adapter)
adapter->mode = MODE_OFF;
adapter->state = STATE_IDLE;
adapter->off_requested = FALSE;
- adapter->name_stored = FALSE;
call_adapter_powered_callbacks(adapter, FALSE);
diff --git a/src/adapter.h b/src/adapter.h
index 610331f..9516a99 100644
--- a/src/adapter.h
+++ b/src/adapter.h
@@ -115,7 +115,7 @@ int adapter_remove_found_device(struct btd_adapter *adapter, bdaddr_t *bdaddr);
void adapter_emit_device_found(struct btd_adapter *adapter,
struct remote_dev_info *dev);
void adapter_mode_changed(struct btd_adapter *adapter, uint8_t scan_mode);
-void adapter_update_local_name(struct btd_adapter *adapter, const char *name);
+int adapter_update_local_name(struct btd_adapter *adapter, const char *name);
void adapter_service_insert(struct btd_adapter *adapter, void *rec);
void adapter_service_remove(struct btd_adapter *adapter, void *rec);
void btd_adapter_class_changed(struct btd_adapter *adapter,
--
1.7.5.4
From c43704f16387fc645260bc68cd9a844dbfa1624c Mon Sep 17 00:00:00 2001
From: Bastien Nocera <hadess@hadess.net>
Date: Wed, 22 Jun 2011 11:56:53 +0100
Subject: [PATCH 2/3] Add ability to block name changes
---
src/adapter.c | 12 ++++++++++++
src/adapter.h | 2 ++
2 files changed, 14 insertions(+), 0 deletions(-)
diff --git a/src/adapter.c b/src/adapter.c
index f8c0370..5420acd 100644
--- a/src/adapter.c
+++ b/src/adapter.c
@@ -111,6 +111,7 @@ struct btd_adapter {
bdaddr_t bdaddr; /* adapter Bluetooth Address */
uint32_t dev_class; /* Class of Device */
char name[MAX_NAME_LENGTH + 1]; /* adapter name */
+ gboolean allow_name_changes; /* whether the adapter name can be changed */
guint discov_timeout_id; /* discoverable timeout id */
guint stop_discov_id; /* stop inquiry/scanning id */
uint32_t discov_timeout; /* discoverable time(sec) */
@@ -916,6 +917,9 @@ int adapter_update_local_name(struct btd_adapter *adapter, const char *name)
{
char *name_ptr;
+ if (adapter->allow_name_changes == FALSE)
+ return -EPERM;
+
if (strncmp(name, adapter->name, MAX_NAME_LENGTH) == 0)
return 0;
@@ -2563,6 +2567,8 @@ gboolean adapter_init(struct btd_adapter *adapter)
* start off as powered */
adapter->up = TRUE;
+ adapter->allow_name_changes = TRUE;
+
adapter_ops->read_bdaddr(adapter->dev_id, &adapter->bdaddr);
if (bacmp(&adapter->bdaddr, BDADDR_ANY) == 0) {
@@ -2663,6 +2669,12 @@ void adapter_get_address(struct btd_adapter *adapter, bdaddr_t *bdaddr)
bacpy(bdaddr, &adapter->bdaddr);
}
+void adapter_set_allow_name_changes(struct btd_adapter *adapter,
+ gboolean allow_name_changes)
+{
+ adapter->allow_name_changes = allow_name_changes;
+}
+
static inline void suspend_discovery(struct btd_adapter *adapter)
{
if (adapter->state != STATE_SUSPENDED)
diff --git a/src/adapter.h b/src/adapter.h
index 9516a99..38ea3ca 100644
--- a/src/adapter.h
+++ b/src/adapter.h
@@ -100,6 +100,8 @@ int adapter_resolve_names(struct btd_adapter *adapter);
struct btd_adapter *adapter_create(DBusConnection *conn, int id);
gboolean adapter_init(struct btd_adapter *adapter);
void adapter_remove(struct btd_adapter *adapter);
+void adapter_set_allow_name_changes(struct btd_adapter *adapter,
+ gboolean allow_name_changes);
uint16_t adapter_get_dev_id(struct btd_adapter *adapter);
const gchar *adapter_get_path(struct btd_adapter *adapter);
void adapter_get_address(struct btd_adapter *adapter, bdaddr_t *bdaddr);
--
1.7.5.4
From 4b8e0372aaaa35219814649b1fb91c222c4c5bab Mon Sep 17 00:00:00 2001
From: Bastien Nocera <hadess@hadess.net>
Date: Wed, 22 Jun 2011 12:03:22 +0100
Subject: [PATCH 3/3] Add adaptername plugin
Replacing the name setting code in src/adapter.c.
Moving the adapter naming allows us to use the /etc/machine-info [1] Moving the adapter naming allows us to use the /etc/machine-info [1]
pretty hostname, as implemented by hostnamed [2] in systemd. pretty hostname, as implemented by hostnamed [2] in systemd.
@ -15,21 +227,22 @@ available, but we should instead number it if it happens not
to be the default adapter. As we cannot be told when the default to be the default adapter. As we cannot be told when the default
adapter changes, we'll behave this way for now. adapter changes, we'll behave this way for now.
Note that when an adapter name is set automatically from
the pretty hostname, changing it through the D-Bus interface
will fail.
[1]: http://0pointer.de/public/systemd-man/machine-info.html [1]: http://0pointer.de/public/systemd-man/machine-info.html
[2]: http://www.freedesktop.org/wiki/Software/systemd/hostnamed [2]: http://www.freedesktop.org/wiki/Software/systemd/hostnamed
--- ---
Makefile.am | 3 + Makefile.am | 3 +
configure.ac | 4 + configure.ac | 4 +
plugins/adaptername.c | 289 +++++++++++++++++++++++++++++++++++++++++++++++++ plugins/adaptername.c | 306 +++++++++++++++++++++++++++++++++++++++++++++++++
src/adapter.c | 107 +++---------------- src/adapter.c | 66 -----------
src/adapter.h | 2 +- 4 files changed, 313 insertions(+), 66 deletions(-)
src/manager.c | 5 +
src/manager.h | 1 +
7 files changed, 319 insertions(+), 92 deletions(-)
create mode 100644 plugins/adaptername.c create mode 100644 plugins/adaptername.c
diff --git a/Makefile.am b/Makefile.am diff --git a/Makefile.am b/Makefile.am
index 175f8c9..2f3051c 100644 index c2a6976..ba9a89d 100644
--- a/Makefile.am --- a/Makefile.am
+++ b/Makefile.am +++ b/Makefile.am
@@ -218,6 +218,9 @@ EXTRA_DIST += plugins/hal.c plugins/formfactor.c @@ -218,6 +218,9 @@ EXTRA_DIST += plugins/hal.c plugins/formfactor.c
@ -43,7 +256,7 @@ index 175f8c9..2f3051c 100644
builtin_modules += maemo6 builtin_modules += maemo6
builtin_sources += plugins/maemo6.c builtin_sources += plugins/maemo6.c
diff --git a/configure.ac b/configure.ac diff --git a/configure.ac b/configure.ac
index 111ff01..987b7e1 100644 index 07b9e55..084ece5 100644
--- a/configure.ac --- a/configure.ac
+++ b/configure.ac +++ b/configure.ac
@@ -35,6 +35,10 @@ AC_FUNC_PPOLL @@ -35,6 +35,10 @@ AC_FUNC_PPOLL
@ -59,15 +272,15 @@ index 111ff01..987b7e1 100644
AC_PATH_ALSA AC_PATH_ALSA
diff --git a/plugins/adaptername.c b/plugins/adaptername.c diff --git a/plugins/adaptername.c b/plugins/adaptername.c
new file mode 100644 new file mode 100644
index 0000000..53c67e9 index 0000000..af4dac8
--- /dev/null --- /dev/null
+++ b/plugins/adaptername.c +++ b/plugins/adaptername.c
@@ -0,0 +1,289 @@ @@ -0,0 +1,306 @@
+/* +/*
+ * + *
+ * BlueZ - Bluetooth protocol stack for Linux + * BlueZ - Bluetooth protocol stack for Linux
+ * + *
+ * Copyright (C) 2004-2010 Marcel Holtmann <marcel@holtmann.org> + * Copyright (C) 2011 Red Hat, Inc.
+ * + *
+ * + *
+ * This program is free software; you can redistribute it and/or modify + * This program is free software; you can redistribute it and/or modify
@ -84,6 +297,8 @@ index 0000000..53c67e9
+ * along with this program; if not, write to the Free Software + * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+ * + *
+ * Author: Bastien Nocera <hadess@hadess.net>
+ *
+ */ + */
+ +
+#ifdef HAVE_CONFIG_H +#ifdef HAVE_CONFIG_H
@ -125,8 +340,6 @@ index 0000000..53c67e9
+ char **lines; + char **lines;
+ guint i; + guint i;
+ +
+ ret = NULL;
+
+ if (g_file_get_contents(MACHINE_INFO_DIR MACHINE_INFO_FILE, + if (g_file_get_contents(MACHINE_INFO_DIR MACHINE_INFO_FILE,
+ &contents, NULL, NULL) == FALSE) { + &contents, NULL, NULL) == FALSE) {
+ return NULL; + return NULL;
@ -137,6 +350,7 @@ index 0000000..53c67e9
+ if (lines == NULL) + if (lines == NULL)
+ return NULL; + return NULL;
+ +
+ ret = NULL;
+ for (i = 0; lines[i] != NULL; i++) { + for (i = 0; lines[i] != NULL; i++) {
+ if (g_str_has_prefix(lines[i], "PRETTY_HOSTNAME=")) { + if (g_str_has_prefix(lines[i], "PRETTY_HOSTNAME=")) {
+ ret = g_strdup(lines[i] + strlen("PRETTY_HOSTNAME=")); + ret = g_strdup(lines[i] + strlen("PRETTY_HOSTNAME="));
@ -145,6 +359,7 @@ index 0000000..53c67e9
+ } + }
+ +
+ g_strfreev(lines); + g_strfreev(lines);
+
+ return ret; + return ret;
+} +}
+ +
@ -207,6 +422,16 @@ index 0000000..53c67e9
+ return dst; + return dst;
+} +}
+ +
+static int
+get_default_adapter_id(void)
+{
+ struct btd_adapter *default_adapter;
+ default_adapter = manager_get_default_adapter();
+ if (default_adapter == NULL)
+ return -1;
+ return adapter_get_dev_id(default_adapter);
+}
+
+static int adaptername_probe(struct btd_adapter *adapter) +static int adaptername_probe(struct btd_adapter *adapter)
+{ +{
+ int current_id; + int current_id;
@ -220,7 +445,10 @@ index 0000000..53c67e9
+ if (pretty_hostname != NULL) { + if (pretty_hostname != NULL) {
+ int default_adapter; + int default_adapter;
+ +
+ default_adapter = manager_get_default_adapter(); + default_adapter = get_default_adapter_id();
+
+ /* Allow us to change the name */
+ adapter_set_allow_name_changes(adapter, TRUE);
+ +
+ /* If it's the first device, let's assume it will + /* If it's the first device, let's assume it will
+ * be the default one, as we're not told when + * be the default one, as we're not told when
@ -243,15 +471,18 @@ index 0000000..53c67e9
+ } + }
+ g_free(pretty_hostname); + g_free(pretty_hostname);
+ +
+ /* And disable the name change now */
+ adapter_set_allow_name_changes(adapter, FALSE);
+
+ return 0; + return 0;
+ } + }
+ +
+ adapter_set_allow_name_changes(adapter, TRUE);
+ adapter_get_address(adapter, &bdaddr); + adapter_get_address(adapter, &bdaddr);
+ +
+ if (read_local_name(&bdaddr, name) < 0) { + if (read_local_name(&bdaddr, name) < 0)
+ expand_name(name, MAX_NAME_LENGTH, main_opts.name, + expand_name(name, MAX_NAME_LENGTH, main_opts.name,
+ current_id); + current_id);
+ }
+ DBG("Setting name '%s' for device 'hci%d'", name, current_id); + DBG("Setting name '%s' for device 'hci%d'", name, current_id);
+ adapter_update_local_name(adapter, name); + adapter_update_local_name(adapter, name);
+ +
@ -313,11 +544,12 @@ index 0000000..53c67e9
+static int adaptername_init(void) +static int adaptername_init(void)
+{ +{
+ int ret; + int ret;
+ int inot_fd;
+ +
+ ret = btd_register_adapter_driver(&adaptername_driver); + ret = btd_register_adapter_driver(&adaptername_driver);
+ +
+ if (ret == 0) { + if (ret != 0)
+ int inot_fd; + return ret;
+ +
+ inot_fd = inotify_init(); + inot_fd = inotify_init();
+ if (inot_fd < 0) { + if (inot_fd < 0) {
@ -326,7 +558,8 @@ index 0000000..53c67e9
+ } + }
+ watch_fd = inotify_add_watch(inot_fd, + watch_fd = inotify_add_watch(inot_fd,
+ MACHINE_INFO_DIR, + MACHINE_INFO_DIR,
+ IN_CLOSE_WRITE | IN_DELETE | IN_CREATE); + IN_CLOSE_WRITE | IN_DELETE | IN_CREATE |
+ IN_MOVED_FROM | IN_MOVED_TO);
+ if (watch_fd < 0) { + if (watch_fd < 0) {
+ error("Failed to setup watch for '%s'", + error("Failed to setup watch for '%s'",
+ MACHINE_INFO_DIR); + MACHINE_INFO_DIR);
@ -342,9 +575,6 @@ index 0000000..53c67e9
+ return 0; + return 0;
+} +}
+ +
+ return ret;
+}
+
+static void adaptername_exit(void) +static void adaptername_exit(void)
+{ +{
+ btd_unregister_adapter_driver(&adaptername_driver); + btd_unregister_adapter_driver(&adaptername_driver);
@ -353,10 +583,10 @@ index 0000000..53c67e9
+BLUETOOTH_PLUGIN_DEFINE(adaptername, VERSION, +BLUETOOTH_PLUGIN_DEFINE(adaptername, VERSION,
+ BLUETOOTH_PLUGIN_PRIORITY_LOW, adaptername_init, adaptername_exit) + BLUETOOTH_PLUGIN_PRIORITY_LOW, adaptername_init, adaptername_exit)
diff --git a/src/adapter.c b/src/adapter.c diff --git a/src/adapter.c b/src/adapter.c
index c30febc..5598d17 100644 index 5420acd..156d534 100644
--- a/src/adapter.c --- a/src/adapter.c
+++ b/src/adapter.c +++ b/src/adapter.c
@@ -182,64 +182,6 @@ static void dev_info_free(struct remote_dev_info *dev) @@ -183,64 +183,6 @@ static void dev_info_free(struct remote_dev_info *dev)
g_free(dev); g_free(dev);
} }
@ -421,73 +651,7 @@ index c30febc..5598d17 100644
int btd_adapter_set_class(struct btd_adapter *adapter, uint8_t major, int btd_adapter_set_class(struct btd_adapter *adapter, uint8_t major,
uint8_t minor) uint8_t minor)
{ {
@@ -912,10 +854,10 @@ void btd_adapter_class_changed(struct btd_adapter *adapter, uint32_t new_class) @@ -2576,14 +2518,6 @@ gboolean adapter_init(struct btd_adapter *adapter)
DBUS_TYPE_UINT32, &new_class);
}
-void adapter_update_local_name(struct btd_adapter *adapter, const char *name)
+int adapter_update_local_name(struct btd_adapter *adapter, const char *name)
{
if (strncmp(name, adapter->name, MAX_NAME_LENGTH) == 0)
- return;
+ return 0;
strncpy(adapter->name, name, MAX_NAME_LENGTH);
@@ -934,38 +876,29 @@ void adapter_update_local_name(struct btd_adapter *adapter, const char *name)
DBUS_TYPE_STRING, &name_ptr);
}
+ if (adapter->up) {
+ int err = adapter_ops->set_name(adapter->dev_id, name);
+ if (err < 0)
+ return -err;
+
+ adapter->name_stored = TRUE;
+ }
+
adapter->name_stored = FALSE;
+
+ return 0;
}
static DBusMessage *set_name(DBusConnection *conn, DBusMessage *msg,
const char *name, void *data)
{
struct btd_adapter *adapter = data;
- char *name_ptr = adapter->name;
-
- if (!g_utf8_validate(name, -1, NULL)) {
- error("Name change failed: supplied name isn't valid UTF-8");
- return btd_error_invalid_args(msg);
- }
-
- if (strncmp(name, adapter->name, MAX_NAME_LENGTH) == 0)
- goto done;
-
- strncpy(adapter->name, name, MAX_NAME_LENGTH);
- write_local_name(&adapter->bdaddr, name);
- emit_property_changed(connection, adapter->path,
- ADAPTER_INTERFACE, "Name",
- DBUS_TYPE_STRING, &name_ptr);
-
- if (adapter->up) {
- int err = adapter_ops->set_name(adapter->dev_id, name);
- if (err < 0)
- return btd_error_failed(msg, strerror(-err));
+ int err;
- adapter->name_stored = TRUE;
- }
+ err = adapter_update_local_name (adapter, name);
+ if (err < 0)
+ return btd_error_failed(msg, strerror(err));
-done:
return dbus_message_new_method_return(msg);
}
@@ -2577,14 +2510,6 @@ gboolean adapter_init(struct btd_adapter *adapter)
return FALSE; return FALSE;
} }
@ -502,44 +666,49 @@ index c30febc..5598d17 100644
sdp_init_services_list(&adapter->bdaddr); sdp_init_services_list(&adapter->bdaddr);
load_drivers(adapter); load_drivers(adapter);
clear_blocked(adapter); clear_blocked(adapter);
diff --git a/src/adapter.h b/src/adapter.h --
index 3526849..13971bf 100644 1.7.5.4
--- a/src/adapter.h
+++ b/src/adapter.h From e93fae6f51494e36f011a4a22efab7769fd0b8fb Mon Sep 17 00:00:00 2001
@@ -115,7 +115,7 @@ int adapter_remove_found_device(struct btd_adapter *adapter, bdaddr_t *bdaddr); From: Bastien Nocera <hadess@hadess.net>
void adapter_emit_device_found(struct btd_adapter *adapter, Date: Wed, 8 Jun 2011 15:09:16 +0200
struct remote_dev_info *dev); Subject: [PATCH] Re-add manager_get_default_adapter()
void adapter_mode_changed(struct btd_adapter *adapter, uint8_t scan_mode);
-void adapter_update_local_name(struct btd_adapter *adapter, const char *name); This will be useful for upcoming plugins like the sixaxis one or the
+int adapter_update_local_name(struct btd_adapter *adapter, const char *name); adaptername one.
void adapter_service_insert(struct btd_adapter *adapter, void *rec); ---
void adapter_service_remove(struct btd_adapter *adapter, void *rec); src/manager.c | 5 +++++
void btd_adapter_class_changed(struct btd_adapter *adapter, src/manager.h | 1 +
2 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/src/manager.c b/src/manager.c diff --git a/src/manager.c b/src/manager.c
index e805e0c..dedec8b 100644 index e805e0c..254ace4 100644
--- a/src/manager.c --- a/src/manager.c
+++ b/src/manager.c +++ b/src/manager.c
@@ -288,6 +288,11 @@ static void manager_remove_adapter(struct btd_adapter *adapter) @@ -262,6 +262,11 @@ static void manager_set_default_adapter(int id)
btd_start_exit_timer(); DBUS_TYPE_INVALID);
} }
+int manager_get_default_adapter (void) +struct btd_adapter *manager_get_default_adapter(void)
+{ +{
+ return default_adapter_id; + return manager_find_adapter_by_id(default_adapter_id);
+} +}
+ +
void manager_cleanup(DBusConnection *conn, const char *path) static void manager_remove_adapter(struct btd_adapter *adapter)
{ {
g_slist_foreach(adapters, (GFunc) manager_remove_adapter, NULL); uint16_t dev_id = adapter_get_dev_id(adapter);
diff --git a/src/manager.h b/src/manager.h diff --git a/src/manager.h b/src/manager.h
index 05c38b3..90d3690 100644 index 05c38b3..4f92d2f 100644
--- a/src/manager.h --- a/src/manager.h
+++ b/src/manager.h +++ b/src/manager.h
@@ -41,3 +41,4 @@ struct btd_adapter *btd_manager_register_adapter(int id); @@ -35,6 +35,7 @@ void manager_cleanup(DBusConnection *conn, const char *path);
int btd_manager_unregister_adapter(int id); const char *manager_get_base_path(void);
void manager_add_adapter(const char *path); struct btd_adapter *manager_find_adapter(const bdaddr_t *sba);
void btd_manager_set_did(uint16_t vendor, uint16_t product, uint16_t version); struct btd_adapter *manager_find_adapter_by_id(int id);
+int manager_get_default_adapter (void); +struct btd_adapter *manager_get_default_adapter(void);
void manager_foreach_adapter(adapter_cb func, gpointer user_data);
GSList *manager_get_adapters(void);
struct btd_adapter *btd_manager_register_adapter(int id);
-- --
1.7.5.1 1.7.5.4