[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH -V3 3/4] target-ppc: Check for error on address tr
From: |
Alexander Graf |
Subject: |
Re: [Qemu-ppc] [PATCH -V3 3/4] target-ppc: Check for error on address translation in memsave command |
Date: |
Sun, 25 Aug 2013 19:32:58 +0100 |
On 23.08.2013, at 06:20, Aneesh Kumar K.V wrote:
> From: "Aneesh Kumar K.V" <address@hidden>
>
> When we translate the virtual address to physical check for error.
>
> Signed-off-by: Aneesh Kumar K.V <address@hidden>
I think this change is sane, but I'd really prefer to see an ack from (or get
this applied by) Luiz.
Alex
> ---
> cpus.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/cpus.c b/cpus.c
> index 0f65e76..658366d 100644
> --- a/cpus.c
> +++ b/cpus.c
> @@ -1309,7 +1309,10 @@ void qmp_memsave(int64_t addr, int64_t size, const
> char *filename,
> l = sizeof(buf);
> if (l > size)
> l = size;
> - cpu_memory_rw_debug(cpu, addr, buf, l, 0);
> + if (cpu_memory_rw_debug(cpu, addr, buf, l, 0) != 0) {
> + error_set(errp, QERR_INVALID_PARAMETER, "addr");
> + goto exit;
> + }
> if (fwrite(buf, 1, l, f) != l) {
> error_set(errp, QERR_IO_ERROR);
> goto exit;
> --
> 1.8.1.2
>
- Re: [Qemu-ppc] [PATCH -V3 2/4] target-ppc: Fix page table lookup with kvm enabled, (continued)