[v4] target/arm: Scalable Matrix Extension

[PATCH v4 38/45] linux-user/aarch64: Verify extra record lock succeeded

Posted by Richard Henderson 3 years, 7 months ago

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
 linux-user/aarch64/signal.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/linux-user/aarch64/signal.c b/linux-user/aarch64/signal.c
index 8fbe98d72f..9ff79da4be 100644
--- a/linux-user/aarch64/signal.c
+++ b/linux-user/aarch64/signal.c
@@ -340,6 +340,9 @@ static int target_restore_sigframe(CPUARMState *env,
             __get_user(extra_size,
                        &((struct target_extra_context *)ctx)->size);
             extra = lock_user(VERIFY_READ, extra_datap, extra_size, 0);
+            if (!extra) {
+                return 1;
+            }
             break;
 
         default:
-- 
2.34.1

Re: [PATCH v4 38/45] linux-user/aarch64: Verify extra record lock succeeded

Posted by Peter Maydell 3 years, 7 months ago

On Tue, 28 Jun 2022 at 06:05, Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>  linux-user/aarch64/signal.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/linux-user/aarch64/signal.c b/linux-user/aarch64/signal.c
> index 8fbe98d72f..9ff79da4be 100644
> --- a/linux-user/aarch64/signal.c
> +++ b/linux-user/aarch64/signal.c
> @@ -340,6 +340,9 @@ static int target_restore_sigframe(CPUARMState *env,
>              __get_user(extra_size,
>                         &((struct target_extra_context *)ctx)->size);
>              extra = lock_user(VERIFY_READ, extra_datap, extra_size, 0);
> +            if (!extra) {
> +                return 1;
> +            }
>              break;
>
>          default:
> --

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

thanks
-- PMM