crypto: keywrap - Add missing ULL suffixes for 64-bit constants
diff mbox series

Message ID 1510742668-21121-1-git-send-email-geert@linux-m68k.org
State New, archived
Headers show
Series
  • crypto: keywrap - Add missing ULL suffixes for 64-bit constants
Related show

Commit Message

Geert Uytterhoeven Nov. 15, 2017, 10:44 a.m. UTC
On 32-bit (e.g. with m68k-linux-gnu-gcc-4.1):

    crypto/keywrap.c: In function ‘crypto_kw_decrypt’:
    crypto/keywrap.c:191: warning: integer constant is too large for ‘long’ type
    crypto/keywrap.c: In function ‘crypto_kw_encrypt’:
    crypto/keywrap.c:224: warning: integer constant is too large for ‘long’ type

Fixes: 9e49451d7a15365d ("crypto: keywrap - simplify code")
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
 crypto/keywrap.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Stephan Müller Nov. 15, 2017, 2:26 p.m. UTC | #1
Am Mittwoch, 15. November 2017, 11:44:28 CET schrieb Geert Uytterhoeven:

Hi Geert,

> On 32-bit (e.g. with m68k-linux-gnu-gcc-4.1):
> 
>     crypto/keywrap.c: In function ‘crypto_kw_decrypt’:
>     crypto/keywrap.c:191: warning: integer constant is too large for ‘long’
> type crypto/keywrap.c: In function ‘crypto_kw_encrypt’:
>     crypto/keywrap.c:224: warning: integer constant is too large for ‘long’
> type
> 
> Fixes: 9e49451d7a15365d ("crypto: keywrap - simplify code")
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>

Thank you

Reviewed-by: Stephan Mueller <smueller@chronox.de>

Ciao
Stephan
Herbert Xu Nov. 29, 2017, 6:36 a.m. UTC | #2
On Wed, Nov 15, 2017 at 11:44:28AM +0100, Geert Uytterhoeven wrote:
> On 32-bit (e.g. with m68k-linux-gnu-gcc-4.1):
> 
>     crypto/keywrap.c: In function ‘crypto_kw_decrypt’:
>     crypto/keywrap.c:191: warning: integer constant is too large for ‘long’ type
>     crypto/keywrap.c: In function ‘crypto_kw_encrypt’:
>     crypto/keywrap.c:224: warning: integer constant is too large for ‘long’ type
> 
> Fixes: 9e49451d7a15365d ("crypto: keywrap - simplify code")
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>

Patch applied.  Thanks.

Patch
diff mbox series

diff --git a/crypto/keywrap.c b/crypto/keywrap.c
index 744e35134c45f300..ec5c6a087c901501 100644
--- a/crypto/keywrap.c
+++ b/crypto/keywrap.c
@@ -188,7 +188,7 @@  static int crypto_kw_decrypt(struct blkcipher_desc *desc,
 	}
 
 	/* Perform authentication check */
-	if (block.A != cpu_to_be64(0xa6a6a6a6a6a6a6a6))
+	if (block.A != cpu_to_be64(0xa6a6a6a6a6a6a6a6ULL))
 		ret = -EBADMSG;
 
 	memzero_explicit(&block, sizeof(struct crypto_kw_block));
@@ -221,7 +221,7 @@  static int crypto_kw_encrypt(struct blkcipher_desc *desc,
 	 * Place the predefined IV into block A -- for encrypt, the caller
 	 * does not need to provide an IV, but he needs to fetch the final IV.
 	 */
-	block.A = cpu_to_be64(0xa6a6a6a6a6a6a6a6);
+	block.A = cpu_to_be64(0xa6a6a6a6a6a6a6a6ULL);
 
 	/*
 	 * src scatterlist is read-only. dst scatterlist is r/w. During the