fixes #868

DialogResult is not currently checked, so it's a minimal/non-impact bug currently. But it's fixed now in case it's used in the future...

CHANGELOG.TXT

@@ -22,6 +22,7 @@ Fixes:
 #971: Portable Settings now apply to any machine they are used on
 #961: Connections file overwritten if correct decryption password not provided
 #893: Removed unneeded files from build/package
+#868: if statement returned the same value
 #762: Increased button size to fit locaized text
 
 

mRemoteV1/UI/Forms/PasswordForm.cs

@@ -55,12 +55,12 @@ namespace mRemoteNG.UI.Forms
 		}
 
 	    private void btnOK_Click(object sender, EventArgs e)
-		{
-			if (Verify && VerifyPassword())
-				DialogResult = DialogResult.OK;
-			else
-				DialogResult = DialogResult.OK;
-		}
+	    {
+	        if (Verify && VerifyPassword())
+	            DialogResult = DialogResult.OK;
+	        else
+	            DialogResult = DialogResult.None;
+	    }
 
 	    private void txtPassword_TextChanged(object sender, EventArgs e)
 		{