[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/5] target-mips: fix compiler warnings (clang 5
From: |
Yongbok Kim |
Subject: |
Re: [Qemu-devel] [PATCH 1/5] target-mips: fix compiler warnings (clang 5) |
Date: |
Mon, 13 Mar 2017 14:59:53 +0000 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
On 04/03/2017 18:56, Philippe Mathieu-Daudé wrote:
> static code analyzer complain:
>
> target/mips/helper.c:453:5: warning: Function call argument is an
> uninitialized value
> qemu_log_mask(CPU_LOG_MMU,
> ^~~~~~~~~~~~~~~~~~~~~~~~~~
>
> 'physical' and 'prot' are uninitialized if 'ret' is not TLBRET_MATCH.
>
> Reported-by: Clang Static Analyzer
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> target/mips/helper.c | 16 ++++++++++++----
> 1 file changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/target/mips/helper.c b/target/mips/helper.c
> index d2e77958fd..e359ca3b44 100644
> --- a/target/mips/helper.c
> +++ b/target/mips/helper.c
> @@ -450,10 +450,18 @@ int mips_cpu_handle_mmu_fault(CPUState *cs, vaddr
> address, int rw,
> access_type = ACCESS_INT;
> ret = get_physical_address(env, &physical, &prot,
> address, rw, access_type);
> - qemu_log_mask(CPU_LOG_MMU,
> - "%s address=%" VADDR_PRIx " ret %d physical " TARGET_FMT_plx
> - " prot %d\n",
> - __func__, address, ret, physical, prot);
> + switch (ret) {
> + case TLBRET_MATCH:
> + qemu_log_mask(CPU_LOG_MMU,
> + "%s address=%" VADDR_PRIx " physical " TARGET_FMT_plx
> + " prot %d\n", __func__, address, physical, prot);
> + break;
> + default:
> + qemu_log_mask(CPU_LOG_MMU,
> + "%s address=%" VADDR_PRIx " ret %d\n", __func__,
> address,
> + ret);
> + break;
> + }
> if (ret == TLBRET_MATCH) {
> tlb_set_page(cs, address & TARGET_PAGE_MASK,
> physical & TARGET_PAGE_MASK, prot | PAGE_EXEC,
>
Reviewed-by: Yongbok Kim <address@hidden>
- [Qemu-devel] [PATCH 0/5] target-mips: fixes, start using trace points, Philippe Mathieu-Daudé, 2017/03/04
- [Qemu-devel] [PATCH 1/5] target-mips: fix compiler warnings (clang 5), Philippe Mathieu-Daudé, 2017/03/04
- Re: [Qemu-devel] [PATCH 1/5] target-mips: fix compiler warnings (clang 5),
Yongbok Kim <=
- [Qemu-devel] [PATCH 2/5] target-mips: remove old & unuseful comments, Philippe Mathieu-Daudé, 2017/03/04
- [Qemu-devel] [PATCH 3/5] target-mips: log bad coprocessor0 register accesses with LOG_UNIMP, Philippe Mathieu-Daudé, 2017/03/04
- [Qemu-devel] [PATCH 5/5] target-mips: replace few LOG_DISAS() with trace points, Philippe Mathieu-Daudé, 2017/03/04
- [Qemu-devel] [PATCH 4/5] target-mips: replace break by goto cp0_unimplemented, Philippe Mathieu-Daudé, 2017/03/04