Skip to contentSkip to conversation

KVM: PPC: fix suspicious use of conditional operator

PATCH Applied Laurentiu Tudor 2 participants 2 messages May 25, 2015

Subsystems (3)
KVM PowerPC KVM PowerPC
lore
Share & tools
lore

Laurentiu Tudor

author
#0 lore raw

PATCH 1/1 KVM: PPC: fix suspicious use of conditional operator

This was signaled by a static code analysis tool.

Signed-off-by Laurentiu Tudor <[email protected]>
Reviewed-by Scott Wood <[email protected]>
---
 arch/powerpc/kvm/e500_mmu.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/powerpc/kvm/e500_mmu.c +1 -1
index 50860e9..29911a0 100644
@@ -377,7 +377,7 @@ int kvmppc_e500_emul_tlbsx(struct kvm_vcpu *vcpu, gva_t ea)
 			| MAS0_NV(vcpu_e500->gtlb_nv[tlbsel]);
 		vcpu->arch.shared->mas1  			  (vcpu->arch.shared->mas6 & MAS6_SPID0)
-			| (vcpu->arch.shared->mas6 & (MAS6_SAS ? MAS1_TS : 0))
+			| ((vcpu->arch.shared->mas6 & MAS6_SAS) ? MAS1_TS : 0)
 			| (vcpu->arch.shared->mas4 & MAS4_TSIZED(~0));
 		vcpu->arch.shared->mas2 &= MAS2_EPN;
 		vcpu->arch.shared->mas2 |= vcpu->arch.shared->mas4 &
Signature · 1 line
1.8.3.1