Bug in example light switch provisioner?

[bart]

SDK Mesh 3.1

I was looking at the provisioner example, and noticed that in the function prov_helper_device_handles_load() there is:

    ERROR_CHECK(dsm_subnet_get_all(&m_provisioner.p_dev_data->m_netkey_handle, &count));

But when i look at dsm_subnet_get_all(), the first argument should be mesh_key_index_t, not dsm_handle_t. Shouldn't the function dsm_net_key_index_to_subnet_handle() be used to get the handle from the id?

[Bjørn Kvaale]

Yes, it seems like you are correct. Or you could use the dsm_subnet_get_all() function, but use the correct first argument. I will report this internally & get back to you.

[Bjørn Kvaale]

I have confirmed with R&D that you are correct.

[bart]

So the correct way would be something like this?

uint32_t count = 1;
mesh_key_index_t key_indices[count] = {0};
ERROR_CHECK(dsm_subnet_get_all(key_indices, &count));
NRF_MESH_ASSERT(count == 1);
m_provisioner.p_dev_data->m_netkey_handle = dsm_net_key_index_to_subnet_handle(key_indices[0]);