Re: [PATCH BlueZ 1/2] mesh: Fix reading device UUID from Join() call

["Gix, Brian" <brian.gix@intel.com>]

Patch Applied

On Fri, 2019-01-11 at 18:40 -0800, Inga Stotland wrote:
> Supply correct parameters to l_dbus_message_iter_get_fixed_array
> to get the location from where to read UUID value.
> ---
>  mesh/mesh.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/mesh/mesh.c b/mesh/mesh.c
> index 169e6f42c..8971f7cc4 100644
> --- a/mesh/mesh.c
> +++ b/mesh/mesh.c
> @@ -564,6 +564,7 @@ static struct l_dbus_message *join_network_call(struct l_dbus *dbus,
>  {
>  	const char *app_path, *sender;
>  	struct l_dbus_message_iter iter_uuid;
> +	uint8_t *uuid;
>  	uint32_t n;
>  
>  	l_debug("Join network request");
> @@ -578,7 +579,7 @@ static struct l_dbus_message *join_network_call(struct l_dbus *dbus,
>  
>  	join_pending = l_new(struct join_data, 1);
>  
> -	l_dbus_message_iter_get_fixed_array(&iter_uuid, join_pending->uuid, &n);
> +	l_dbus_message_iter_get_fixed_array(&iter_uuid, &uuid, &n);
>  
>  	if (n != 16) {
>  		l_free(join_pending);
> @@ -587,6 +588,8 @@ static struct l_dbus_message *join_network_call(struct l_dbus *dbus,
>  							"Bad device UUID");
>  	}
>  
> +	memcpy(join_pending->uuid, uuid, 16);
> +
>  	sender = l_dbus_message_get_sender(msg);
>  
>  	join_pending->sender = l_strdup(sender);