Fixed class allowing same name twice.

Author: KN4CK3R

Nodes/BaseNode.cs

@@ -28,7 +28,7 @@ public abstract class BaseNode
 		private string comment;
 
 		/// <summary>Gets or sets the name of the node. If a new name was set the property changed event gets fired.</summary>
-		public string Name { get { return name; } set { if (value != null && name != value) { name = value; NameChanged?.Invoke(this); } } }
+		public virtual string Name { get { return name; } set { if (value != null && name != value) { name = value; NameChanged?.Invoke(this); } } }
 
 		/// <summary>Gets or sets the offset of the node.</summary>
 		public IntPtr Offset { get; set; }

Nodes/ClassNode.cs

@@ -14,6 +14,22 @@ public class ClassNode : BaseContainerNode
 		/// <summary>Size of the node in bytes.</summary>
 		public override int MemorySize => Nodes.Sum(n => n.MemorySize);
 
+		public override string Name
+		{
+			get { return base.Name; }
+			set
+			{
+				if (!ClassManager.Classes.Any(c => c.Name == value))
+				{
+					base.Name = value;
+				}
+				else
+				{
+					Program.Logger.Log(Logger.LogLevel.Warning, $"The class name {value} is already in use.");
+				}
+			}
+		}
+
 		public IntPtr Address
 		{
 			set

UI/ClassNodeView.cs

@@ -158,18 +158,6 @@ public ClassNodeView()
 
 		#region Event Handler
 
-		private void classesTreeView_AfterLabelEdit(object sender, NodeLabelEditEventArgs e)
-		{
-			if (!string.IsNullOrEmpty(e.Label))
-			{
-				var node = e.Node as ClassTreeNode;
-				if (node != null)
-				{
-					node.ClassNode.Name = e.Label;
-				}
-			}
-		}
-
 		private void classesTreeView_AfterSelect(object sender, TreeViewEventArgs e)
 		{
 			if (e.Node.Level == 0)
@@ -241,6 +229,25 @@ private void renameClassToolStripMenuItem_Click(object sender, EventArgs e)
 			}
 		}
 
+		private void classesTreeView_AfterLabelEdit(object sender, NodeLabelEditEventArgs e)
+		{
+			if (!string.IsNullOrEmpty(e.Label))
+			{
+				var node = e.Node as ClassTreeNode;
+				if (node != null)
+				{
+					node.ClassNode.Name = e.Label;
+
+					// Cancel the edit if the class refused the name.
+					// This prevents the tree node from using the wrong name.
+					if (node.ClassNode.Name != e.Label)
+					{
+						e.CancelEdit = true;
+					}
+				}
+			}
+		}
+
 		private void autoExpandHierarchyViewToolStripMenuItem_Click(object sender, EventArgs e)
 		{
 			autoExpandHierarchyViewToolStripMenuItem.Checked = !autoExpandHierarchyViewToolStripMenuItem.Checked;