From ffad599ea12f3f14156292e4ce13e794d7777c18 Mon Sep 17 00:00:00 2001
From: Vital7 <vitalyachuma@rambler.ru>
Date: Mon, 25 Jun 2018 01:39:02 +0300
Subject: [PATCH] Fix logical issues found with static analyzer (#828)

* Fix possible logical issues found with static analyzer
Bot:2216 - there is already a null check before
IPC:612 - shouldn't it be 'if' because we get out of the loop anyway?

* Revert IPC.cs change
---
 ArchiSteamFarm/Bot.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ArchiSteamFarm/Bot.cs b/ArchiSteamFarm/Bot.cs
index 34b7385f1..075057678 100755
--- a/ArchiSteamFarm/Bot.cs
+++ b/ArchiSteamFarm/Bot.cs
@@ -2213,7 +2213,7 @@ namespace ArchiSteamFarm {
 			}
 
 			string loginKey = callback.LoginKey;
-			if (!string.IsNullOrEmpty(loginKey) && (BotConfig.PasswordFormat != CryptoHelper.ECryptoMethod.PlainText)) {
+			if (BotConfig.PasswordFormat != CryptoHelper.ECryptoMethod.PlainText) {
 				loginKey = CryptoHelper.Encrypt(BotConfig.PasswordFormat, loginKey);
 			}