From 565438a6305013831f65a30fa5c108b49b72fded Mon Sep 17 00:00:00 2001
From: Philipp Crocoll <philipp@crocoll.net>
Date: Tue, 29 Sep 2015 01:20:25 +0200
Subject: [PATCH] check for duplicate keys in entry edit

---
 src/keepass2android/EntryEditActivity.cs         | 11 ++++++++++-
 src/keepass2android/Resources/values/strings.xml |  1 +
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/keepass2android/EntryEditActivity.cs b/src/keepass2android/EntryEditActivity.cs
index f55b3bc9..68fb3beb 100644
--- a/src/keepass2android/EntryEditActivity.cs
+++ b/src/keepass2android/EntryEditActivity.cs
@@ -1015,6 +1015,7 @@ namespace keepass2android
 
 
 			LinearLayout container = (LinearLayout) FindViewById(Resource.Id.advanced_container);
+			HashSet<string> allKeys = new HashSet<string>();
 			for (int i = 0; i < container.ChildCount; i++) {
 				View ees = container.GetChildAt(i);
 				
@@ -1025,7 +1026,15 @@ namespace keepass2android
 					Toast.MakeText(this, Resource.String.error_string_key, ToastLength.Long).Show();
 					return false;
 				}
-				
+
+				if (allKeys.Contains(key))
+				{
+					Toast.MakeText(this, GetString(Resource.String.error_string_duplicate_key, new Java.Lang.Object[]{key}), ToastLength.Long).Show();
+					return false;
+				}
+
+				allKeys.Add(key);
+
 			}
 			
 			return true;
diff --git a/src/keepass2android/Resources/values/strings.xml b/src/keepass2android/Resources/values/strings.xml
index 488e1891..f5eea0cf 100644
--- a/src/keepass2android/Resources/values/strings.xml
+++ b/src/keepass2android/Resources/values/strings.xml
@@ -277,6 +277,7 @@
 
 	<string name="error_invalid_expiry_date">Invalid date/time format for expiry date!</string>
   <string name="error_string_key">A field name is required for each string.</string>
+  <string name="error_string_duplicate_key">A field name cannot be used twice (%1$s).</string>
   <string name="field_name">Field Name</string>
   <string name="field_value">Field value</string>
   <string name="protection">Protected field</string>