On 25.03.2024 21:45, Jason Andryuk wrote:
> @@ -145,13 +150,18 @@ elf_errorstatus elf_xen_parse_note(struct elf_binary
> *elf,
> elf_msg(elf, "ELF: note: %s = \"%s\"\n", note_desc[type].name, str);
> parms->elf_notes[type].type = XEN_ENT_STR;
> parms->elf_notes[type].data.str = str;
> - }
> - else
> - {
> + break;
> +
> + case ELFNOTE_INT:
> val = elf_note_numeric(elf, note);
> elf_msg(elf, "ELF: note: %s = %#" PRIx64 "\n", note_desc[type].name,
> val);
> parms->elf_notes[type].type = XEN_ENT_LONG;
> parms->elf_notes[type].data.num = val;
> + break;
> +
> + case ELFNOTE_NAME:
> + elf_msg(elf, "ELF: note: %s", note_desc[type].name);
> + break;
As indicated before, I think a brief comment is warranted here towards
the seemingly missing newline. With that added
Reviewed-by: Jan Beulich <jbeul...@suse.com>
Jan
