[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 3/6] target/ppc: Don't initialize some local variables in ppc_rad
From: |
Greg Kurz |
Subject: |
[PATCH 3/6] target/ppc: Don't initialize some local variables in ppc_radix64_xlate() |
Date: |
Thu, 07 May 2020 19:26:50 +0200 |
User-agent: |
StGit/unknown-version |
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.
Signed-off-by: Greg Kurz <address@hidden>
---
target/ppc/mmu-radix64.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/ppc/mmu-radix64.c b/target/ppc/mmu-radix64.c
index c76879f65b78..5e2d912ee346 100644
--- a/target/ppc/mmu-radix64.c
+++ b/target/ppc/mmu-radix64.c
@@ -433,7 +433,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;
- [PATCH 0/6] target/ppc: Various clean-up and fixes for radix64, Greg Kurz, 2020/05/07
- [PATCH 1/6] target/ppc: Pass const pointer to ppc_radix64_get_prot_amr(), Greg Kurz, 2020/05/07
- [PATCH 2/6] target/ppc: Pass const pointer to ppc_radix64_get_fully_qualified_addr(), Greg Kurz, 2020/05/07
- [PATCH 3/6] target/ppc: Don't initialize some local variables in ppc_radix64_xlate(),
Greg Kurz <=
- [PATCH 4/6] target/ppc: Add missing braces in ppc_radix64_partition_scoped_xlate(), Greg Kurz, 2020/05/07
- [PATCH 5/6] target/ppc: Fix arguments to ppc_radix64_partition_scoped_xlate(), Greg Kurz, 2020/05/07
- [PATCH 6/6] target/ppc: Don't update radix PTE R/C bits with gdbstub, Greg Kurz, 2020/05/07
- Re: [PATCH 0/6] target/ppc: Various clean-up and fixes for radix64, David Gibson, 2020/05/10