[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 08/15] target/ppc: Don't initialize some local variables in ppc_ra
From: |
David Gibson |
Subject: |
[PULL 08/15] target/ppc: Don't initialize some local variables in ppc_radix64_xlate() |
Date: |
Wed, 27 May 2020 15:38:02 +1000 |
From: Greg Kurz <address@hidden>
It is the job of the ppc_radix64_get_fully_qualified_addr() function
which is called at the beginning of ppc_radix64_xlate() to set both
lpid *and* pid. It doesn't buy us anything to initialize them first.
Worse, a bug in ppc_radix64_get_fully_qualified_addr(), eg. failing to
set either lpid or pid, would be undetectable by static analysis tools
like coverity.
Some recent versions of gcc (eg. gcc-9.3.1-2.fc30) may still think
that lpid or pid is used uninitialized though, so this also adds
default cases in the switch statements to make it clear this cannot
happen.
Signed-off-by: Greg Kurz <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Cédric Le Goater <address@hidden>
Signed-off-by: David Gibson <address@hidden>
---
target/ppc/mmu-radix64.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/target/ppc/mmu-radix64.c b/target/ppc/mmu-radix64.c
index c76879f65b..07f956c986 100644
--- a/target/ppc/mmu-radix64.c
+++ b/target/ppc/mmu-radix64.c
@@ -50,6 +50,8 @@ static bool ppc_radix64_get_fully_qualified_addr(const
CPUPPCState *env,
*lpid = 0;
*pid = 0;
break;
+ default:
+ g_assert_not_reached();
}
} else { /* !MSR[HV] -> Guest */
switch (eaddr & R_EADDR_QUADRANT) {
@@ -64,6 +66,8 @@ static bool ppc_radix64_get_fully_qualified_addr(const
CPUPPCState *env,
*lpid = env->spr[SPR_LPIDR];
*pid = 0; /* pid set to 0 -> addresses guest operating system */
break;
+ default:
+ g_assert_not_reached();
}
}
@@ -433,7 +437,7 @@ static int ppc_radix64_xlate(PowerPCCPU *cpu, vaddr eaddr,
int rwx,
bool cause_excp)
{
CPUPPCState *env = &cpu->env;
- uint64_t lpid = 0, pid = 0;
+ uint64_t lpid, pid;
ppc_v3_pate_t pate;
int psize, prot;
hwaddr g_raddr;
--
2.26.2
- [PULL 00/15] ppc-for-5.1 queue 20200527, David Gibson, 2020/05/27
- [PULL 03/15] target/ppc: Untabify excp_helper.c, David Gibson, 2020/05/27
- [PULL 01/15] ppc/pnv: Fix NMI system reset SRR1 value, David Gibson, 2020/05/27
- [PULL 08/15] target/ppc: Don't initialize some local variables in ppc_radix64_xlate(),
David Gibson <=
- [PULL 05/15] ppc/spapr: Add hotremovable flag on DIMM LMBs on drmem_v2, David Gibson, 2020/05/27
- [PULL 02/15] ppc/spapr: add a POWER10 CPU model, David Gibson, 2020/05/27
- [PULL 10/15] target/ppc: Fix arguments to ppc_radix64_partition_scoped_xlate(), David Gibson, 2020/05/27
- [PULL 07/15] target/ppc: Pass const pointer to ppc_radix64_get_fully_qualified_addr(), David Gibson, 2020/05/27
- [PULL 06/15] target/ppc: Pass const pointer to ppc_radix64_get_prot_amr(), David Gibson, 2020/05/27
- [PULL 12/15] hw/pci-bridge/dec: Remove dead debug code, David Gibson, 2020/05/27
- [PULL 04/15] target/ppc: Add support for scv and rfscv instructions, David Gibson, 2020/05/27
- [PULL 09/15] target/ppc: Add missing braces in ppc_radix64_partition_scoped_xlate(), David Gibson, 2020/05/27
- [PULL 11/15] target/ppc: Don't update radix PTE R/C bits with gdbstub, David Gibson, 2020/05/27
- [PULL 14/15] target/ppc: Fix argument to ppc_radix64_partition_scoped_xlate() again, David Gibson, 2020/05/27