[U-Boot] [PATCH V2 3/4] cmd: aes: use map_sysmem when accessing memory
Simon Glass
sjg at chromium.org
Mon Sep 23 17:33:24 UTC 2019
On Mon, 23 Sep 2019 at 09:39, Philippe Reynes
<philippe.reynes at softathome.com> wrote:
>
> The aes command used to segfault when accessing memory in sandbox.
> The pointer accesses should be mapped.
>
> Signed-off-by: Philippe Reynes <philippe.reynes at softathome.com>
> ---
> cmd/aes.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/cmd/aes.c b/cmd/aes.c
> index 7ff4a71..3db110c 100644
> --- a/cmd/aes.c
> +++ b/cmd/aes.c
> @@ -11,6 +11,7 @@
> #include <malloc.h>
> #include <asm/byteorder.h>
> #include <linux/compiler.h>
> +#include <mapmem.h>
>
> /**
> * do_aes() - Handle the "aes" command-line command
> @@ -46,10 +47,10 @@ static int do_aes(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[])
> dst_addr = simple_strtoul(argv[5], NULL, 16);
> len = simple_strtoul(argv[6], NULL, 16);
>
> - key_ptr = (uint8_t *)key_addr;
> - iv_ptr = (uint8_t *)iv_addr;
> - src_ptr = (uint8_t *)src_addr;
> - dst_ptr = (uint8_t *)dst_addr;
> + key_ptr = (uint8_t *)map_sysmem(key_addr, 128 / 8);
> + iv_ptr = (uint8_t *)map_sysmem(iv_addr, 128 / 8);
> + src_ptr = (uint8_t *)map_sysmem(src_addr, len);
> + dst_ptr = (uint8_t *)map_sysmem(dst_addr, len);
>
> /* First we expand the key. */
> aes_expand_key(key_ptr, key_exp);
Please can you unmap_sysmem() as well?
Regards,
SImon
More information about the U-Boot
mailing list