vendor/libsodium/src/libsodium/crypto_pwhash/argon2/argon2-encoding.c in rbnacl-libsodium-1.0.11 vs vendor/libsodium/src/libsodium/crypto_pwhash/argon2/argon2-encoding.c in rbnacl-libsodium-1.0.13

- old
+ new

@@ -1,11 +1,11 @@ +#include "argon2-encoding.h" +#include "argon2-core.h" +#include <limits.h> #include <stdio.h> #include <stdlib.h> #include <string.h> -#include <limits.h> -#include "argon2-core.h" -#include "argon2-encoding.h" /* * Example code for a decoder and encoder of "hash strings", with Argon2 * parameters. * @@ -20,15 +20,10 @@ * * -- The second section is specific to Argon2. It encodes and decodes * the parameters, salts and outputs. It does not compute the hash * itself. * - * -- The third section is test code, with a main() function. With - * this section, the whole file compiles as a stand-alone program - * that exercises the encoding and decoding functions with some - * test vectors. - * * The code was originally written by Thomas Pornin <pornin@bolet.org>, * to whom comments and remarks may be sent. It is released under what * should amount to Public Domain or its closest equivalent; the * following mantra is supposed to incarnate that fact with all the * proper legal rituals: @@ -57,31 +52,36 @@ /* * Some macros for constant-time comparisons. These work over values in * the 0..255 range. Returned value is 0x00 on "false", 0xFF on "true". */ -#define EQ(x, y) ((((0U - ((unsigned)(x) ^ (unsigned)(y))) >> 8) & 0xFF) ^ 0xFF) -#define GT(x, y) ((((unsigned)(y) - (unsigned)(x)) >> 8) & 0xFF) +#define EQ(x, y) \ + ((((0U - ((unsigned) (x) ^ (unsigned) (y))) >> 8) & 0xFF) ^ 0xFF) +#define GT(x, y) ((((unsigned) (y) - (unsigned) (x)) >> 8) & 0xFF) #define GE(x, y) (GT(y, x) ^ 0xFF) #define LT(x, y) GT(y, x) #define LE(x, y) GE(y, x) /* * Convert value x (0..63) to corresponding Base64 character. */ -static int b64_byte_to_char(unsigned x) { +static int +b64_byte_to_char(unsigned x) +{ return (LT(x, 26) & (x + 'A')) | (GE(x, 26) & LT(x, 52) & (x + ('a' - 26))) | (GE(x, 52) & LT(x, 62) & (x + ('0' - 52))) | (EQ(x, 62) & '+') | (EQ(x, 63) & '/'); } /* * Convert character c to the corresponding 6-bit value. If character c * is not a Base64 character, then 0xFF (255) is returned. */ -static unsigned b64_char_to_byte(int c) { +static unsigned +b64_char_to_byte(int c) +{ unsigned x; x = (GE(c, 'A') & LE(c, 'Z') & (c - 'A')) | (GE(c, 'a') & LE(c, 'z') & (c - ('a' - 26))) | (GE(c, '0') & LE(c, '9') & (c - ('0' - 52))) | (EQ(c, '+') & 62) | @@ -95,15 +95,16 @@ * receive the result (including the terminating 0), then (size_t)-1 * is returned. Otherwise, the zero-terminated Base64 string is written * in the buffer, and the output length (counted WITHOUT the terminating * zero) is returned. */ -static size_t to_base64(char *dst, size_t dst_len, const void *src, - size_t src_len) { - size_t olen; +static size_t +to_base64(char *dst, size_t dst_len, const void *src, size_t src_len) +{ + size_t olen; const unsigned char *buf; - unsigned acc, acc_len; + unsigned acc, acc_len; olen = (src_len / 3) << 2; switch (src_len % 3) { case 2: olen++; @@ -111,25 +112,25 @@ case 1: olen += 2; break; } if (dst_len <= olen) { - return (size_t)-1; + return (size_t) -1; } - acc = 0; + acc = 0; acc_len = 0; - buf = (const unsigned char *)src; + buf = (const unsigned char *) src; while (src_len-- > 0) { acc = (acc << 8) + (*buf++); acc_len += 8; while (acc_len >= 6) { acc_len -= 6; - *dst++ = (char)b64_byte_to_char((acc >> acc_len) & 0x3F); + *dst++ = (char) b64_byte_to_char((acc >> acc_len) & 0x3F); } } if (acc_len > 0) { - *dst++ = (char)b64_byte_to_char((acc << (6 - acc_len)) & 0x3F); + *dst++ = (char) b64_byte_to_char((acc << (6 - acc_len)) & 0x3F); } *dst++ = 0; return olen; } @@ -144,18 +145,20 @@ * buffer is too small, invalid last characters leading to unprocessed * buffered bits), then NULL is returned; otherwise, the returned value * points to the first non-Base64 character in the source stream, which * may be the terminating zero. */ -static const char *from_base64(void *dst, size_t *dst_len, const char *src) { - size_t len; +static const char * +from_base64(void *dst, size_t *dst_len, const char *src) +{ + size_t len; unsigned char *buf; - unsigned acc, acc_len; + unsigned acc, acc_len; - buf = (unsigned char *)dst; - len = 0; - acc = 0; + buf = (unsigned char *) dst; + len = 0; + acc = 0; acc_len = 0; for (;;) { unsigned d; d = b64_char_to_byte(*src); @@ -192,13 +195,15 @@ * Returned value is a pointer to the next non-decimal character in the * string. If there is no digit at all, or the value encoding is not * minimal (extra leading zeros), or the value does not fit in an * 'unsigned long', then NULL is returned. */ -static const char *decode_decimal(const char *str, unsigned long *v) { - const char *orig; - unsigned long acc; +static const char * +decode_decimal(const char *str, unsigned long *v) +{ + const char *orig; + unsigned long acc; acc = 0; for (orig = str;; str++) { int c; @@ -209,14 +214,14 @@ c -= '0'; if (acc > (ULONG_MAX / 10)) { return NULL; } acc *= 10; - if ((unsigned long)c > (ULONG_MAX - acc)) { + if ((unsigned long) c > (ULONG_MAX - acc)) { return NULL; } - acc += (unsigned long)c; + acc += (unsigned long) c; } if (str == orig || (*orig == '0' && str != (orig + 1))) { return NULL; } *v = acc; @@ -227,107 +232,125 @@ /* * Code specific to Argon2. * * The code below applies the following format: * - * $argon2<T>$v=<num>$m=<num>,t=<num>,p=<num>[,keyid=<bin>][,data=<bin>][$<bin>[$<bin>]] + * $argon2<T>[$v=<num>]$m=<num>,t=<num>,p=<num>$<bin>$<bin> * - * where <T> is either 'd' or 'i', <num> is a decimal integer (positive, fits in an 'unsigned long') - * and <bin> is Base64-encoded data (no '=' padding characters, no newline - * or whitespace). The "keyid" is a binary identifier for a key (up to 8 - * bytes); "data" is associated data (up to 32 bytes). When the 'keyid' - * (resp. the 'data') is empty, then it is ommitted from the output. + * where <T> is either 'i', <num> is a decimal integer (positive, fits in an + * 'unsigned long') and <bin> is Base64-encoded data (no '=' padding characters, + * no newline or whitespace). * * The last two binary chunks (encoded in Base64) are, in that order, - * the salt and the output. Both are optional, but you cannot have an - * output without a salt. The binary salt length is between 8 and 48 bytes. - * The output length is always exactly 32 bytes. + * the salt and the output. Both are required. The binary salt length and the + * output length must be in the allowed ranges defined in argon2.h. + * + * The ctx struct must contain buffers large enough to hold the salt and pwd + * when it is fed into decode_string. */ /* * Decode an Argon2i hash string into the provided structure 'ctx'. * Returned value is ARGON2_OK on success. */ -int decode_string(argon2_context *ctx, const char *str, argon2_type type) { - /* Prefix checking */ -#define CC(prefix) \ - do { \ - size_t cc_len = strlen(prefix); \ - if (strncmp(str, prefix, cc_len) != 0) { \ - return ARGON2_DECODING_FAIL; \ - } \ - str += cc_len; \ - } while ((void)0, 0) +int +decode_string(argon2_context *ctx, const char *str, argon2_type type) +{ +/* Prefix checking */ +#define CC(prefix) \ + do { \ + size_t cc_len = strlen(prefix); \ + if (strncmp(str, prefix, cc_len) != 0) { \ + return ARGON2_DECODING_FAIL; \ + } \ + str += cc_len; \ + } while ((void) 0, 0) - /* Prefix checking with supplied code */ -#define CC_opt(prefix, code) \ - do { \ - size_t cc_len = strlen(prefix); \ - if (strncmp(str, prefix, cc_len) == 0) { \ - str += cc_len; \ - { code; } \ - } \ - } while ((void)0, 0) +/* Optional prefix checking with supplied code */ +#define CC_opt(prefix, code) \ + do { \ + size_t cc_len = strlen(prefix); \ + if (strncmp(str, prefix, cc_len) == 0) { \ + str += cc_len; \ + { \ + code; \ + } \ + } \ + } while ((void) 0, 0) - /* Decoding prefix into decimal */ -#define DECIMAL(x) \ - do { \ - unsigned long dec_x; \ - str = decode_decimal(str, &dec_x); \ - if (str == NULL) { \ - return ARGON2_DECODING_FAIL; \ - } \ - (x) = dec_x; \ - } while ((void)0, 0) +/* Decoding prefix into decimal */ +#define DECIMAL(x) \ + do { \ + unsigned long dec_x; \ + str = decode_decimal(str, &dec_x); \ + if (str == NULL) { \ + return ARGON2_DECODING_FAIL; \ + } \ + (x) = dec_x; \ + } while ((void) 0, 0) - /* Decoding prefix into binary */ -#define BIN(buf, max_len, len) \ - do { \ - size_t bin_len = (max_len); \ - str = from_base64(buf, &bin_len, str); \ - if (str == NULL || bin_len > UINT32_MAX) { \ - return ARGON2_DECODING_FAIL; \ - } \ - (len) = (uint32_t)bin_len; \ +/* Decoding prefix into uint32_t decimal */ +#define DECIMAL_U32(x) \ + do { \ + unsigned long dec_x; \ + str = decode_decimal(str, &dec_x); \ + if (str == NULL || dec_x > UINT32_MAX) { \ + return ARGON2_DECODING_FAIL; \ + } \ + (x) = (uint32_t)dec_x; \ } while ((void)0, 0) - size_t maxadlen = ctx->adlen; - size_t maxsaltlen = ctx->saltlen; - size_t maxoutlen = ctx->outlen; - unsigned long version = 0; - int validation_result; +/* Decoding base64 into a binary buffer */ +#define BIN(buf, max_len, len) \ + do { \ + size_t bin_len = (max_len); \ + str = from_base64(buf, &bin_len, str); \ + if (str == NULL || bin_len > UINT32_MAX) { \ + return ARGON2_DECODING_FAIL; \ + } \ + (len) = (uint32_t) bin_len; \ + } while ((void) 0, 0) - ctx->adlen = 0; + size_t maxsaltlen = ctx->saltlen; + size_t maxoutlen = ctx->outlen; + int validation_result; + uint32_t version = 0; + ctx->saltlen = 0; - ctx->outlen = 0; - if (type == Argon2_i) { + ctx->outlen = 0; + + if (type == Argon2_id) { + CC("$argon2id"); + } else if (type == Argon2_i) { CC("$argon2i"); } else { return ARGON2_INCORRECT_TYPE; } CC("$v="); - DECIMAL(version); + DECIMAL_U32(version); if (version != ARGON2_VERSION_NUMBER) { return ARGON2_INCORRECT_TYPE; } CC("$m="); - DECIMAL(ctx->m_cost); + DECIMAL_U32(ctx->m_cost); + if (ctx->m_cost > UINT32_MAX) { + return ARGON2_INCORRECT_TYPE; + } CC(",t="); - DECIMAL(ctx->t_cost); + DECIMAL_U32(ctx->t_cost); + if (ctx->t_cost > UINT32_MAX) { + return ARGON2_INCORRECT_TYPE; + } CC(",p="); - DECIMAL(ctx->lanes); + DECIMAL_U32(ctx->lanes); + if (ctx->lanes > UINT32_MAX) { + return ARGON2_INCORRECT_TYPE; + } ctx->threads = ctx->lanes; - CC_opt(",data=", BIN(ctx->ad, maxadlen, ctx->adlen)); - if (*str == 0) { - return ARGON2_OK; - } CC("$"); BIN(ctx->salt, maxsaltlen, ctx->saltlen); - if (*str == 0) { - return ARGON2_OK; - } CC("$"); BIN(ctx->out, maxoutlen, ctx->outlen); validation_result = validate_inputs(ctx); if (validation_result != ARGON2_OK) { return validation_result; @@ -343,11 +366,13 @@ #undef BIN } #define U32_STR_MAXSIZE 11U -static void u32_to_string(char *str, uint32_t x) { +static void +u32_to_string(char *str, uint32_t x) +{ char tmp[U32_STR_MAXSIZE - 1U]; size_t i; i = sizeof tmp; do { @@ -368,45 +393,49 @@ * (no output). if pp->salt_len is also 0, then the string will be a * parameter-only string (no salt and no output). * * On success, ARGON2_OK is returned. */ -int encode_string(char *dst, size_t dst_len, argon2_context *ctx, - argon2_type type) { -#define SS(str) \ - do { \ - size_t pp_len = strlen(str); \ - if (pp_len >= dst_len) { \ - return ARGON2_ENCODING_FAIL; \ - } \ - memcpy(dst, str, pp_len + 1); \ - dst += pp_len; \ - dst_len -= pp_len; \ - } while ((void)0, 0) +int +encode_string(char *dst, size_t dst_len, argon2_context *ctx, argon2_type type) +{ +#define SS(str) \ + do { \ + size_t pp_len = strlen(str); \ + if (pp_len >= dst_len) { \ + return ARGON2_ENCODING_FAIL; \ + } \ + memcpy(dst, str, pp_len + 1); \ + dst += pp_len; \ + dst_len -= pp_len; \ + } while ((void) 0, 0) -#define SX(x) \ - do { \ - char tmp[U32_STR_MAXSIZE]; \ - u32_to_string(tmp, x); \ - SS(tmp); \ - } while ((void)0, 0) +#define SX(x) \ + do { \ + char tmp[U32_STR_MAXSIZE]; \ + u32_to_string(tmp, x); \ + SS(tmp); \ + } while ((void) 0, 0) -#define SB(buf, len) \ - do { \ - size_t sb_len = to_base64(dst, dst_len, buf, len); \ - if (sb_len == (size_t)-1) { \ - return ARGON2_ENCODING_FAIL; \ - } \ - dst += sb_len; \ - dst_len -= sb_len; \ - } while ((void)0, 0) +#define SB(buf, len) \ + do { \ + size_t sb_len = to_base64(dst, dst_len, buf, len); \ + if (sb_len == (size_t) -1) { \ + return ARGON2_ENCODING_FAIL; \ + } \ + dst += sb_len; \ + dst_len -= sb_len; \ + } while ((void) 0, 0) int validation_result; - if (type == Argon2_i) { - SS("$argon2i$v="); - } else { + switch (type) { + case Argon2_id: + SS("$argon2id$v="); break; + case Argon2_i: + SS("$argon2i$v="); break; + default: return ARGON2_ENCODING_FAIL; } validation_result = validate_inputs(ctx); if (validation_result != ARGON2_OK) { return validation_result; @@ -417,23 +446,12 @@ SS(",t="); SX(ctx->t_cost); SS(",p="); SX(ctx->lanes); - if (ctx->adlen > 0) { - SS(",data="); - SB(ctx->ad, ctx->adlen); - } - - if (ctx->saltlen == 0) { - return ARGON2_OK; - } SS("$"); SB(ctx->salt, ctx->saltlen); - if (ctx->outlen == 0) { - return ARGON2_OK; - } SS("$"); SB(ctx->out, ctx->outlen); return ARGON2_OK; #undef SS