Hopefully fix crash on unknown commands when readline is not supported

by bbackupqyuery, thanks to Achim J Latz for reporting and helping to debug.

2 files changed, 10 insertions, 19 deletions

diff --git a/bin/bbackupquery/BackupQueries.cpp b/bin/bbackupquery/BackupQueries.cpp
index a47274ff..aea6c896 100644
--- a/bin/bbackupquery/BackupQueries.cpp
+++ b/bin/bbackupquery/BackupQueries.cpp
@@ -108,8 +108,15 @@ BackupQueries::~BackupQueries()
 //
 // --------------------------------------------------------------------------
 void BackupQueries::DoCommand(ParsedCommand& rCommand)
-{	
+{
 	// Check...
+
+	if(rCommand.mFailed)
+	{
+		BOX_ERROR("Parse failed");
+		return;
+	}
+
 	if(rCommand.mCmdElements.size() < 1)
 	{
 		// blank command
diff --git a/bin/bbackupquery/bbackupquery.cpp b/bin/bbackupquery/bbackupquery.cpp
index 77f1d6e3..788db51f 100644
--- a/bin/bbackupquery/bbackupquery.cpp
+++ b/bin/bbackupquery/bbackupquery.cpp
@@ -469,15 +469,7 @@ int main(int argc, const char *argv[])
 		while(c < argc && !context.Stop())
 		{
 			BackupQueries::ParsedCommand cmd(argv[c++], true);
-
-			if(cmd.mFailed)
-			{
-				BOX_ERROR("Parse failed");
-			}
-			else
-			{
-				context.DoCommand(cmd);
-			}
+			context.DoCommand(cmd);
 		}
 	}
 	
@@ -525,15 +517,7 @@ int main(int argc, const char *argv[])
 			}
 			
 			BackupQueries::ParsedCommand cmd(command, false);
-			
-			if(cmd.mFailed)
-			{
-				BOX_ERROR("Parse failed");
-			}
-			else
-			{
-				context.DoCommand(cmd);
-			}
+			context.DoCommand(cmd);
 
 			if(last_cmd != 0 && ::strcmp(last_cmd, command) == 0)
 			{