summaryrefslogtreecommitdiff
path: root/dump.c
diff options
context:
space:
mode:
authorAneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>2013-10-01 21:49:32 +0530
committerAlexander Graf <agraf@suse.de>2013-10-25 23:25:48 +0200
commitbb6b684363e83586c90d20127b0d0a79793ab1e2 (patch)
tree4cec3e16954a626d780f0943b1b4b991fc243e30 /dump.c
parentd83af16786ca672bea9a206490f801bec7a057eb (diff)
downloadqemu-bb6b684363e83586c90d20127b0d0a79793ab1e2.tar.gz
dump-guest-memory: Check for the correct return value
We should check for error with s->note_size Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com> Signed-off-by: Alexander Graf <agraf@suse.de>
Diffstat (limited to 'dump.c')
-rw-r--r--dump.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/dump.c b/dump.c
index 846155cbc9..80a9116c77 100644
--- a/dump.c
+++ b/dump.c
@@ -66,7 +66,7 @@ typedef struct DumpState {
uint32_t sh_info;
bool have_section;
bool resume;
- size_t note_size;
+ ssize_t note_size;
hwaddr memory_offset;
int fd;
@@ -765,7 +765,7 @@ static int dump_init(DumpState *s, int fd, bool paging, bool has_filter,
s->note_size = cpu_get_note_size(s->dump_info.d_class,
s->dump_info.d_machine, nr_cpus);
- if (ret < 0) {
+ if (s->note_size < 0) {
error_set(errp, QERR_UNSUPPORTED);
goto cleanup;
}