Re: [PATCH v2 05/10] target/xtensa: add FIXME for translation memory leak

[Max Filippov]

On Wed, Apr 1, 2020 at 2:48 AM Alex Bennée <address@hidden> wrote:
>
> Dynamically allocating a new structure within the DisasContext can
> potentially leak as we can longjmp out of the translation loop (see
> test_phys_mem). The proper fix would be to use static allocation
> within the DisasContext but as the Xtensa translator imports it's code
> from elsewhere I leave that as an exercise for the maintainer.
>
> Signed-off-by: Alex Bennée <address@hidden>
> Cc: Max Filippov <address@hidden>
> ---
>  target/xtensa/translate.c | 5 +++++
>  1 file changed, 5 insertions(+)

Acked-by: Max Filippov <address@hidden>

-- 
Thanks.
-- Max