Skip to content

Commit

Permalink
mac802154: Use skcipher
Browse files Browse the repository at this point in the history
This patch replaces uses of blkcipher with skcipher.

Signed-off-by: Herbert Xu <[email protected]>
Reviewed-by: Stefan Schmidt <[email protected]>
  • Loading branch information
herbertx committed Jan 27, 2016
1 parent e59dd98 commit 9695371
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 20 deletions.
41 changes: 23 additions & 18 deletions net/mac802154/llsec.c
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@
#include <linux/err.h>
#include <linux/bug.h>
#include <linux/completion.h>
#include <linux/crypto.h>
#include <linux/ieee802154.h>
#include <crypto/aead.h>
#include <crypto/skcipher.h>

#include "ieee802154_i.h"
#include "llsec.h"
Expand Down Expand Up @@ -144,18 +144,18 @@ llsec_key_alloc(const struct ieee802154_llsec_key *template)
goto err_tfm;
}

key->tfm0 = crypto_alloc_blkcipher("ctr(aes)", 0, CRYPTO_ALG_ASYNC);
key->tfm0 = crypto_alloc_skcipher("ctr(aes)", 0, CRYPTO_ALG_ASYNC);
if (IS_ERR(key->tfm0))
goto err_tfm;

if (crypto_blkcipher_setkey(key->tfm0, template->key,
IEEE802154_LLSEC_KEY_SIZE))
if (crypto_skcipher_setkey(key->tfm0, template->key,
IEEE802154_LLSEC_KEY_SIZE))
goto err_tfm0;

return key;

err_tfm0:
crypto_free_blkcipher(key->tfm0);
crypto_free_skcipher(key->tfm0);
err_tfm:
for (i = 0; i < ARRAY_SIZE(key->tfm); i++)
if (key->tfm[i])
Expand All @@ -175,7 +175,7 @@ static void llsec_key_release(struct kref *ref)
for (i = 0; i < ARRAY_SIZE(key->tfm); i++)
crypto_free_aead(key->tfm[i]);

crypto_free_blkcipher(key->tfm0);
crypto_free_skcipher(key->tfm0);
kzfree(key);
}

Expand Down Expand Up @@ -620,15 +620,17 @@ llsec_do_encrypt_unauth(struct sk_buff *skb, const struct mac802154_llsec *sec,
{
u8 iv[16];
struct scatterlist src;
struct blkcipher_desc req = {
.tfm = key->tfm0,
.info = iv,
.flags = 0,
};
SKCIPHER_REQUEST_ON_STACK(req, key->tfm0);
int err;

llsec_geniv(iv, sec->params.hwaddr, &hdr->sec);
sg_init_one(&src, skb->data, skb->len);
return crypto_blkcipher_encrypt_iv(&req, &src, &src, skb->len);
skcipher_request_set_tfm(req, key->tfm0);
skcipher_request_set_callback(req, 0, NULL, NULL);
skcipher_request_set_crypt(req, &src, &src, skb->len, iv);
err = crypto_skcipher_encrypt(req);
skcipher_request_zero(req);
return err;
}

static struct crypto_aead*
Expand Down Expand Up @@ -830,19 +832,22 @@ llsec_do_decrypt_unauth(struct sk_buff *skb, const struct mac802154_llsec *sec,
unsigned char *data;
int datalen;
struct scatterlist src;
struct blkcipher_desc req = {
.tfm = key->tfm0,
.info = iv,
.flags = 0,
};
SKCIPHER_REQUEST_ON_STACK(req, key->tfm0);
int err;

llsec_geniv(iv, dev_addr, &hdr->sec);
data = skb_mac_header(skb) + skb->mac_len;
datalen = skb_tail_pointer(skb) - data;

sg_init_one(&src, data, datalen);

return crypto_blkcipher_decrypt_iv(&req, &src, &src, datalen);
skcipher_request_set_tfm(req, key->tfm0);
skcipher_request_set_callback(req, 0, NULL, NULL);
skcipher_request_set_crypt(req, &src, &src, datalen, iv);

err = crypto_skcipher_decrypt(req);
skcipher_request_zero(req);
return err;
}

static int
Expand Down
3 changes: 1 addition & 2 deletions net/mac802154/llsec.h
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@

#include <linux/slab.h>
#include <linux/hashtable.h>
#include <linux/crypto.h>
#include <linux/kref.h>
#include <linux/spinlock.h>
#include <net/af_ieee802154.h>
Expand All @@ -30,7 +29,7 @@ struct mac802154_llsec_key {

/* one tfm for each authsize (4/8/16) */
struct crypto_aead *tfm[3];
struct crypto_blkcipher *tfm0;
struct crypto_skcipher *tfm0;

struct kref ref;
};
Expand Down

0 comments on commit 9695371

Please sign in to comment.