From c51c135192313f9f25148286e150637b57eeda1e Mon Sep 17 00:00:00 2001 From: Ludwig Ortmann Date: Sat, 1 Mar 2014 12:29:05 +0100 Subject: [PATCH 1/2] sys/crypto: remove assert.h include It is not used and most platforms don't have it. --- sys/include/crypto/3des.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sys/include/crypto/3des.h b/sys/include/crypto/3des.h index 658110acb4..88d3fa56be 100644 --- a/sys/include/crypto/3des.h +++ b/sys/include/crypto/3des.h @@ -22,7 +22,7 @@ #include #include #include -#include "assert.h" + #include "crypto/ciphers.h" #ifndef THREEDES_H_ From 6089de5ef2f85461e1580e7941a9063bc22bb1f7 Mon Sep 17 00:00:00 2001 From: Ludwig Ortmann Date: Sat, 1 Mar 2014 16:27:18 +0100 Subject: [PATCH 2/2] Remove useless assert comments address: https://github.com/RIOT-OS/RIOT/pull/824#issuecomment-36426297 --- sys/crypto/aes/aes.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/sys/crypto/aes/aes.c b/sys/crypto/aes/aes.c index fdb5c0e7e9..ac65a0e3ce 100644 --- a/sys/crypto/aes/aes.c +++ b/sys/crypto/aes/aes.c @@ -961,7 +961,6 @@ int aes_encrypt(cipher_context_t *context, uint8_t *plainBlock, int r; #endif /* ?FULL_UNROLL */ - //assert(plainBlock && cipherBlock && key); is this really needed? rk = key->rd_key; /* @@ -1222,7 +1221,6 @@ int aes_decrypt(cipher_context_t *context, uint8_t *cipherBlock, int r; #endif /* ?FULL_UNROLL */ - // assert(cipherBlock && plainBlock && key); // nedded? see above rk = key->rd_key; /*