rsa: crash in br_i32_decode() called from rsa_gen_key_prop()
authorRobert Reither <robert.reither@external.thalesgroup.com>
Mon, 14 Sep 2020 11:12:02 +0000 (13:12 +0200)
committerHeinrich Schuchardt <xypron.glpk@gmx.de>
Mon, 14 Sep 2020 21:28:52 +0000 (23:28 +0200)
Fixes problem for unaligned 32bit big-endian access in
lib/rsa/rsa-keyprop.c.

Exchanges br_i32_decode() with get_unaligned_be32().

This will keep the unaligned access for architectures capable and will do
some byte-shift magic for the not so capable ones.

Reported-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Signed-by: Robert Reither <robert.reither@external.thalesgroup.com>
Remove unused include.
Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
lib/rsa/rsa-keyprop.c

index 1e83eed..98855f6 100644 (file)
@@ -12,9 +12,9 @@
 #include <common.h>
 #include <image.h>
 #include <malloc.h>
-#include <asm/byteorder.h>
 #include <crypto/internal/rsa.h>
 #include <u-boot/rsa-mod-exp.h>
+#include <asm/unaligned.h>
 
 /**
  * br_dec16be() - Convert 16-bit big-endian integer to native
@@ -23,7 +23,7 @@
  */
 static unsigned br_dec16be(const void *src)
 {
-       return be16_to_cpup(src);
+       return get_unaligned_be16(src);
 }
 
 /**
@@ -33,7 +33,7 @@ static unsigned br_dec16be(const void *src)
  */
 static uint32_t br_dec32be(const void *src)
 {
-       return be32_to_cpup(src);
+       return get_unaligned_be32(src);
 }
 
 /**