Fix code copy-paste errors discovered by Coverity (#2639)

* Fix swapped arguments. * Fix a copy-paste error.
dotnet · Jan 26, 2017 · 8d2eb04 · 8d2eb04
1 parent 13459c1
commit 8d2eb04
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/OrleansRuntime/GrainDirectory/LocalGrainDirectory.cs b/src/OrleansRuntime/GrainDirectory/LocalGrainDirectory.cs
@@ -1028,7 +1028,7 @@ public override string ToString()
             sb.AppendLine("   Since last call:");
             sb.AppendFormat("      Local lookups: {0}", localLookupsDelta).AppendLine();
             sb.AppendFormat("      Local found: {0}", localLookupsSucceededDelta).AppendLine();
-            if (localLookupsCurrent > 0)
+            if (localLookupsDelta > 0)
                 sb.AppendFormat("      Hit rate: {0:F1}%", (100.0 * localLookupsSucceededDelta) / localLookupsDelta).AppendLine();
 
             sb.AppendFormat("      Full lookups: {0}", fullLookupsDelta).AppendLine();

diff --git a/src/OrleansRuntime/MembershipService/MembershipOracleData.cs b/src/OrleansRuntime/MembershipService/MembershipOracleData.cs
@@ -210,7 +210,7 @@ private static MembershipEntry CreateNewMembershipEntry(NodeConfiguration nodeCo
 
         internal void UpdateMyFaultAndUpdateZone(MembershipEntry entry)
         {
-            this.myFaultAndUpdateZones = new UpdateFaultCombo(entry.FaultZone, entry.UpdateZone);
+            this.myFaultAndUpdateZones = new UpdateFaultCombo(entry.UpdateZone, entry.FaultZone);
         }
 
         internal bool TryUpdateStatusAndNotify(MembershipEntry entry)