From 79bd81c8da1a845ea22d18eb3012224abb0c6ed4 Mon Sep 17 00:00:00 2001
From: Jon Siwek <jsiwek@illinois.edu>
Date: Thu, 19 Sep 2013 11:39:52 -0500
Subject: [PATCH] Fix nesting/indent level whitespace mismatch.

The EndData() doesn't make sense as part of the condition that implies
the state is such that it's not inside data.  It might make sense as
part of an else block, but it also seems fine to unconditionally
EndData() like it currently does.  That way unexpected states (mail !=
0) are dealt with sooner rather than later.
---
 src/analyzer/protocol/smtp/SMTP.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/analyzer/protocol/smtp/SMTP.cc b/src/analyzer/protocol/smtp/SMTP.cc
index b496e97c69..c3fb21b6a4 100644
--- a/src/analyzer/protocol/smtp/SMTP.cc
+++ b/src/analyzer/protocol/smtp/SMTP.cc
@@ -599,7 +599,7 @@ void SMTP_Analyzer::UpdateState(const int cmd_code, const int reply_code)
 			case 0:
 				if ( st != SMTP_IN_DATA )
 					UnexpectedCommand(cmd_code, reply_code);
-					EndData();
+				EndData();
 				state = SMTP_AFTER_DATA;
 				break;