Fix ParseMachineId (#459)

Fix issue parsing manual machine id; fixes #458 - second parameter of `Substring` is 'length' not 'index'
Azure · Oct 22, 2019 · 47f5ec7 · 47f5ec7
1 parent d4bb7ae
commit 47f5ec7
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/DotNetty.Transport/Channels/DefaultChannelId.cs b/src/DotNetty.Transport/Channels/DefaultChannelId.cs
@@ -123,7 +123,7 @@ static byte[] ParseMachineId(string value)
             var machineId = new byte[MachineIdLen];
             for (int i = 0; i < value.Length; i += 2)
             {
-                machineId[i] = (byte)int.Parse(value.Substring(i, i + 2), NumberStyles.AllowHexSpecifier);
+                machineId[i] = (byte)int.Parse(value.Substring(i, 2), NumberStyles.AllowHexSpecifier);
             }
             return machineId;
         }
@@ -251,4 +251,4 @@ public override bool Equals(object obj)
 
         public override string ToString() => this.AsShortText();
     }
-}
+}