On Wed, Oct 26, 2016 at 02:09:26PM +0300, Cristian Stoica wrote: > Signed-off-by: Cristian Stoica <cristian.sto...@nxp.com> > --- > tests/async_hmac.c | 2 -- > tests/cipher_comp.c | 2 +- > tests/hash_comp.c | 5 ++--- > tests/hmac.c | 2 -- > tests/sha_speed.c | 1 - > tests/speed.c | 1 - > 6 files changed, 3 insertions(+), 10 deletions(-) > > diff --git a/tests/async_hmac.c b/tests/async_hmac.c > index 94a02c0..85d19c6 100644 > --- a/tests/async_hmac.c > +++ b/tests/async_hmac.c > @@ -185,8 +185,6 @@ test_extras(int cfd) > struct session_op sess; > struct crypt_op cryp; > uint8_t mac[AALG_MAX_RESULT_LEN]; > - uint8_t oldmac[AALG_MAX_RESULT_LEN]; > - uint8_t md5_hmac_out[] = > "\x75\x0c\x78\x3e\x6a\xb0\xb5\x03\xea\xa8\x6e\x31\x0a\x5d\xb7\x38"; > uint8_t sha1_out[] = > "\x8f\x82\x03\x94\xf9\x53\x35\x18\x20\x45\xda\x24\xf3\x4d\xe5\x2b\xf8\xbc\x34\x32"; > int i; > > diff --git a/tests/cipher_comp.c b/tests/cipher_comp.c > index 03f67bf..dbf9977 100644 > --- a/tests/cipher_comp.c > +++ b/tests/cipher_comp.c > @@ -33,7 +33,7 @@ test_crypto(int cfd, struct session_op *sess, int datalen) > > struct crypt_op cryp; > > - int ret = 0, fail = 0; > + int ret = 0; > > data = malloc(datalen); > encrypted = malloc(datalen); > diff --git a/tests/hash_comp.c b/tests/hash_comp.c > index e6a4346..73f85ed 100644 > --- a/tests/hash_comp.c > +++ b/tests/hash_comp.c > @@ -36,7 +36,7 @@ test_crypto(int cfd, struct session_op *sess, int datalen) > > struct crypt_op cryp; > > - int ret = 0, fail = 0; > + int ret = 0; > > data = malloc(datalen); > memset(data, datalen & 0xff, datalen); > @@ -66,7 +66,7 @@ test_crypto(int cfd, struct session_op *sess, int datalen) > > if (memcmp(mac, mac_comp, AALG_MAX_RESULT_LEN)) { > printf("fail for datalen %d, MACs do not match!\n", datalen); > - fail = 1; > + ret = 1; > printf("wrong mac: "); > printhex(mac, 20); > printf("right mac: "); > @@ -88,7 +88,6 @@ main(int argc, char **argv) > struct session_op sess; > int datalen = BLOCK_SIZE; > int datalen_end = MAX_DATALEN; > - int i; > > if (argc > 1) { > datalen = min(max(atoi(argv[1]), BLOCK_SIZE), MAX_DATALEN); > diff --git a/tests/hmac.c b/tests/hmac.c > index 80a2c42..3b248f3 100644 > --- a/tests/hmac.c > +++ b/tests/hmac.c > @@ -212,8 +212,6 @@ test_extras(int cfd) > #endif > struct crypt_op cryp; > uint8_t mac[AALG_MAX_RESULT_LEN]; > - uint8_t oldmac[AALG_MAX_RESULT_LEN]; > - uint8_t md5_hmac_out[] = > "\x75\x0c\x78\x3e\x6a\xb0\xb5\x03\xea\xa8\x6e\x31\x0a\x5d\xb7\x38"; > uint8_t sha1_out[] = > "\x8f\x82\x03\x94\xf9\x53\x35\x18\x20\x45\xda\x24\xf3\x4d\xe5\x2b\xf8\xbc\x34\x32"; > int i; > > diff --git a/tests/sha_speed.c b/tests/sha_speed.c > index 5034b58..1e67260 100644 > --- a/tests/sha_speed.c > +++ b/tests/sha_speed.c > @@ -131,7 +131,6 @@ int main(void) > { > int fd, i, fdc = -1, alignmask = 0; > struct session_op sess; > - char keybuf[32]; > #ifdef CIOCGSESSINFO > struct session_info_op siop; > #endif > diff --git a/tests/speed.c b/tests/speed.c > index 0b40bcf..84a5bf0 100644 > --- a/tests/speed.c > +++ b/tests/speed.c > @@ -76,7 +76,6 @@ int encrypt_data(struct session_op *sess, int fdc, int > chunksize, int alignmask) > { > struct crypt_op cop; > char *buffer, iv[32]; > - uint8_t mac[HASH_MAX_LEN]; > static int val = 23; > struct timeval start, end; > double total = 0;
The last one was actually added by you two patches before. Please get rid of that nit and respin. Thanks, Phil _______________________________________________ Cryptodev-linux-devel mailing list Cryptodev-linux-devel@gna.org https://mail.gna.org/listinfo/cryptodev-linux-devel