On Fri, Jul 19, 2013 at 8:12 AM, Tanu Kaskinen
<tanu.kaskinen at linux.intel.com> wrote:
> On Fri, 2013-07-12 at 15:06 -0300, jprvita at gmail.com wrote:
>> From: Jo?o Paulo Rechi Vita <jprvita at openbossa.org>
>>
>> ---
>> src/modules/bluetooth/bluez5-util.c | 116 +++++++++++++++++++++++++++++++++++-
>> src/modules/bluetooth/bluez5-util.h | 3 +
>> 2 files changed, 117 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/modules/bluetooth/bluez5-util.c b/src/modules/bluetooth/bluez5-util.c
>> index 3e6eb1c..a68f8ca 100644
>> --- a/src/modules/bluetooth/bluez5-util.c
>> +++ b/src/modules/bluetooth/bluez5-util.c
>> @@ -350,11 +350,123 @@ fail:
>> }
>>
>> static DBusMessage *endpoint_set_configuration(DBusConnection *conn, DBusMessage *m, void *userdata) {
>> + pa_bluetooth_discovery *y = userdata;
>> + pa_bluetooth_device *d;
>> + pa_bluetooth_transport *t;
>> + const char *sender, *path, *endpoint_path, *dev_path = NULL, *uuid = NULL;
>> + uint8_t *config = NULL;
>> + int size = 0;
>> + pa_bluetooth_profile_t p = PROFILE_OFF;
>> + DBusMessageIter args, props;
>> DBusMessage *r;
>> + bool old_any_connected;
>>
>> - pa_assert_se(r = dbus_message_new_error(m, BLUEZ_MEDIA_ENDPOINT_INTERFACE ".Error.NotImplemented",
>> - "Method not implemented"));
>> + if (!dbus_message_iter_init(m, &args) || !pa_streq(dbus_message_get_signature(m), "oa{sv}")) {
>> + pa_log_error("Invalid signature for method SetConfiguration()");
>> + goto fail2;
>> + }
>> +
>> + dbus_message_iter_get_basic(&args, &path);
>> +
>> + if (pa_hashmap_get(y->transports, path)) {
>> + pa_log_error("Endpoint SetConfiguration(): Transport %s is already configured.", path);
>> + goto fail2;
>> + }
>> +
>> + pa_assert_se(dbus_message_iter_next(&args));
>> +
>> + dbus_message_iter_recurse(&args, &props);
>> + if (dbus_message_iter_get_arg_type(&props) != DBUS_TYPE_DICT_ENTRY)
>> + goto fail;
>> +
>> + /* Read transport properties */
>> + while (dbus_message_iter_get_arg_type(&props) == DBUS_TYPE_DICT_ENTRY) {
>> + const char *key;
>> + DBusMessageIter value, entry;
>> + int var;
>> +
>> + dbus_message_iter_recurse(&props, &entry);
>> + dbus_message_iter_get_basic(&entry, &key);
>> +
>> + dbus_message_iter_next(&entry);
>> + dbus_message_iter_recurse(&entry, &value);
>> +
>> + var = dbus_message_iter_get_arg_type(&value);
>> +
>> + if (strcasecmp(key, "UUID") == 0) {
>> + if (var != DBUS_TYPE_STRING)
>> + goto fail;
>> +
>> + dbus_message_iter_get_basic(&value, &uuid);
>> + } else if (strcasecmp(key, "Device") == 0) {
>> + if (var != DBUS_TYPE_OBJECT_PATH)
>> + goto fail;
>> +
>> + dbus_message_iter_get_basic(&value, &dev_path);
>> + } else if (strcasecmp(key, "Configuration") == 0) {
>> + DBusMessageIter array;
>> + if (var != DBUS_TYPE_ARRAY)
>> + goto fail;
>> +
>> + dbus_message_iter_recurse(&value, &array);
>> + dbus_message_iter_get_fixed_array(&array, &config, &size);
>
> I think it would be good to validate the received configuration here,
> that is, check that it looks sane before we try to use the
> configuration.
>
I think I've already mentioned in another reply, but I'll fix that.
--
Jo?o Paulo Rechi Vita
http://about.me/jprvita
