diff options
author | Dmitry Belyavskiy <beldmit@gmail.com> | 2018-12-29 15:01:32 +0300 |
---|---|---|
committer | Dmitry Belyavskiy <beldmit@gmail.com> | 2018-12-29 15:01:32 +0300 |
commit | 6087586263eb1351a5fa370b233b8d747d0e1f59 (patch) | |
tree | 466a5c73b04cb2ec3cbb3a309df0feac2fabe908 | |
parent | b9935fbf77164cf491fb12df74f7a85e877813cd (diff) | |
download | gost-engine-6087586263eb1351a5fa370b233b8d747d0e1f59.zip gost-engine-6087586263eb1351a5fa370b233b8d747d0e1f59.tar.gz gost-engine-6087586263eb1351a5fa370b233b8d747d0e1f59.tar.bz2 |
Some PVS Studio nits
-rw-r--r-- | benchmark/sign.c | 4 | ||||
-rw-r--r-- | gost_ec_keyx.c | 3 | ||||
-rw-r--r-- | gost_keyexpimp.c | 2 | ||||
-rw-r--r-- | test_tlstree.c | 2 |
4 files changed, 7 insertions, 4 deletions
diff --git a/benchmark/sign.c b/benchmark/sign.c index 417d8dd..2d1d786 100644 --- a/benchmark/sign.c +++ b/benchmark/sign.c @@ -131,6 +131,10 @@ int main(int argc, char **argv) fflush(stdout); siglen = EVP_PKEY_size(pkey); sigbuf = malloc(siglen * cycles); + if (!sigbuf) { + fprintf(stderr, "No tests were run, malloc failure.\n"); + exit(1); + } for (pass = 0; pass < 2; pass++) { struct timespec ts; diff --git a/gost_ec_keyx.c b/gost_ec_keyx.c index a95df33..6efa3e8 100644 --- a/gost_ec_keyx.c +++ b/gost_ec_keyx.c @@ -251,8 +251,7 @@ static int pkey_GOST_ECcp_encrypt(EVP_PKEY_CTX *pctx, unsigned char *out, EVP_PKEY *sec_key = EVP_PKEY_CTX_get0_peerkey(pctx); if (data->shared_ukm) { memcpy(ukm, data->shared_ukm, 8); - } else if (out) { - + } else { if (RAND_bytes(ukm, 8) <= 0) { GOSTerr(GOST_F_PKEY_GOST_ECCP_ENCRYPT, GOST_R_RNG_ERROR); return 0; diff --git a/gost_keyexpimp.c b/gost_keyexpimp.c index caa6171..1f058a1 100644 --- a/gost_keyexpimp.c +++ b/gost_keyexpimp.c @@ -84,7 +84,7 @@ int gost_kexp15(const unsigned char *shared_key, const int shared_len, /* * Function expects that shared_key is a preallocated buffer - * with length defined as expkeylen - mac_len defined by mac_nid + * with length defined as expkeylen + mac_len defined by mac_nid * */ int gost_kimp15(const unsigned char *expkey, const size_t expkeylen, int cipher_nid, const unsigned char *cipher_key, diff --git a/test_tlstree.c b/test_tlstree.c index 08caf90..df25742 100644 --- a/test_tlstree.c +++ b/test_tlstree.c @@ -132,7 +132,7 @@ int main(void) EVP_Cipher(enc, data0_processed+sizeof(data0), mac0, 16); hexdump(stderr, "ENC0 result", data0_processed, 31); - if (memcmp(enc0_etl, data0_processed, 16) != 0) { + if (memcmp(enc0_etl, data0_processed, sizeof(data0_processed)) != 0) { fprintf(stderr, "ENC0 mismatch"); exit(1); } |