Skip to content

Commit 1973c73

Browse files
committed
Bugfix: Reinitialise buffer lengths that have been used as outputs
1 parent 788038d commit 1973c73

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/tests.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1937,6 +1937,7 @@ void test_ecdsa_end_to_end(void) {
19371937
CHECK(secp256k1_ecdsa_signature_parse_der(ctx, &signature[0], sig, siglen) == 1);
19381938
CHECK(secp256k1_ecdsa_verify(ctx, &signature[0], message, &pubkey) == 1);
19391939
/* Serialize/destroy/parse DER and verify again. */
1940+
siglen = 74;
19401941
CHECK(secp256k1_ecdsa_signature_serialize_der(ctx, sig, &siglen, &signature[0]) == 1);
19411942
sig[secp256k1_rand32() % siglen] += 1 + (secp256k1_rand32() % 255);
19421943
CHECK(secp256k1_ecdsa_signature_parse_der(ctx, &signature[0], sig, siglen) == 0 ||
@@ -2154,6 +2155,7 @@ void test_ecdsa_edge_cases(void) {
21542155
};
21552156
size_t outlen = 300;
21562157
CHECK(!secp256k1_ec_privkey_export(ctx, privkey, &outlen, seckey, 0));
2158+
outlen = 300;
21572159
CHECK(!secp256k1_ec_privkey_export(ctx, privkey, &outlen, seckey, 1));
21582160
}
21592161
}

0 commit comments

Comments
 (0)