dearsky/pico-fido
1
0
Fork 0
Code Issues 46 Pull Requests 1 Actions Packages Projects Releases 22 Wiki Activity

Fix credProtect should not be returned on getAssertion.

Browse Source 
Signed-off-by: Pol Henarejos <pol.henarejos@cttc.es>
This commit is contained in:
Pol Henarejos
2022-12-07 15:57:41 +01:00
parent fa5926a3cc
commit 1376c51528
1 changed files with 0 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/fido/cbor_get_assertion.c
View File

@@ -174,7 +174,6 @@ int cbor_get_assertion(const uint8_t *data, size_t len, bool next) {
CBOR_PARSE_MAP_END(_f2, 3); CBOR_PARSE_MAP_END(_f2, 3);
continue; continue;
} }
CBOR_FIELD_KEY_TEXT_VAL_UINT(2, "credProtect", extensions.credProtect);
CBOR_FIELD_KEY_TEXT_VAL_BOOL(2, "credBlob", credBlob); CBOR_FIELD_KEY_TEXT_VAL_BOOL(2, "credBlob", credBlob);
CBOR_ADVANCE(2); CBOR_ADVANCE(2);
} }
@@ -379,8 +378,6 @@ int cbor_get_assertion(const uint8_t *data, size_t len, bool next) {
extensions.hmac_secret = NULL; extensions.hmac_secret = NULL;
if (extensions.hmac_secret != NULL) if (extensions.hmac_secret != NULL)
l++; l++;
if (extensions.credProtect != 0)
l++;
if (credBlob == ptrue) if (credBlob == ptrue)
l++; l++;
CBOR_CHECK(cbor_encoder_create_map(&encoder, &mapEncoder, l)); CBOR_CHECK(cbor_encoder_create_map(&encoder, &mapEncoder, l));
@@ -391,10 +388,6 @@ int cbor_get_assertion(const uint8_t *data, size_t len, bool next) {
else else
CBOR_CHECK(cbor_encode_byte_string(&mapEncoder, NULL, 0)); CBOR_CHECK(cbor_encode_byte_string(&mapEncoder, NULL, 0));
} }
if (extensions.credProtect != 0) {
CBOR_CHECK(cbor_encode_text_stringz(&mapEncoder, "credProtect"));
CBOR_CHECK(cbor_encode_uint(&mapEncoder, extensions.credProtect));
}
if (extensions.hmac_secret != NULL) { if (extensions.hmac_secret != NULL) {
CBOR_CHECK(cbor_encode_text_stringz(&mapEncoder, "hmac-secret")); CBOR_CHECK(cbor_encode_text_stringz(&mapEncoder, "hmac-secret"));