Browse Source

ppbloom:remove unnecessay bloom_check

bbloom_add and bbloom_check are in fact the same function.
let's make bbloom_add return collision information.
pull/2582/head
babarosaM33 5 years ago
parent
commit
9ae59819be
2 changed files with 2 additions and 3 deletions
  1. 3
      src/aead.c
  2. 2
      src/ppbloom.c

3
src/aead.c

@ -695,11 +695,10 @@ aead_decrypt(buffer_t *ciphertext, cipher_ctx_t *cipher_ctx, size_t capacity)
// Add the salt to bloom filter
if (cipher_ctx->init == 1) {
if (ppbloom_check((void *)cipher_ctx->salt, salt_len) == 1) {
if (ppbloom_add((void *)cipher_ctx->salt, salt_len) == 1) {
LOGE("crypto: AEAD: repeat salt detected");
return CRYPTO_ERROR;
}
ppbloom_add((void *)cipher_ctx->salt, salt_len);
cipher_ctx->init = 2;
}

2
src/ppbloom.c

@ -80,7 +80,7 @@ ppbloom_add(const void *buffer, int len)
{
int err;
err = bloom_add(ppbloom + current, buffer, len);
if (err == -1)
if (err == -1 || err == 1)
return err;
bloom_count[current]++;

Loading…
Cancel
Save