dearsky/pico-openpgp
1
0
Fork 0
Code Issues 10 Pull Requests Actions Packages Projects Releases 15 Wiki Activity

Signature counter is only increased on PSO:SIGN and not for authentication.

Browse Source 
Signed-off-by: Pol Henarejos <pol.henarejos@cttc.es>
This commit is contained in:
Pol Henarejos
2023-02-04 01:31:34 +01:00
parent e1407636b8
commit d9ed002af2
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/openpgp/openpgp.c
View File

@@ -1374,7 +1374,6 @@ int rsa_sign(mbedtls_rsa_context *ctx, const uint8_t *data, size_t data_len, uin
free(signature); free(signature);
} }
*out_len = key_size; *out_len = key_size;
inc_sig_count();
return r; return r;
} }
@@ -1391,7 +1390,6 @@ int ecdsa_sign(mbedtls_ecdsa_context *ctx, const uint8_t *data, size_t data_len,
} }
mbedtls_mpi_free(&ri); mbedtls_mpi_free(&ri);
mbedtls_mpi_free(&si); mbedtls_mpi_free(&si);
inc_sig_count();
return r; return r;
} }
@@ -1474,6 +1472,7 @@ static int cmd_pso() {
return SW_EXEC_ERROR(); return SW_EXEC_ERROR();
res_APDU_size = olen; res_APDU_size = olen;
//apdu.ne = key_size; //apdu.ne = key_size;
inc_sig_count();
} }
else if (P1(apdu) == 0x80 && P2(apdu) == 0x86) { else if (P1(apdu) == 0x80 && P2(apdu) == 0x86) {
if (apdu.nc < key_size) //needs padding if (apdu.nc < key_size) //needs padding
@@ -1502,6 +1501,7 @@ static int cmd_pso() {
if (r != 0) if (r != 0)
return SW_EXEC_ERROR(); return SW_EXEC_ERROR();
res_APDU_size = olen; res_APDU_size = olen;
inc_sig_count();
} }
else if (P1(apdu) == 0x80 && P2(apdu) == 0x86) { else if (P1(apdu) == 0x80 && P2(apdu) == 0x86) {
mbedtls_ecdh_context ctx; mbedtls_ecdh_context ctx;