Hi Michael, On 10/16/2017 10:47 PM, Michael Simon wrote: > The glib iterator was not moved to the second array, > which contains the parameter. > --- > src/gatt-client.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/src/gatt-client.c b/src/gatt-client.c > index 32b3a87..58f14b8 100644 > --- a/src/gatt-client.c > +++ b/src/gatt-client.c > @@ -596,6 +596,8 @@ static DBusMessage > *descriptor_write_value(DBusConnection *conn, > if (parse_value_arg(&iter, &value, &value_len)) > return btd_error_invalid_args(msg); > > + dbus_message_iter_next (&iter); > + > if (parse_options(&iter, &offset)) > return btd_error_invalid_args(msg); > > @@ -986,6 +988,8 @@ static DBusMessage > *characteristic_write_value(DBusConnection *conn, > if (parse_value_arg(&iter, &value, &value_len)) > return btd_error_invalid_args(msg); > > + dbus_message_iter_next (&iter); > + > if (parse_options(&iter, &offset)) > return btd_error_invalid_args(msg); > This patch can not be applied, and the following messages output: Applying: dbus: Fix for unused parameter in WriteValue fatal: corrupt patch at line 10 Patch failed at 0001 dbus: Fix for unused parameter in WriteValue Since this patch perhaps has extra line feeds and uses spaces as indent. Regards, Eramoto -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
