frost/secp256k1-zkp
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Normalize ge produced from secp256k1_pubkey_load

Browse Source 
The output ge is normalized when sizeof(secp256k1_ge_storage) = 64
but not when it's not 64. ARG_CHECK at the end of the function
assumes normalization. So normalize ge in the other code path too.
This commit is contained in:
stratospher 2023-06-15 23:36:19 +05:30
parent 67214f5f7d
commit f1652528be
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/secp256k1.c
View File

@ -247,8 +247,8 @@ static int secp256k1_pubkey_load(const secp256k1_context* ctx, secp256k1_ge* ge,
} else { } else {
/* Otherwise, fall back to 32-byte big endian for X and Y. */ /* Otherwise, fall back to 32-byte big endian for X and Y. */
secp256k1_fe x, y; secp256k1_fe x, y;
secp256k1_fe_set_b32_mod(&x, pubkey->data); ARG_CHECK(secp256k1_fe_set_b32_limit(&x, pubkey->data));
secp256k1_fe_set_b32_mod(&y, pubkey->data + 32); ARG_CHECK(secp256k1_fe_set_b32_limit(&y, pubkey->data + 32));
secp256k1_ge_set_xy(ge, &x, &y); secp256k1_ge_set_xy(ge, &x, &y);
} }
ARG_CHECK(!secp256k1_fe_is_zero(&ge->x)); ARG_CHECK(!secp256k1_fe_is_zero(&ge->x));