diff options
| -rw-r--r-- | libar2_decode_params.c | 4 | ||||
| -rw-r--r-- | test.c | 5 |
2 files changed, 7 insertions, 2 deletions
diff --git a/libar2_decode_params.c b/libar2_decode_params.c index 1f7d270..86fae6c 100644 --- a/libar2_decode_params.c +++ b/libar2_decode_params.c @@ -8,7 +8,7 @@ decode_u32(const char *s, uint_least32_t *outp) uint_least32_t digit; size_t i; - if ((s[0] == '0' && s[1] != '0') || !isdigit(s[0])) { + if (s[0] == '0' || !isdigit(s[0])) { errno = EINVAL; return 0; } @@ -59,6 +59,8 @@ libar2_decode_params(const char *str, struct libar2_argon2_parameters *params, c goto fail; if (u32 > (uint_least32_t)INT_MAX) goto erange; + if (!u32) + goto einval; params->version = (enum libar2_argon2_version)u32; str += n + 2; if (*str++ != '$') @@ -1135,8 +1135,11 @@ check_failures(void) CHECK("$argon2id$p=128,m=128,t=128$AAAABBBBCCCC$"); CHECK("$argon2id$p=128,t=128,m=128$AAAABBBBCCCC$"); CHECK("$argon2id$m=0128,t=128,p=128$AAAABBBBCCCC$"); + CHECK("$argon2id$m=00128,t=128,p=128$AAAABBBBCCCC$"); CHECK("$argon2id$m=128,t=0128,p=128$AAAABBBBCCCC$"); - CHECK("$argon2id$m=128,t=128,p=0128$AAAABBBBCCCC$"); + CHECK("$argon2id$m=128,t=00128,p=128$AAAABBBBCCCC$"); + CHECK("$argon2id$m=128,t=128,p=0128$AAAABBBBCCCC$");; + CHECK("$argon2id$m=128,t=128,p=00128$AAAABBBBCCCC$"); CHECK("$argon2id$v=0$m=128,t=128,p=128$AAAABBBBCCCC$"); CHECK("$argon2id$v=016$m=128,t=128,p=128$AAAABBBBCCCC$"); errno = 0; |