summaryrefslogtreecommitdiff
path: root/dump.c
diff options
context:
space:
mode:
authorMarc-André Lureau <marcandre.lureau@redhat.com>2017-12-12 15:53:59 +0100
committerMarc-André Lureau <marcandre.lureau@redhat.com>2018-01-02 14:49:54 +0100
commitc983ca845782bf91171b5ec4e59d122a21229745 (patch)
treef9507b25d23631e007ab1ae988bb28978e6612f6 /dump.c
parent281f327487c9c9b1599f93c589a408bbf4a651b8 (diff)
downloadqemu-c983ca845782bf91171b5ec4e59d122a21229745.tar.gz
dump: fix note_name_equal()
Use the function argument "name" instead of hardcoded "VMCOREINFO". All callers use "VMCOREINFO" as argument, so this isn't an exposed bug, thankfully. Simplify a little bit the code while touching this. Suggested-by: Andrew Jones <drjones@redhat.com> Reported-by: Laszlo Ersek <lersek@redhat.com> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Diffstat (limited to 'dump.c')
-rw-r--r--dump.c7
1 files changed, 1 insertions, 6 deletions
diff --git a/dump.c b/dump.c
index d4a8c942eb..e9dfed060a 100644
--- a/dump.c
+++ b/dump.c
@@ -788,12 +788,7 @@ static bool note_name_equal(DumpState *s,
get_note_sizes(s, note, &head_size, &name_size, NULL);
head_size = ROUND_UP(head_size, 4);
- if (name_size != len ||
- memcmp(note + head_size, "VMCOREINFO", len)) {
- return false;
- }
-
- return true;
+ return name_size == len && memcmp(note + head_size, name, len) == 0;
}
/* write common header, sub header and elf note to vmcore */