[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] dump: do not dump non-existent guest memory
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH] dump: do not dump non-existent guest memory |
Date: |
Tue, 12 Sep 2017 10:29:57 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Mon, Sep 11, 2017 at 03:26:27PM +0200, Cornelia Huck wrote:
> It does not really make sense to dump memory that is not there.
>
> Moreover, that fixes a segmentation fault when calling dump-guest-memory
> with no filter for a machine with no memory defined.
>
> New behaviour is:
>
> (qemu) dump-guest-memory /dev/null
> dump: no guest memory to dump
> (qemu) dump-guest-memory /dev/null 0 4096
> dump: no guest memory to dump
>
> Signed-off-by: Cornelia Huck <address@hidden>
Reviewed-by: Peter Xu <address@hidden>
> ---
>
> Another unmaintained file. Joy. cc:ing some more-or-less random folks.
>
> ---
> dump.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/dump.c b/dump.c
> index a79773d0f7..d2093e141b 100644
> --- a/dump.c
> +++ b/dump.c
> @@ -1536,6 +1536,12 @@ static void dump_init(DumpState *s, int fd, bool
> has_format,
> fprintf(stderr, "DUMP: total memory to dump: %lu\n", s->total_size);
> #endif
>
> + /* it does not make sense to dump non-existent memory */
> + if (!s->total_size) {
> + error_setg(errp, "dump: no guest memory to dump");
> + goto cleanup;
> + }
> +
> s->start = get_start_block(s);
> if (s->start == -1) {
> error_setg(errp, QERR_INVALID_PARAMETER, "begin");
> --
> 2.13.5
>
--
Peter Xu