This patch moves the SCO socket handling from hfp_hf_bluez5 plugin to
handsfree-audio.c file.
This is the initial step to be able to support sending the file
descriptor through the Agent NewConnection method.
---
plugins/hfp_hf_bluez5.c | 102 +----------------------------------------------
src/handsfree-audio.c | 103 +++++++++++++++++++++++++++++++++++++++++++++++-
2 files changed, 103 insertions(+), 102 deletions(-)
diff --git a/plugins/hfp_hf_bluez5.c b/plugins/hfp_hf_bluez5.c
index dea0f34..ea2c848 100644
--- a/plugins/hfp_hf_bluez5.c
+++ b/plugins/hfp_hf_bluez5.c
@@ -67,8 +67,6 @@ struct hfp {
};
static GDBusClient *bluez = NULL;
-static guint sco_watch = 0;
-static uint16_t local_hfp_version = HFP_VERSION_1_6;
static void hfp_debug(const char *str, void *user_data)
{
@@ -482,100 +480,11 @@ static const GDBusMethodTable profile_methods[] = {
{ }
};
-static ofono_bool_t slc_match(struct ofono_modem *modem, void *userdata)
-{
- const char *remote = userdata;
- const char *value = ofono_modem_get_string(modem, "Remote");
-
- if (value == NULL)
- return FALSE;
-
- /* Make sure SLC has been established */
- if (ofono_modem_get_powered(modem) != TRUE)
- return FALSE;
-
- return g_str_equal(remote, value);
-}
-
-static gboolean sco_accept(GIOChannel *io, GIOCondition cond,
- gpointer user_data)
-{
- struct sockaddr_sco saddr;
- socklen_t alen;
- int sk, nsk;
- char remote[18];
-
- if (cond & (G_IO_ERR | G_IO_HUP | G_IO_NVAL))
- return FALSE;
-
- sk = g_io_channel_unix_get_fd(io);
-
- memset(&saddr, 0, sizeof(saddr));
- alen = sizeof(saddr);
-
- nsk = accept(sk, (struct sockaddr *) &saddr, &alen);
- if (nsk < 0)
- return TRUE;
-
- bt_ba2str(&saddr.sco_bdaddr, remote);
-
- if (ofono_modem_find(slc_match, remote) == NULL) {
- ofono_error("Rejecting SCO: No SLC connection found!");
- close(nsk);
- return TRUE;
- }
-
- return TRUE;
-}
-
-static int sco_init(void)
-{
- GIOChannel *sco_io;
- struct sockaddr_sco saddr;
- int sk, defer_setup = 1;
-
- sk = socket(PF_BLUETOOTH, SOCK_SEQPACKET | O_NONBLOCK | SOCK_CLOEXEC,
- BTPROTO_SCO);
- if (sk < 0)
- return -errno;
-
- /* Bind to local address */
- memset(&saddr, 0, sizeof(saddr));
- saddr.sco_family = AF_BLUETOOTH;
- bt_bacpy(&saddr.sco_bdaddr, BDADDR_ANY);
-
- if (bind(sk, (struct sockaddr *) &saddr, sizeof(saddr)) < 0) {
- close(sk);
- return -errno;
- }
-
- if (setsockopt(sk, SOL_BLUETOOTH, BT_DEFER_SETUP,
- &defer_setup, sizeof(defer_setup)) < 0) {
- ofono_warn("Can't enable deferred setup: %s (%d)",
- strerror(errno), errno);
- local_hfp_version = HFP_VERSION_1_5;
- }
-
- if (listen(sk, 5) < 0) {
- close(sk);
- return -errno;
- }
-
- sco_io = g_io_channel_unix_new(sk);
- sco_watch = g_io_add_watch(sco_io,
- G_IO_IN | G_IO_ERR | G_IO_HUP | G_IO_NVAL,
- sco_accept, NULL);
-
- g_io_channel_unref(sco_io);
-
- return 0;
-}
-
static void connect_handler(DBusConnection *conn, void *user_data)
{
DBG("Registering External Profile handler ...");
- bt_register_profile(conn, HFP_HS_UUID, local_hfp_version, "hfp_hf",
+ bt_register_profile(conn, HFP_HS_UUID, HFP_VERSION_1_5, "hfp_hf",
HFP_EXT_PROFILE_PATH);
}
@@ -695,12 +604,6 @@ static int hfp_init(void)
if (DBUS_TYPE_UNIX_FD < 0)
return -EBADF;
- err = sco_init();
- if (err < 0) {
- ofono_error("SCO: %s(%d)", strerror(-err), -err);
- return err;
- }
-
/* Registers External Profile handler */
if (!g_dbus_register_interface(conn, HFP_EXT_PROFILE_PATH,
BLUEZ_PROFILE_INTERFACE,
@@ -742,9 +645,6 @@ static void hfp_exit(void)
BLUEZ_PROFILE_INTERFACE);
ofono_modem_driver_unregister(&hfp_driver);
g_dbus_client_unref(bluez);
-
- if (sco_watch > 0)
- g_source_remove(sco_watch);
}
OFONO_PLUGIN_DEFINE(hfp_bluez5, "External Hands-Free Profile Plugin", VERSION,
diff --git a/src/handsfree-audio.c b/src/handsfree-audio.c
index b2d4b97..14488ac 100644
--- a/src/handsfree-audio.c
+++ b/src/handsfree-audio.c
@@ -25,12 +25,19 @@
#include <errno.h>
#include <stdio.h>
+#include <stdint.h>
#include <string.h>
+#include <fcntl.h>
+#include <unistd.h>
+#include <sys/socket.h>
#include <gdbus.h>
+#include <gatchat.h>
+#include <drivers/hfpmodem/slc.h>
#include <ofono/handsfree-audio.h>
+#include "bluetooth.h"
#include "ofono.h"
#define HFP_AUDIO_MANAGER_INTERFACE OFONO_SERVICE ".HandsfreeAudioManager"
@@ -60,6 +67,97 @@ struct agent {
static struct agent *agent = NULL;
static int ref_count = 0;
static GSList *card_list = 0;
+static guint sco_watch = 0;
+static uint16_t local_hfp_version = HFP_VERSION_1_6;
+
+static ofono_bool_t slc_match(struct ofono_modem *modem, void *userdata)
+{
+ const char *remote = userdata;
+ const char *value = ofono_modem_get_string(modem, "Remote");
+
+ if (value == NULL)
+ return FALSE;
+
+ /* Make sure SLC has been established */
+ if (ofono_modem_get_powered(modem) != TRUE)
+ return FALSE;
+
+ return g_str_equal(remote, value);
+}
+
+static gboolean sco_accept(GIOChannel *io, GIOCondition cond,
+ gpointer user_data)
+{
+ struct sockaddr_sco saddr;
+ socklen_t alen;
+ int sk, nsk;
+ char remote[18];
+
+ if (cond & (G_IO_ERR | G_IO_HUP | G_IO_NVAL))
+ return FALSE;
+
+ sk = g_io_channel_unix_get_fd(io);
+
+ memset(&saddr, 0, sizeof(saddr));
+ alen = sizeof(saddr);
+
+ nsk = accept(sk, (struct sockaddr *) &saddr, &alen);
+ if (nsk < 0)
+ return TRUE;
+
+ bt_ba2str(&saddr.sco_bdaddr, remote);
+
+ if (ofono_modem_find(slc_match, remote) == NULL) {
+ ofono_error("Rejecting SCO: No SLC connection found!");
+ close(nsk);
+ return TRUE;
+ }
+
+ return TRUE;
+}
+
+static int sco_init(void)
+{
+ GIOChannel *sco_io;
+ struct sockaddr_sco saddr;
+ int sk, defer_setup = 1;
+
+ sk = socket(PF_BLUETOOTH, SOCK_SEQPACKET | O_NONBLOCK | SOCK_CLOEXEC,
+ BTPROTO_SCO);
+ if (sk < 0)
+ return -errno;
+
+ /* Bind to local address */
+ memset(&saddr, 0, sizeof(saddr));
+ saddr.sco_family = AF_BLUETOOTH;
+ bt_bacpy(&saddr.sco_bdaddr, BDADDR_ANY);
+
+ if (bind(sk, (struct sockaddr *) &saddr, sizeof(saddr)) < 0) {
+ close(sk);
+ return -errno;
+ }
+
+ if (setsockopt(sk, SOL_BLUETOOTH, BT_DEFER_SETUP,
+ &defer_setup, sizeof(defer_setup)) < 0) {
+ ofono_warn("Can't enable deferred setup: %s (%d)",
+ strerror(errno), errno);
+ local_hfp_version = HFP_VERSION_1_5;
+ }
+
+ if (listen(sk, 5) < 0) {
+ close(sk);
+ return -errno;
+ }
+
+ sco_io = g_io_channel_unix_new(sk);
+ sco_watch = g_io_add_watch(sco_io,
+ G_IO_IN | G_IO_ERR | G_IO_HUP | G_IO_NVAL,
+ sco_accept, NULL);
+
+ g_io_channel_unref(sco_io);
+
+ return 0;
+}
static void card_append_properties(struct ofono_handsfree_card *card,
DBusMessageIter *dict)
@@ -430,7 +528,7 @@ void ofono_handsfree_audio_unref(void)
int __ofono_handsfree_audio_manager_init(void)
{
- return 0;
+ return sco_init();
}
void __ofono_handsfree_audio_manager_cleanup(void)
@@ -443,4 +541,7 @@ void __ofono_handsfree_audio_manager_cleanup(void)
ref_count = 1;
ofono_handsfree_audio_unref();
+
+ if (sco_watch > 0)
+ g_source_remove(sco_watch);
}
--
1.7.11.7
_______________________________________________
ofono mailing list
[email protected]
http://lists.ofono.org/listinfo/ofono