alex/esp-rainmaker
1
0
Fork 0
mirror of https://github.com/espressif/esp-rainmaker.git synced 2026-01-15 08:19:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'bugfix/random_bytes_logic' into 'master'

Browse Source 
wifi_prov: Fixed a bug which was causing name/pop to change on every reboot

See merge request app-frameworks/esp-rainmaker!218
This commit is contained in:
Piyush Shah
2021-01-08 18:00:20 +08:00
parent 3a0621dccf 0ad17a96e8
commit 012aadf8f4
1 changed files with 16 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
components/esp_rainmaker/src/core/esp_rmaker_claim.c
View File

@@ -788,8 +788,7 @@ static void event_handler(void* arg, esp_event_base_t event_base,
#endif /* CONFIG_ESP_RMAKER_ASSISTED_CLAIM */
esp_err_t __esp_rmaker_claim_init(esp_rmaker_claim_data_t *claim_data)
{
uint8_t random_bytes[ESP_RMAKER_RANDOM_NUMBER_LEN];
esp_err_t err;
esp_err_t err = ESP_OK;
char *key = esp_rmaker_get_client_key();
if (key) {
@@ -814,14 +813,22 @@ esp_err_t __esp_rmaker_claim_init(esp_rmaker_claim_data_t *claim_data)
return err;
}
}
/* Generate random bytes for general purpose use */
esp_fill_random(&random_bytes, sizeof(random_bytes));
/* Check if the general purpose random bytes are already present in the storage */
char *stored_random_bytes = esp_rmaker_storage_get(ESP_RMAKER_CLIENT_RANDOM_NVS_KEY);
if (stored_random_bytes == NULL) {
/* Generate random bytes for general purpose use */
uint8_t random_bytes[ESP_RMAKER_RANDOM_NUMBER_LEN];
esp_fill_random(&random_bytes, sizeof(random_bytes));
/* Store the PoP in the storage */
err = esp_rmaker_storage_set(ESP_RMAKER_CLIENT_RANDOM_NVS_KEY, random_bytes, sizeof(random_bytes));
if (err != ESP_OK) {
ESP_LOGE(TAG, "Failed to store random bytes to storage.");
return err;
/* Store the PoP in the storage */
err = esp_rmaker_storage_set(ESP_RMAKER_CLIENT_RANDOM_NVS_KEY, random_bytes, sizeof(random_bytes));
if (err != ESP_OK) {
ESP_LOGE(TAG, "Failed to store random bytes to storage.");
return err;
}
} else {
/* Free the copy of the random bytes as it isn't required here. */
free(stored_random_bytes);
}
#ifdef CONFIG_ESP_RMAKER_SELF_CLAIM
err = esp_rmaker_claim_generate_csr(claim_data, esp_rmaker_get_node_id());