Re: [PATCH] plugins/execlog.c: correct dump of registers values

[Pierrick Bouvier <pierrick.bouvier@linaro.org>]

On 6/20/24 01:38, Frédéric Pétrot wrote:
> Register values are dumped as 'sz' chunks of two nibbles in the execlog
> plugin, sz was 1 too big.
> 
> Signed-off-by: Frédéric Pétrot <frederic.petrot@univ-grenoble-alpes.fr>
> ---
>   contrib/plugins/execlog.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/contrib/plugins/execlog.c b/contrib/plugins/execlog.c
> index 371db97eb1..1c1601cc0b 100644
> --- a/contrib/plugins/execlog.c
> +++ b/contrib/plugins/execlog.c
> @@ -101,7 +101,7 @@ static void insn_check_regs(CPU *cpu)
>               GByteArray *temp = reg->last;
>               g_string_append_printf(cpu->last_exec, ", %s -> 0x", reg->name);
>               /* TODO: handle BE properly */
> -            for (int i = sz; i >= 0; i--) {
> +            for (int i = sz - 1; i >= 0; i--) {
>                   g_string_append_printf(cpu->last_exec, "%02x",
>                                          reg->new->data[i]);
>               }

Good catch, thanks!
Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>

Pierrick