net: ipv6: stop checking crypto_ahash_alignmask
authorEric Biggers <ebiggers@google.com>
Sun, 22 Oct 2023 08:10:49 +0000 (01:10 -0700)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 27 Oct 2023 10:04:29 +0000 (18:04 +0800)
Now that the alignmask for ahash and shash algorithms is always 0,
crypto_ahash_alignmask() always returns 0 and will be removed.  In
preparation for this, stop checking crypto_ahash_alignmask() in ah6.c.

Signed-off-by: Eric Biggers <ebiggers@google.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
net/ipv6/ah6.c

index 56f9282ec5df4338ecd44d96385a204f7e045617..2016e90e6e1d21a49696c9933f1b77320cc71953 100644 (file)
@@ -51,9 +51,7 @@ static void *ah_alloc_tmp(struct crypto_ahash *ahash, int nfrags,
 {
        unsigned int len;
 
-       len = size + crypto_ahash_digestsize(ahash) +
-             (crypto_ahash_alignmask(ahash) &
-              ~(crypto_tfm_ctx_alignment() - 1));
+       len = size + crypto_ahash_digestsize(ahash);
 
        len = ALIGN(len, crypto_tfm_ctx_alignment());
 
@@ -75,10 +73,9 @@ static inline u8 *ah_tmp_auth(u8 *tmp, unsigned int offset)
        return tmp + offset;
 }
 
-static inline u8 *ah_tmp_icv(struct crypto_ahash *ahash, void *tmp,
-                            unsigned int offset)
+static inline u8 *ah_tmp_icv(void *tmp, unsigned int offset)
 {
-       return PTR_ALIGN((u8 *)tmp + offset, crypto_ahash_alignmask(ahash) + 1);
+       return tmp + offset;
 }
 
 static inline struct ahash_request *ah_tmp_req(struct crypto_ahash *ahash,
@@ -299,7 +296,7 @@ static void ah6_output_done(void *data, int err)
 
        iph_base = AH_SKB_CB(skb)->tmp;
        iph_ext = ah_tmp_ext(iph_base);
-       icv = ah_tmp_icv(ahp->ahash, iph_ext, extlen);
+       icv = ah_tmp_icv(iph_ext, extlen);
 
        memcpy(ah->auth_data, icv, ahp->icv_trunc_len);
        memcpy(top_iph, iph_base, IPV6HDR_BASELEN);
@@ -362,7 +359,7 @@ static int ah6_output(struct xfrm_state *x, struct sk_buff *skb)
 
        iph_ext = ah_tmp_ext(iph_base);
        seqhi = (__be32 *)((char *)iph_ext + extlen);
-       icv = ah_tmp_icv(ahash, seqhi, seqhi_len);
+       icv = ah_tmp_icv(seqhi, seqhi_len);
        req = ah_tmp_req(ahash, icv);
        sg = ah_req_sg(ahash, req);
        seqhisg = sg + nfrags;
@@ -468,7 +465,7 @@ static void ah6_input_done(void *data, int err)
 
        work_iph = AH_SKB_CB(skb)->tmp;
        auth_data = ah_tmp_auth(work_iph, hdr_len);
-       icv = ah_tmp_icv(ahp->ahash, auth_data, ahp->icv_trunc_len);
+       icv = ah_tmp_icv(auth_data, ahp->icv_trunc_len);
 
        err = crypto_memneq(icv, auth_data, ahp->icv_trunc_len) ? -EBADMSG : 0;
        if (err)
@@ -576,7 +573,7 @@ static int ah6_input(struct xfrm_state *x, struct sk_buff *skb)
 
        auth_data = ah_tmp_auth((u8 *)work_iph, hdr_len);
        seqhi = (__be32 *)(auth_data + ahp->icv_trunc_len);
-       icv = ah_tmp_icv(ahash, seqhi, seqhi_len);
+       icv = ah_tmp_icv(seqhi, seqhi_len);
        req = ah_tmp_req(ahash, icv);
        sg = ah_req_sg(ahash, req);
        seqhisg = sg + nfrags;