diff --git a/.github/workflows/build_and_run_test_app.yml b/.github/workflows/build_and_run_test_app.yml index a554f74..e3da25e 100644 --- a/.github/workflows/build_and_run_test_app.yml +++ b/.github/workflows/build_and_run_test_app.yml @@ -72,12 +72,10 @@ jobs: idf_target: esp32s3 # ESP32S3 support started with version 4.4 runs-on: [self-hosted, linux, docker, esp32] # Unfortunately `${{ matrix.idf_target }}` is not accepted here container: - image: python:3.7-slim-buster + image: python:3.7-buster options: --privileged # Privileged mode has access to serial ports steps: - - uses: actions/checkout@v2 - with: - fetch-depth: 0 + - uses: actions/checkout@v3 - uses: actions/download-artifact@v2 with: name: test_app_bin_${{ matrix.idf_target }}_${{ matrix.idf_ver }} diff --git a/.github/workflows/pre-commit.yml b/.github/workflows/pre-commit.yml index 8174cc1..739c974 100644 --- a/.github/workflows/pre-commit.yml +++ b/.github/workflows/pre-commit.yml @@ -8,8 +8,6 @@ jobs: pre-commit: runs-on: ubuntu-latest steps: - - uses: actions/checkout@v2 - with: - fetch-depth: 0 + - uses: actions/checkout@v3 - uses: actions/setup-python@v2 - uses: pre-commit/action@v2.0.3 diff --git a/esp_encrypted_img/idf_component.yml b/esp_encrypted_img/idf_component.yml index 86f8397..6de2e05 100644 --- a/esp_encrypted_img/idf_component.yml +++ b/esp_encrypted_img/idf_component.yml @@ -1,4 +1,4 @@ -version: "1.0.2" +version: "2.0.0" description: ESP Encrypted Image Abstraction Layer url: https://github.com/espressif/idf-extra-components/tree/master/esp_encrypted_img dependencies: diff --git a/esp_encrypted_img/include/esp_encrypted_img.h b/esp_encrypted_img/include/esp_encrypted_img.h index 49418ba..ad184dc 100644 --- a/esp_encrypted_img/include/esp_encrypted_img.h +++ b/esp_encrypted_img/include/esp_encrypted_img.h @@ -86,7 +86,7 @@ esp_decrypt_handle_t esp_encrypted_img_decrypt_start(const esp_decrypt_cfg_t *cf * - ESP_ERR_DECRYPT_IN_PROGRESS Decryption is in process * - ESP_OK Success */ -esp_err_t esp_encrypted_img_decrypt_data(esp_decrypt_handle_t *ctx, pre_enc_decrypt_arg_t *args); +esp_err_t esp_encrypted_img_decrypt_data(esp_decrypt_handle_t ctx, pre_enc_decrypt_arg_t *args); /** @@ -98,7 +98,7 @@ esp_err_t esp_encrypted_img_decrypt_data(esp_decrypt_handle_t *ctx, pre_enc_decr * - ESP_FAIL On failure * - ESP_OK */ -esp_err_t esp_encrypted_img_decrypt_end(esp_decrypt_handle_t *ctx); +esp_err_t esp_encrypted_img_decrypt_end(esp_decrypt_handle_t ctx); #ifdef __cplusplus diff --git a/esp_encrypted_img/src/esp_encrypted_img.c b/esp_encrypted_img/src/esp_encrypted_img.c index 6110cec..16c2582 100644 --- a/esp_encrypted_img/src/esp_encrypted_img.c +++ b/esp_encrypted_img/src/esp_encrypted_img.c @@ -1,481 +1,456 @@ -/* - * SPDX-FileCopyrightText: 2015-2022 Espressif Systems (Shanghai) CO LTD - * - * SPDX-License-Identifier: Apache-2.0 - */ - -#include -#include "esp_encrypted_img.h" -#include -#include -#include - -#include "mbedtls/version.h" -#include "mbedtls/pk.h" -#include "mbedtls/entropy.h" -#include "mbedtls/ctr_drbg.h" -#include "mbedtls/gcm.h" -#include "sys/param.h" - -static const char *TAG = "esp_encrypted_img"; - -typedef enum { - ESP_PRE_ENC_IMG_READ_MAGIC, - ESP_PRE_ENC_IMG_READ_GCM, - ESP_PRE_ENC_IMG_READ_IV, - ESP_PRE_ENC_IMG_READ_BINSIZE, - ESP_PRE_ENC_IMG_READ_AUTH, - ESP_PRE_ENC_IMG_READ_EXTRA_HEADER, - ESP_PRE_ENC_DATA_DECODE_STATE, -} esp_encrypted_img_state; - -struct esp_encrypted_img_handle { - const char *rsa_pem; - size_t rsa_len; - uint32_t binary_file_len; - uint32_t binary_file_read; - char *gcm_key; - char *iv; - char auth_tag[16]; - esp_encrypted_img_state state; - mbedtls_gcm_context gcm_ctx; - size_t cache_buf_len; - char *cache_buf; -}; - -#define GCM_KEY_SIZE 32 -#define MAGIC_SIZE 4 -#define ENC_GCM_KEY_SIZE 384 -#define IV_SIZE 16 -#define BIN_SIZE_DATA 4 -#define AUTH_SIZE 16 -#define RESERVED_HEADER 88 - -typedef struct { - char magic[MAGIC_SIZE]; - char enc_gcm[ENC_GCM_KEY_SIZE]; - char iv[IV_SIZE]; - char bin_size[BIN_SIZE_DATA]; - char auth[AUTH_SIZE]; - char extra_header[RESERVED_HEADER]; -} pre_enc_bin_header; -#define HEADER_DATA_SIZE sizeof(pre_enc_bin_header) - -// Magic Byte is created using command: echo -n "esp_encrypted_img" | sha256sum -static uint32_t esp_enc_img_magic = 0x0788b6cf; - -typedef struct esp_encrypted_img_handle esp_encrypted_img_t; - -static int decipher_gcm_key(char *enc_gcm, esp_encrypted_img_t *handle) -{ - int ret = 1; - handle->gcm_key = calloc(1, GCM_KEY_SIZE); - if (!handle->gcm_key) { - return ESP_ERR_NO_MEM; - } - size_t olen = 0; - mbedtls_pk_context pk; - mbedtls_entropy_context entropy; - mbedtls_ctr_drbg_context ctr_drbg; - const char *pers = "mbedtls_pk_encrypt"; - - mbedtls_ctr_drbg_init( &ctr_drbg ); - mbedtls_entropy_init( &entropy ); - mbedtls_pk_init( &pk ); - - if ((ret = mbedtls_ctr_drbg_seed( &ctr_drbg, mbedtls_entropy_func, - &entropy, (const unsigned char *) pers, - strlen(pers))) != 0) { - ESP_LOGE(TAG, "failed\n ! mbedtls_ctr_drbg_seed returned -0x%04x\n", (unsigned int) - ret); - free(handle->gcm_key); - goto exit; - } - - ESP_LOGI(TAG, "Reading RSA private key"); - -#if (MBEDTLS_VERSION_NUMBER < 0x03000000) - if ( (ret = mbedtls_pk_parse_key(&pk, (const unsigned char *) handle->rsa_pem, handle->rsa_len, NULL, 0)) != 0) { -#else - if ( (ret = mbedtls_pk_parse_key(&pk, (const unsigned char *) handle->rsa_pem, handle->rsa_len, NULL, 0, mbedtls_ctr_drbg_random, &ctr_drbg)) != 0) { -#endif - ESP_LOGE(TAG, "failed\n ! mbedtls_pk_parse_keyfile returned -0x%04x\n", (unsigned int) - ret ); - free(handle->gcm_key); - goto exit; - } - - if (( ret = mbedtls_pk_decrypt( &pk, (const unsigned char *)enc_gcm, ENC_GCM_KEY_SIZE, (unsigned char *)handle->gcm_key, &olen, GCM_KEY_SIZE, - mbedtls_ctr_drbg_random, &ctr_drbg ) ) != 0 ) { - ESP_LOGE(TAG, "failed\n ! mbedtls_pk_decrypt returned -0x%04x\n", (unsigned int) - ret ); - free(handle->gcm_key); - goto exit; - } - handle->cache_buf = realloc(handle->cache_buf, 16); - if (!handle->cache_buf) { - return ESP_ERR_NO_MEM; - } - handle->state = ESP_PRE_ENC_IMG_READ_IV; - handle->binary_file_read = 0; - handle->cache_buf_len = 0; - handle->iv = calloc(1, IV_SIZE); - if (!handle->iv) { - return ESP_ERR_NO_MEM; - } - -exit: - mbedtls_pk_free( &pk ); - mbedtls_entropy_free( &entropy ); - mbedtls_ctr_drbg_free( &ctr_drbg ); - free((void *)handle->rsa_pem); - - return (ret); -} - -esp_decrypt_handle_t esp_encrypted_img_decrypt_start(const esp_decrypt_cfg_t *cfg) -{ - if (cfg == NULL || cfg->rsa_pub_key == NULL) { - ESP_LOGE(TAG, "esp_encrypted_img_decrypt_start : Invalid argument"); - return NULL; - } - ESP_LOGI(TAG, "Starting Decryption Process"); - - esp_encrypted_img_t *handle = calloc(1, sizeof(esp_encrypted_img_t)); - if (!handle) { - ESP_LOGE(TAG, "Couldn't allocate memory to handle"); - goto failure; - } - - handle->rsa_pem = calloc(1, cfg->rsa_pub_key_len); - if (!handle->rsa_pem) { - ESP_LOGE(TAG, "Couldn't allocate memory to handle->rsa_pem"); - goto failure; - } - - handle->cache_buf = calloc(1, ENC_GCM_KEY_SIZE); - if (!handle->cache_buf) { - ESP_LOGE(TAG, "Couldn't allocate memory to handle->cache_buf"); - goto failure; - } - - memcpy((void *)handle->rsa_pem, cfg->rsa_pub_key, cfg->rsa_pub_key_len); - handle->rsa_len = cfg->rsa_pub_key_len; - handle->state = ESP_PRE_ENC_IMG_READ_MAGIC; - - esp_decrypt_handle_t *ctx = (esp_decrypt_handle_t *)handle; - return ctx; - -failure: - if (!handle) { - return NULL; - } - if (handle->rsa_pem) { - free((void *)handle->rsa_pem); - } - if (handle) { - free(handle); - } - return NULL; -} - -static esp_err_t process_bin(esp_encrypted_img_t *handle, pre_enc_decrypt_arg_t *args, int curr_index) -{ - size_t data_len = args->data_in_len; - size_t data_out_size = args->data_out_len; -#if !(MBEDTLS_VERSION_NUMBER < 0x03000000) - size_t olen; -#endif - handle->binary_file_read += data_len - curr_index; - int dec_len = 0; - if (handle->binary_file_read != handle->binary_file_len) { - size_t copy_len = 0; - - if ((handle->cache_buf_len + (data_len - curr_index)) - (handle->cache_buf_len + (data_len - curr_index)) % 16 > 0) { - data_out_size = (handle->cache_buf_len + (data_len - curr_index)) - (handle->cache_buf_len + (data_len - curr_index)) % 16; - args->data_out = realloc(args->data_out, data_out_size); - if (!args->data_out) { - return ESP_ERR_NO_MEM; - } - } - if (handle->cache_buf_len != 0) { - copy_len = MIN(16 - handle->cache_buf_len, data_len - curr_index); - memcpy(handle->cache_buf + handle->cache_buf_len, args->data_in + curr_index, copy_len); - handle->cache_buf_len += copy_len; - if (handle->cache_buf_len != 16) { - args->data_out_len = 0; - return ESP_ERR_NOT_FINISHED; - } -#if (MBEDTLS_VERSION_NUMBER < 0x03000000) - if (mbedtls_gcm_update(&handle->gcm_ctx, 16, (const unsigned char *)handle->cache_buf, (unsigned char *) args->data_out) != 0) { -#else - if (mbedtls_gcm_update(&handle->gcm_ctx, (const unsigned char *)handle->cache_buf, 16, (unsigned char *) args->data_out, data_out_size, &olen) != 0) { -#endif - return ESP_FAIL; - } - dec_len = 16; - } - handle->cache_buf_len = (data_len - curr_index - copy_len) % 16; - if (handle->cache_buf_len != 0) { - data_len -= handle->cache_buf_len; - memcpy(handle->cache_buf, args->data_in + (data_len), handle->cache_buf_len); - } - - if (data_len - copy_len - curr_index > 0) { -#if (MBEDTLS_VERSION_NUMBER < 0x03000000) - if (mbedtls_gcm_update(&handle->gcm_ctx, data_len - copy_len - curr_index, (const unsigned char *)args->data_in + curr_index + copy_len, (unsigned char *)args->data_out + dec_len) != 0) { -#else - if (mbedtls_gcm_update(&handle->gcm_ctx, (const unsigned char *)args->data_in + curr_index + copy_len, data_len - copy_len - curr_index, (unsigned char *)args->data_out + dec_len, data_out_size - dec_len, &olen) != 0) { -#endif - return ESP_FAIL; - } - } - args->data_out_len = dec_len + data_len - curr_index - copy_len; - return ESP_ERR_NOT_FINISHED; - } - data_out_size = handle->cache_buf_len + data_len - curr_index; - args->data_out = realloc(args->data_out, data_out_size); - if (!args->data_out) { - return ESP_ERR_NO_MEM; - } - size_t copy_len = 0; - - copy_len = MIN(16 - handle->cache_buf_len, data_len - curr_index); - memcpy(handle->cache_buf + handle->cache_buf_len, args->data_in + curr_index, copy_len); - handle->cache_buf_len += copy_len; -#if (MBEDTLS_VERSION_NUMBER < 0x03000000) - if (mbedtls_gcm_update(&handle->gcm_ctx, handle->cache_buf_len, (const unsigned char *)handle->cache_buf, (unsigned char *)args->data_out) != 0) { -#else - if (mbedtls_gcm_update(&handle->gcm_ctx, (const unsigned char *)handle->cache_buf, handle->cache_buf_len, (unsigned char *)args->data_out, data_out_size, &olen) != 0) { -#endif - return ESP_FAIL; - } - if (data_len - curr_index - copy_len > 0) { -#if (MBEDTLS_VERSION_NUMBER < 0x03000000) - if (mbedtls_gcm_update(&handle->gcm_ctx, data_len - curr_index - copy_len, (const unsigned char *)(args->data_in + curr_index + copy_len), (unsigned char *)(args->data_out + 16)) != 0) { -#else - if (mbedtls_gcm_update(&handle->gcm_ctx, (const unsigned char *)(args->data_in + curr_index + copy_len), data_len - curr_index - copy_len, (unsigned char *)(args->data_out + 16), data_out_size - 16, &olen) != 0) { -#endif - return ESP_FAIL; - } - } - - args->data_out_len = handle->cache_buf_len + data_len - copy_len - curr_index; - handle->cache_buf_len = 0; - - return ESP_OK; -} - -static void read_and_cache_data(esp_encrypted_img_t *handle, pre_enc_decrypt_arg_t *args, int *curr_index, int data_size) -{ - const int data_left = data_size - handle->binary_file_read; - const int data_recv = args->data_in_len - *curr_index; - if (handle->state == ESP_PRE_ENC_IMG_READ_IV && handle->iv) { - memcpy(handle->iv + handle->cache_buf_len, args->data_in + *curr_index, MIN(data_recv, data_left)); - } else if (handle->state == ESP_PRE_ENC_IMG_READ_AUTH) { - memcpy(handle->auth_tag + handle->cache_buf_len, args->data_in + *curr_index, MIN(data_recv, data_left)); - } else { - memcpy(handle->cache_buf + handle->cache_buf_len, args->data_in + *curr_index, MIN(data_recv, data_left)); - } - handle->cache_buf_len += MIN(data_recv, data_left); - int temp = *curr_index; - *curr_index += MIN(data_recv, data_left); - handle->binary_file_read += MIN(args->data_in_len - temp, data_left); -} - -esp_err_t esp_encrypted_img_decrypt_data(esp_decrypt_handle_t *ctx, pre_enc_decrypt_arg_t *args) -{ - if (ctx == NULL || args == NULL || args->data_in == NULL) { - return ESP_ERR_INVALID_ARG; - } - esp_encrypted_img_t *handle = (esp_encrypted_img_t *)ctx; - if (handle == NULL) { - ESP_LOGE(TAG, "esp_encrypted_img_decrypt_data: Invalid argument"); - return ESP_ERR_INVALID_ARG; - } - - esp_err_t err; - int curr_index = 0; - - switch (handle->state) { - case ESP_PRE_ENC_IMG_READ_MAGIC: - if (handle->cache_buf_len == 0 && (args->data_in_len - curr_index) >= MAGIC_SIZE) { - uint32_t recv_magic = *(uint32_t *)args->data_in; - - if (recv_magic != esp_enc_img_magic) { - ESP_LOGE(TAG, "Magic Verification failed"); - free((void *)handle->rsa_pem); - return ESP_FAIL; - } - curr_index += MAGIC_SIZE; - } else { - read_and_cache_data(handle, args, &curr_index, MAGIC_SIZE); - if (handle->binary_file_read == MAGIC_SIZE) { - uint32_t recv_magic = *(uint32_t *)handle->cache_buf; - - if (recv_magic != esp_enc_img_magic) { - ESP_LOGE(TAG, "Magic Verification failed"); - free((void *)handle->rsa_pem); - return ESP_FAIL; - } - handle->binary_file_read = 0; - handle->cache_buf_len = 0; - } else { - return ESP_ERR_NOT_FINISHED; - } - } - ESP_LOGI(TAG, "Magic Verified"); - handle->state = ESP_PRE_ENC_IMG_READ_GCM; - /* falls through */ - case ESP_PRE_ENC_IMG_READ_GCM: - if (handle->cache_buf_len == 0 && args->data_in_len - curr_index >= ENC_GCM_KEY_SIZE) { - if (decipher_gcm_key(args->data_in + curr_index, handle) != 0) { - ESP_LOGE(TAG, "Unable to decipher GCM key"); - return ESP_FAIL; - } - curr_index += ENC_GCM_KEY_SIZE; - } else { - read_and_cache_data(handle, args, &curr_index, ENC_GCM_KEY_SIZE); - if (handle->cache_buf_len == ENC_GCM_KEY_SIZE) { - if (decipher_gcm_key(handle->cache_buf, handle) != 0) { - ESP_LOGE(TAG, "Unable to decipher GCM key"); - return ESP_FAIL; - } - } else { - return ESP_ERR_NOT_FINISHED; - } - } - /* falls through */ - case ESP_PRE_ENC_IMG_READ_IV: - if (handle->cache_buf_len == 0 && args->data_in_len - curr_index >= IV_SIZE) { - memcpy(handle->iv, args->data_in + curr_index, IV_SIZE); - handle->binary_file_read = IV_SIZE; - curr_index += IV_SIZE; - } else { - read_and_cache_data(handle, args, &curr_index, IV_SIZE); - } - if (handle->binary_file_read == IV_SIZE) { - handle->state = ESP_PRE_ENC_IMG_READ_BINSIZE; - handle->binary_file_read = 0; - handle->cache_buf_len = 0; - mbedtls_gcm_init(&handle->gcm_ctx); - if ((err = mbedtls_gcm_setkey(&handle->gcm_ctx, MBEDTLS_CIPHER_ID_AES, (const unsigned char *)handle->gcm_key, GCM_KEY_SIZE * 8)) != 0) { - ESP_LOGE(TAG, "Error: mbedtls_gcm_set_key: -0x%04x\n", (unsigned int) - err); - return ESP_FAIL; - } - free(handle->gcm_key); -#if (MBEDTLS_VERSION_NUMBER < 0x03000000) - if (mbedtls_gcm_starts(&handle->gcm_ctx, MBEDTLS_GCM_DECRYPT, (const unsigned char *)handle->iv, IV_SIZE, NULL, 0) != 0) { -#else - if (mbedtls_gcm_starts(&handle->gcm_ctx, MBEDTLS_GCM_DECRYPT, (const unsigned char *)handle->iv, IV_SIZE) != 0) { -#endif - ESP_LOGE(TAG, "Error: mbedtls_gcm_starts: -0x%04x\n", (unsigned int) - err); - return ESP_FAIL; - } - free(handle->iv); - handle->iv = NULL; - } else { - return ESP_ERR_NOT_FINISHED; - } - /* falls through */ - case ESP_PRE_ENC_IMG_READ_BINSIZE: - if (handle->cache_buf_len == 0 && (args->data_in_len - curr_index) >= BIN_SIZE_DATA) { - handle->binary_file_len = *(uint32_t *)(args->data_in + curr_index); - curr_index += BIN_SIZE_DATA; - } else { - read_and_cache_data(handle, args, &curr_index, BIN_SIZE_DATA); - if (handle->binary_file_read == BIN_SIZE_DATA) { - handle->binary_file_len = *(uint32_t *)handle->cache_buf; - } else { - return ESP_ERR_NOT_FINISHED; - } - } - handle->state = ESP_PRE_ENC_IMG_READ_AUTH; - handle->binary_file_read = 0; - handle->cache_buf_len = 0; - /* falls through */ - case ESP_PRE_ENC_IMG_READ_AUTH: - if (handle->cache_buf_len == 0 && args->data_in_len - curr_index >= AUTH_SIZE) { - memcpy(handle->auth_tag, args->data_in + curr_index, AUTH_SIZE); - handle->binary_file_read = AUTH_SIZE; - curr_index += AUTH_SIZE; - } else { - read_and_cache_data(handle, args, &curr_index, AUTH_SIZE); - } - if (handle->binary_file_read == AUTH_SIZE) { - handle->state = ESP_PRE_ENC_IMG_READ_EXTRA_HEADER; - handle->binary_file_read = 0; - handle->cache_buf_len = 0; - } else { - return ESP_ERR_NOT_FINISHED; - } - /* falls through */ - case ESP_PRE_ENC_IMG_READ_EXTRA_HEADER: { - int temp = curr_index; - curr_index += MIN(args->data_in_len - curr_index, RESERVED_HEADER - handle->binary_file_read); - handle->binary_file_read += MIN(args->data_in_len - temp, RESERVED_HEADER - handle->binary_file_read); - if (handle->binary_file_read == RESERVED_HEADER) { - handle->state = ESP_PRE_ENC_DATA_DECODE_STATE; - handle->binary_file_read = 0; - handle->cache_buf_len = 0; - } else { - return ESP_ERR_NOT_FINISHED; - } - } -/* falls through */ - case ESP_PRE_ENC_DATA_DECODE_STATE: - err = process_bin(handle, args, curr_index); - return err; - } - return ESP_OK; -} - -esp_err_t esp_encrypted_img_decrypt_end(esp_decrypt_handle_t *ctx) -{ - if (ctx == NULL) { - return ESP_ERR_INVALID_ARG; - } - esp_encrypted_img_t *handle = (esp_encrypted_img_t *)ctx; - esp_err_t err = ESP_OK; - if (handle == NULL) { - ESP_LOGE(TAG, "esp_encrypted_img_decrypt_data: Invalid argument"); - return ESP_ERR_INVALID_ARG; - } - if (handle->state == ESP_PRE_ENC_DATA_DECODE_STATE) { - if (handle->cache_buf_len != 0 || handle->binary_file_read != handle->binary_file_len) { - ESP_LOGE(TAG, "Invalid operation"); - err = ESP_FAIL; - goto exit; - } - - char *got_auth = calloc(1, AUTH_SIZE); - if (!got_auth) { - ESP_LOGE(TAG, "Unable to allocate memory"); - err = ESP_FAIL; - goto exit; - } -#if (MBEDTLS_VERSION_NUMBER < 0x03000000) - err = mbedtls_gcm_finish(&handle->gcm_ctx, (unsigned char *)got_auth, AUTH_SIZE); -#else - size_t olen; - err = mbedtls_gcm_finish(&handle->gcm_ctx, NULL, 0, &olen, (unsigned char *)got_auth, AUTH_SIZE); -#endif - if (err != 0) { - ESP_LOGE(TAG, "Error: %d", err); - free(got_auth); - err = ESP_FAIL; - goto exit; - } - if (memcmp(got_auth, handle->auth_tag, AUTH_SIZE) != 0) { - ESP_LOGE(TAG, "Invalid Auth"); - free(got_auth); - err = ESP_FAIL; - goto exit; - } - - free(got_auth); - } - err = ESP_OK; -exit: - mbedtls_gcm_free(&handle->gcm_ctx); - free(handle->cache_buf); - free(handle); - return err; -} +/* + * SPDX-FileCopyrightText: 2015-2022 Espressif Systems (Shanghai) CO LTD + * + * SPDX-License-Identifier: Apache-2.0 + */ + +#include +#include "esp_encrypted_img.h" +#include +#include +#include + +#include "mbedtls/version.h" +#include "mbedtls/pk.h" +#include "mbedtls/entropy.h" +#include "mbedtls/ctr_drbg.h" +#include "mbedtls/gcm.h" +#include "sys/param.h" + +static const char *TAG = "esp_encrypted_img"; + +typedef enum { + ESP_PRE_ENC_IMG_READ_MAGIC, + ESP_PRE_ENC_IMG_READ_GCM, + ESP_PRE_ENC_IMG_READ_IV, + ESP_PRE_ENC_IMG_READ_BINSIZE, + ESP_PRE_ENC_IMG_READ_AUTH, + ESP_PRE_ENC_IMG_READ_EXTRA_HEADER, + ESP_PRE_ENC_DATA_DECODE_STATE, +} esp_encrypted_img_state; + +#define GCM_KEY_SIZE 32 +#define MAGIC_SIZE 4 +#define ENC_GCM_KEY_SIZE 384 +#define IV_SIZE 16 +#define BIN_SIZE_DATA 4 +#define AUTH_SIZE 16 +#define RESERVED_HEADER 88 + +struct esp_encrypted_img_handle { + char *rsa_pem; + size_t rsa_len; + uint32_t binary_file_len; + uint32_t binary_file_read; + char gcm_key[GCM_KEY_SIZE]; + char iv[IV_SIZE]; + char auth_tag[AUTH_SIZE]; + esp_encrypted_img_state state; + mbedtls_gcm_context gcm_ctx; + size_t cache_buf_len; + char *cache_buf; +}; + +typedef struct { + char magic[MAGIC_SIZE]; + char enc_gcm[ENC_GCM_KEY_SIZE]; + char iv[IV_SIZE]; + char bin_size[BIN_SIZE_DATA]; + char auth[AUTH_SIZE]; + char extra_header[RESERVED_HEADER]; +} pre_enc_bin_header; +#define HEADER_DATA_SIZE sizeof(pre_enc_bin_header) + +// Magic Byte is created using command: echo -n "esp_encrypted_img" | sha256sum +static uint32_t esp_enc_img_magic = 0x0788b6cf; + +typedef struct esp_encrypted_img_handle esp_encrypted_img_t; + +static int decipher_gcm_key(char *enc_gcm, esp_encrypted_img_t *handle) +{ + int ret = 1; + size_t olen = 0; + mbedtls_pk_context pk; + mbedtls_entropy_context entropy; + mbedtls_ctr_drbg_context ctr_drbg; + const char *pers = "mbedtls_pk_encrypt"; + + mbedtls_ctr_drbg_init( &ctr_drbg ); + mbedtls_entropy_init( &entropy ); + mbedtls_pk_init( &pk ); + + if ((ret = mbedtls_ctr_drbg_seed( &ctr_drbg, mbedtls_entropy_func, + &entropy, (const unsigned char *) pers, + strlen(pers))) != 0) { + ESP_LOGE(TAG, "failed\n ! mbedtls_ctr_drbg_seed returned -0x%04x\n", (unsigned int) - ret); + goto exit; + } + + ESP_LOGI(TAG, "Reading RSA private key"); + +#if (MBEDTLS_VERSION_NUMBER < 0x03000000) + if ( (ret = mbedtls_pk_parse_key(&pk, (const unsigned char *) handle->rsa_pem, handle->rsa_len, NULL, 0)) != 0) { +#else + if ( (ret = mbedtls_pk_parse_key(&pk, (const unsigned char *) handle->rsa_pem, handle->rsa_len, NULL, 0, mbedtls_ctr_drbg_random, &ctr_drbg)) != 0) { +#endif + ESP_LOGE(TAG, "failed\n ! mbedtls_pk_parse_keyfile returned -0x%04x\n", (unsigned int) - ret ); + goto exit; + } + + if (( ret = mbedtls_pk_decrypt( &pk, (const unsigned char *)enc_gcm, ENC_GCM_KEY_SIZE, (unsigned char *)handle->gcm_key, &olen, GCM_KEY_SIZE, + mbedtls_ctr_drbg_random, &ctr_drbg ) ) != 0 ) { + ESP_LOGE(TAG, "failed\n ! mbedtls_pk_decrypt returned -0x%04x\n", (unsigned int) - ret ); + goto exit; + } + handle->cache_buf = realloc(handle->cache_buf, 16); + if (!handle->cache_buf) { + return ESP_ERR_NO_MEM; + } + handle->state = ESP_PRE_ENC_IMG_READ_IV; + handle->binary_file_read = 0; + handle->cache_buf_len = 0; +exit: + mbedtls_pk_free( &pk ); + mbedtls_entropy_free( &entropy ); + mbedtls_ctr_drbg_free( &ctr_drbg ); + free(handle->rsa_pem); + handle->rsa_pem = NULL; + + return (ret); +} + +esp_decrypt_handle_t esp_encrypted_img_decrypt_start(const esp_decrypt_cfg_t *cfg) +{ + if (cfg == NULL || cfg->rsa_pub_key == NULL) { + ESP_LOGE(TAG, "esp_encrypted_img_decrypt_start : Invalid argument"); + return NULL; + } + ESP_LOGI(TAG, "Starting Decryption Process"); + + esp_encrypted_img_t *handle = calloc(1, sizeof(esp_encrypted_img_t)); + if (!handle) { + ESP_LOGE(TAG, "Couldn't allocate memory to handle"); + goto failure; + } + + handle->rsa_pem = calloc(1, cfg->rsa_pub_key_len); + if (!handle->rsa_pem) { + ESP_LOGE(TAG, "Couldn't allocate memory to handle->rsa_pem"); + goto failure; + } + + handle->cache_buf = calloc(1, ENC_GCM_KEY_SIZE); + if (!handle->cache_buf) { + ESP_LOGE(TAG, "Couldn't allocate memory to handle->cache_buf"); + goto failure; + } + + memcpy(handle->rsa_pem, cfg->rsa_pub_key, cfg->rsa_pub_key_len); + handle->rsa_len = cfg->rsa_pub_key_len; + handle->state = ESP_PRE_ENC_IMG_READ_MAGIC; + + esp_decrypt_handle_t ctx = (esp_decrypt_handle_t)handle; + return ctx; + +failure: + if (handle) { + free(handle->rsa_pem); + free(handle); + } + return NULL; +} + +static esp_err_t process_bin(esp_encrypted_img_t *handle, pre_enc_decrypt_arg_t *args, int curr_index) +{ + size_t data_len = args->data_in_len; + size_t data_out_size = args->data_out_len; +#if !(MBEDTLS_VERSION_NUMBER < 0x03000000) + size_t olen; +#endif + handle->binary_file_read += data_len - curr_index; + int dec_len = 0; + if (handle->binary_file_read != handle->binary_file_len) { + size_t copy_len = 0; + + if ((handle->cache_buf_len + (data_len - curr_index)) - (handle->cache_buf_len + (data_len - curr_index)) % 16 > 0) { + data_out_size = (handle->cache_buf_len + (data_len - curr_index)) - (handle->cache_buf_len + (data_len - curr_index)) % 16; + args->data_out = realloc(args->data_out, data_out_size); + if (!args->data_out) { + return ESP_ERR_NO_MEM; + } + } + if (handle->cache_buf_len != 0) { + copy_len = MIN(16 - handle->cache_buf_len, data_len - curr_index); + memcpy(handle->cache_buf + handle->cache_buf_len, args->data_in + curr_index, copy_len); + handle->cache_buf_len += copy_len; + if (handle->cache_buf_len != 16) { + args->data_out_len = 0; + return ESP_ERR_NOT_FINISHED; + } +#if (MBEDTLS_VERSION_NUMBER < 0x03000000) + if (mbedtls_gcm_update(&handle->gcm_ctx, 16, (const unsigned char *)handle->cache_buf, (unsigned char *) args->data_out) != 0) { +#else + if (mbedtls_gcm_update(&handle->gcm_ctx, (const unsigned char *)handle->cache_buf, 16, (unsigned char *) args->data_out, data_out_size, &olen) != 0) { +#endif + return ESP_FAIL; + } + dec_len = 16; + } + handle->cache_buf_len = (data_len - curr_index - copy_len) % 16; + if (handle->cache_buf_len != 0) { + data_len -= handle->cache_buf_len; + memcpy(handle->cache_buf, args->data_in + (data_len), handle->cache_buf_len); + } + + if (data_len - copy_len - curr_index > 0) { +#if (MBEDTLS_VERSION_NUMBER < 0x03000000) + if (mbedtls_gcm_update(&handle->gcm_ctx, data_len - copy_len - curr_index, (const unsigned char *)args->data_in + curr_index + copy_len, (unsigned char *)args->data_out + dec_len) != 0) { +#else + if (mbedtls_gcm_update(&handle->gcm_ctx, (const unsigned char *)args->data_in + curr_index + copy_len, data_len - copy_len - curr_index, (unsigned char *)args->data_out + dec_len, data_out_size - dec_len, &olen) != 0) { +#endif + return ESP_FAIL; + } + } + args->data_out_len = dec_len + data_len - curr_index - copy_len; + return ESP_ERR_NOT_FINISHED; + } + data_out_size = handle->cache_buf_len + data_len - curr_index; + args->data_out = realloc(args->data_out, data_out_size); + if (!args->data_out) { + return ESP_ERR_NO_MEM; + } + size_t copy_len = 0; + + copy_len = MIN(16 - handle->cache_buf_len, data_len - curr_index); + memcpy(handle->cache_buf + handle->cache_buf_len, args->data_in + curr_index, copy_len); + handle->cache_buf_len += copy_len; +#if (MBEDTLS_VERSION_NUMBER < 0x03000000) + if (mbedtls_gcm_update(&handle->gcm_ctx, handle->cache_buf_len, (const unsigned char *)handle->cache_buf, (unsigned char *)args->data_out) != 0) { +#else + if (mbedtls_gcm_update(&handle->gcm_ctx, (const unsigned char *)handle->cache_buf, handle->cache_buf_len, (unsigned char *)args->data_out, data_out_size, &olen) != 0) { +#endif + return ESP_FAIL; + } + if (data_len - curr_index - copy_len > 0) { +#if (MBEDTLS_VERSION_NUMBER < 0x03000000) + if (mbedtls_gcm_update(&handle->gcm_ctx, data_len - curr_index - copy_len, (const unsigned char *)(args->data_in + curr_index + copy_len), (unsigned char *)(args->data_out + 16)) != 0) { +#else + if (mbedtls_gcm_update(&handle->gcm_ctx, (const unsigned char *)(args->data_in + curr_index + copy_len), data_len - curr_index - copy_len, (unsigned char *)(args->data_out + 16), data_out_size - 16, &olen) != 0) { +#endif + return ESP_FAIL; + } + } + + args->data_out_len = handle->cache_buf_len + data_len - copy_len - curr_index; + handle->cache_buf_len = 0; + + return ESP_OK; +} + +static void read_and_cache_data(esp_encrypted_img_t *handle, pre_enc_decrypt_arg_t *args, int *curr_index, int data_size) +{ + const int data_left = data_size - handle->binary_file_read; + const int data_recv = args->data_in_len - *curr_index; + if (handle->state == ESP_PRE_ENC_IMG_READ_IV && handle->iv) { + memcpy(handle->iv + handle->cache_buf_len, args->data_in + *curr_index, MIN(data_recv, data_left)); + } else if (handle->state == ESP_PRE_ENC_IMG_READ_AUTH) { + memcpy(handle->auth_tag + handle->cache_buf_len, args->data_in + *curr_index, MIN(data_recv, data_left)); + } else { + memcpy(handle->cache_buf + handle->cache_buf_len, args->data_in + *curr_index, MIN(data_recv, data_left)); + } + handle->cache_buf_len += MIN(data_recv, data_left); + int temp = *curr_index; + *curr_index += MIN(data_recv, data_left); + handle->binary_file_read += MIN(args->data_in_len - temp, data_left); +} + +esp_err_t esp_encrypted_img_decrypt_data(esp_decrypt_handle_t ctx, pre_enc_decrypt_arg_t *args) +{ + if (ctx == NULL || args == NULL || args->data_in == NULL) { + return ESP_ERR_INVALID_ARG; + } + esp_encrypted_img_t *handle = (esp_encrypted_img_t *)ctx; + if (handle == NULL) { + ESP_LOGE(TAG, "esp_encrypted_img_decrypt_data: Invalid argument"); + return ESP_ERR_INVALID_ARG; + } + + esp_err_t err; + int curr_index = 0; + + switch (handle->state) { + case ESP_PRE_ENC_IMG_READ_MAGIC: + if (handle->cache_buf_len == 0 && (args->data_in_len - curr_index) >= MAGIC_SIZE) { + uint32_t recv_magic = *(uint32_t *)args->data_in; + + if (recv_magic != esp_enc_img_magic) { + ESP_LOGE(TAG, "Magic Verification failed"); + free(handle->rsa_pem); + handle->rsa_pem = NULL; + return ESP_FAIL; + } + curr_index += MAGIC_SIZE; + } else { + read_and_cache_data(handle, args, &curr_index, MAGIC_SIZE); + if (handle->binary_file_read == MAGIC_SIZE) { + uint32_t recv_magic = *(uint32_t *)handle->cache_buf; + + if (recv_magic != esp_enc_img_magic) { + ESP_LOGE(TAG, "Magic Verification failed"); + free(handle->rsa_pem); + handle->rsa_pem = NULL; + return ESP_FAIL; + } + handle->binary_file_read = 0; + handle->cache_buf_len = 0; + } else { + return ESP_ERR_NOT_FINISHED; + } + } + ESP_LOGI(TAG, "Magic Verified"); + handle->state = ESP_PRE_ENC_IMG_READ_GCM; + /* falls through */ + case ESP_PRE_ENC_IMG_READ_GCM: + if (handle->cache_buf_len == 0 && args->data_in_len - curr_index >= ENC_GCM_KEY_SIZE) { + if (decipher_gcm_key(args->data_in + curr_index, handle) != 0) { + ESP_LOGE(TAG, "Unable to decipher GCM key"); + return ESP_FAIL; + } + curr_index += ENC_GCM_KEY_SIZE; + } else { + read_and_cache_data(handle, args, &curr_index, ENC_GCM_KEY_SIZE); + if (handle->cache_buf_len == ENC_GCM_KEY_SIZE) { + if (decipher_gcm_key(handle->cache_buf, handle) != 0) { + ESP_LOGE(TAG, "Unable to decipher GCM key"); + return ESP_FAIL; + } + } else { + return ESP_ERR_NOT_FINISHED; + } + } + /* falls through */ + case ESP_PRE_ENC_IMG_READ_IV: + if (handle->cache_buf_len == 0 && args->data_in_len - curr_index >= IV_SIZE) { + memcpy(handle->iv, args->data_in + curr_index, IV_SIZE); + handle->binary_file_read = IV_SIZE; + curr_index += IV_SIZE; + } else { + read_and_cache_data(handle, args, &curr_index, IV_SIZE); + } + if (handle->binary_file_read == IV_SIZE) { + handle->state = ESP_PRE_ENC_IMG_READ_BINSIZE; + handle->binary_file_read = 0; + handle->cache_buf_len = 0; + mbedtls_gcm_init(&handle->gcm_ctx); + if ((err = mbedtls_gcm_setkey(&handle->gcm_ctx, MBEDTLS_CIPHER_ID_AES, (const unsigned char *)handle->gcm_key, GCM_KEY_SIZE * 8)) != 0) { + ESP_LOGE(TAG, "Error: mbedtls_gcm_set_key: -0x%04x\n", (unsigned int) - err); + return ESP_FAIL; + } +#if (MBEDTLS_VERSION_NUMBER < 0x03000000) + if (mbedtls_gcm_starts(&handle->gcm_ctx, MBEDTLS_GCM_DECRYPT, (const unsigned char *)handle->iv, IV_SIZE, NULL, 0) != 0) { +#else + if (mbedtls_gcm_starts(&handle->gcm_ctx, MBEDTLS_GCM_DECRYPT, (const unsigned char *)handle->iv, IV_SIZE) != 0) { +#endif + ESP_LOGE(TAG, "Error: mbedtls_gcm_starts: -0x%04x\n", (unsigned int) - err); + return ESP_FAIL; + } + } else { + return ESP_ERR_NOT_FINISHED; + } + /* falls through */ + case ESP_PRE_ENC_IMG_READ_BINSIZE: + if (handle->cache_buf_len == 0 && (args->data_in_len - curr_index) >= BIN_SIZE_DATA) { + handle->binary_file_len = *(uint32_t *)(args->data_in + curr_index); + curr_index += BIN_SIZE_DATA; + } else { + read_and_cache_data(handle, args, &curr_index, BIN_SIZE_DATA); + if (handle->binary_file_read == BIN_SIZE_DATA) { + handle->binary_file_len = *(uint32_t *)handle->cache_buf; + } else { + return ESP_ERR_NOT_FINISHED; + } + } + handle->state = ESP_PRE_ENC_IMG_READ_AUTH; + handle->binary_file_read = 0; + handle->cache_buf_len = 0; + /* falls through */ + case ESP_PRE_ENC_IMG_READ_AUTH: + if (handle->cache_buf_len == 0 && args->data_in_len - curr_index >= AUTH_SIZE) { + memcpy(handle->auth_tag, args->data_in + curr_index, AUTH_SIZE); + handle->binary_file_read = AUTH_SIZE; + curr_index += AUTH_SIZE; + } else { + read_and_cache_data(handle, args, &curr_index, AUTH_SIZE); + } + if (handle->binary_file_read == AUTH_SIZE) { + handle->state = ESP_PRE_ENC_IMG_READ_EXTRA_HEADER; + handle->binary_file_read = 0; + handle->cache_buf_len = 0; + } else { + return ESP_ERR_NOT_FINISHED; + } + /* falls through */ + case ESP_PRE_ENC_IMG_READ_EXTRA_HEADER: { + int temp = curr_index; + curr_index += MIN(args->data_in_len - curr_index, RESERVED_HEADER - handle->binary_file_read); + handle->binary_file_read += MIN(args->data_in_len - temp, RESERVED_HEADER - handle->binary_file_read); + if (handle->binary_file_read == RESERVED_HEADER) { + handle->state = ESP_PRE_ENC_DATA_DECODE_STATE; + handle->binary_file_read = 0; + handle->cache_buf_len = 0; + } else { + return ESP_ERR_NOT_FINISHED; + } + } +/* falls through */ + case ESP_PRE_ENC_DATA_DECODE_STATE: + err = process_bin(handle, args, curr_index); + return err; + } + return ESP_OK; +} + +esp_err_t esp_encrypted_img_decrypt_end(esp_decrypt_handle_t ctx) +{ + if (ctx == NULL) { + return ESP_ERR_INVALID_ARG; + } + esp_encrypted_img_t *handle = (esp_encrypted_img_t *)ctx; + esp_err_t err = ESP_OK; + if (handle == NULL) { + ESP_LOGE(TAG, "esp_encrypted_img_decrypt_data: Invalid argument"); + return ESP_ERR_INVALID_ARG; + } + if (handle->state == ESP_PRE_ENC_DATA_DECODE_STATE) { + if (handle->cache_buf_len != 0 || handle->binary_file_read != handle->binary_file_len) { + ESP_LOGE(TAG, "Invalid operation"); + err = ESP_FAIL; + goto exit; + } + + unsigned char got_auth[AUTH_SIZE] = {0}; +#if (MBEDTLS_VERSION_NUMBER < 0x03000000) + err = mbedtls_gcm_finish(&handle->gcm_ctx, got_auth, AUTH_SIZE); +#else + size_t olen; + err = mbedtls_gcm_finish(&handle->gcm_ctx, NULL, 0, &olen, got_auth, AUTH_SIZE); +#endif + if (err != 0) { + ESP_LOGE(TAG, "Error: %d", err); + err = ESP_FAIL; + goto exit; + } + if (memcmp(got_auth, handle->auth_tag, AUTH_SIZE) != 0) { + ESP_LOGE(TAG, "Invalid Auth"); + err = ESP_FAIL; + goto exit; + } + } + err = ESP_OK; +exit: + mbedtls_gcm_free(&handle->gcm_ctx); + free(handle->cache_buf); + free(handle->rsa_pem); + free(handle); + return err; +} diff --git a/esp_encrypted_img/test/test.c b/esp_encrypted_img/test/test.c index fa8f6c5..efb4afc 100644 --- a/esp_encrypted_img/test/test.c +++ b/esp_encrypted_img/test/test.c @@ -4,8 +4,14 @@ * SPDX-License-Identifier: Unlicense OR CC0-1.0 */ #include +#include + #include "unity.h" +#if __has_include("esp_random.h") +#include "esp_random.h" +#else #include "esp_system.h" +#endif #include "esp_encrypted_img.h" extern const uint8_t rsa_private_pem_start[] asm("_binary_test_rsa_private_key_pem_start"); @@ -20,7 +26,7 @@ TEST_CASE("Sending all data at once", "[encrypted_img]") .rsa_pub_key = (char *)rsa_private_pem_start, .rsa_pub_key_len = rsa_private_pem_end - rsa_private_pem_start, }; - esp_decrypt_handle_t *ctx = esp_encrypted_img_decrypt_start(&cfg); + esp_decrypt_handle_t ctx = esp_encrypted_img_decrypt_start(&cfg); TEST_ASSERT_NOT_NULL(ctx); pre_enc_decrypt_arg_t *args = calloc(1, sizeof(pre_enc_decrypt_arg_t)); @@ -50,7 +56,7 @@ TEST_CASE("Sending 1 byte data at once", "[encrypted_img]") .rsa_pub_key = (char *)rsa_private_pem_start, .rsa_pub_key_len = rsa_private_pem_end - rsa_private_pem_start, }; - esp_decrypt_handle_t *ctx = esp_encrypted_img_decrypt_start(&cfg); + esp_decrypt_handle_t ctx = esp_encrypted_img_decrypt_start(&cfg); TEST_ASSERT_NOT_NULL(ctx); pre_enc_decrypt_arg_t *args = calloc(1, sizeof(pre_enc_decrypt_arg_t)); @@ -151,7 +157,7 @@ TEST_CASE("Invalid Magic", "[encrypted_img]") .rsa_pub_key = (char *)rsa_private_pem_start, .rsa_pub_key_len = rsa_private_pem_end - rsa_private_pem_start, }; - esp_decrypt_handle_t *ctx = esp_encrypted_img_decrypt_start(&cfg); + esp_decrypt_handle_t ctx = esp_encrypted_img_decrypt_start(&cfg); TEST_ASSERT_NOT_NULL(ctx); pre_enc_decrypt_arg_t *args = calloc(1, sizeof(pre_enc_decrypt_arg_t)); @@ -244,7 +250,7 @@ TEST_CASE("Invalid Image", "[encrypted_img]") .rsa_pub_key = (char *)rsa_private_pem_start, .rsa_pub_key_len = rsa_private_pem_end - rsa_private_pem_start, }; - esp_decrypt_handle_t *ctx = esp_encrypted_img_decrypt_start(&cfg); + esp_decrypt_handle_t ctx = esp_encrypted_img_decrypt_start(&cfg); TEST_ASSERT_NOT_NULL(ctx); pre_enc_decrypt_arg_t *args = calloc(1, sizeof(pre_enc_decrypt_arg_t)); @@ -272,7 +278,7 @@ TEST_CASE("Sending random size data at once", "[encrypted_img]") .rsa_pub_key = (char *)rsa_private_pem_start, .rsa_pub_key_len = rsa_private_pem_end - rsa_private_pem_start, }; - esp_decrypt_handle_t *ctx = esp_encrypted_img_decrypt_start(&cfg); + esp_decrypt_handle_t ctx = esp_encrypted_img_decrypt_start(&cfg); TEST_ASSERT_NOT_NULL(ctx); pre_enc_decrypt_arg_t *args = calloc(1, sizeof(pre_enc_decrypt_arg_t)); @@ -304,3 +310,20 @@ TEST_CASE("Sending random size data at once", "[encrypted_img]") } free(args); } + +TEST_CASE("Test canceling decryption frees memory", "[encrypted_img]") +{ + esp_decrypt_cfg_t cfg = { + .rsa_pub_key = (char *)rsa_private_pem_start, + .rsa_pub_key_len = rsa_private_pem_end - rsa_private_pem_start, + }; + int free_bytes_start = xPortGetFreeHeapSize(); + esp_decrypt_handle_t ctx = esp_encrypted_img_decrypt_start(&cfg); + TEST_ASSERT_NOT_NULL(ctx); + + (void) esp_encrypted_img_decrypt_end(ctx); + int free_bytes_end = xPortGetFreeHeapSize(); + + // +/- 16 bytes to allow for some small fluctuations + TEST_ASSERT(abs(free_bytes_start - free_bytes_end) <= 16); +} diff --git a/esp_encrypted_img/tools/esp_enc_img_gen.py b/esp_encrypted_img/tools/esp_enc_img_gen.py index 62c257f..d9222a8 100644 --- a/esp_encrypted_img/tools/esp_enc_img_gen.py +++ b/esp_encrypted_img/tools/esp_enc_img_gen.py @@ -56,7 +56,7 @@ def encrypt(input_file: str, rsa_key_file_name: str, output_file: str) -> None: encrypted_gcm_key = public_key.encrypt(gcm_key, padding.PKCS1v15()) ciphertext, authtag = encrypt_binary(data, gcm_key, iv) - with open(output_file, 'ab') as image: + with open(output_file, 'wb') as image: image.write(esp_enc_img_magic.to_bytes(MAGIC_SIZE, 'little')) image.write((encrypted_gcm_key)) image.write((iv)) @@ -98,7 +98,7 @@ def decrypt(input_file: str, rsa_key: str, output_file: str) -> None: decrypted_binary = decrypt_binary(enc_bin, auth, gcm_key, iv) - with open(output_file, 'ab') as file: + with open(output_file, 'wb') as file: file.write(decrypted_binary) print('Done') diff --git a/libsodium/libsodium b/libsodium/libsodium index 4f5e89f..f0ad119 160000 --- a/libsodium/libsodium +++ b/libsodium/libsodium @@ -1 +1 @@ -Subproject commit 4f5e89fa84ce1d178a6765b8b46f2b6f91216677 +Subproject commit f0ad119206032b39790fa2882d666443fc7481b7 diff --git a/libsodium/port/randombytes_esp32.c b/libsodium/port/randombytes_esp32.c index 4d98538..13234d6 100644 --- a/libsodium/port/randombytes_esp32.c +++ b/libsodium/port/randombytes_esp32.c @@ -3,8 +3,13 @@ * * SPDX-License-Identifier: Apache-2.0 */ -#include "randombytes_internal.h" +#include "sdkconfig.h" +#if __has_include("esp_random.h") +#include "esp_random.h" +#else #include "esp_system.h" +#endif +#include "randombytes_internal.h" static const char *randombytes_esp32xx_implementation_name(void) {