Hi Grzegorz,
On Wednesday, 28 February 2018 13:41:21 CET Grzegorz Kolodziejczyk wrote:
> Included values should be string type instead of address of pointer to
> string.
> ---
> tools/btpclient.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/tools/btpclient.c b/tools/btpclient.c
> index 61bde820e..d37232f07 100644
> --- a/tools/btpclient.c
> +++ b/tools/btpclient.c
> @@ -888,19 +888,19 @@ static bool ad_includes_getter(struct l_dbus *dbus,
> if (ad.tx_power) {
> const char *str = "tx-power";
>
> - l_dbus_message_builder_append_basic(builder, 's', &str);
> + l_dbus_message_builder_append_basic(builder, 's', str);
> }
>
> if (ad.name) {
> const char *str = "local-name";
>
> - l_dbus_message_builder_append_basic(builder, 's', &str);
> + l_dbus_message_builder_append_basic(builder, 's', str);
> }
>
> if (ad.appearance) {
> const char *str = "appearance";
>
> - l_dbus_message_builder_append_basic(builder, 's', &str);
> + l_dbus_message_builder_append_basic(builder, 's', str);
> }
>
> l_dbus_message_builder_leave_array(builder);
All patches applied (after fixing commit message in patch 5 and dead code in
last patch). Thanks.
--
pozdrawiam
Szymon Janc
--
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
