From: Michael Friedrich <michael.friedrich@netways.de>
Date: Sun, 15 Feb 2015 13:37:16 +0000 (+0100)
Subject: Fix NetString::ReadStringFromStream() EoF case in 'object list' command
X-Git-Tag: v2.3.0~198
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=2036b87943960a2ac03eb03a88b05a1531a0c7c1;p=icinga2

Fix NetString::ReadStringFromStream() EoF case in 'object list' command

refs #6109
---

diff --git a/lib/cli/objectlistcommand.cpp b/lib/cli/objectlistcommand.cpp
index 3efe16485..8cb346a72 100644
--- a/lib/cli/objectlistcommand.cpp
+++ b/lib/cli/objectlistcommand.cpp
@@ -83,8 +83,6 @@ int ObjectListCommand::Run(const boost::program_options::variables_map& vm, cons
 	unsigned long objects_count = 0;
 	std::map<String, int> type_count;
 
-	String message;
-	StreamReadContext src;
 	String name_filter, type_filter;
 
 	if (vm.count("name"))
@@ -94,7 +92,17 @@ int ObjectListCommand::Run(const boost::program_options::variables_map& vm, cons
 
 	bool first = true;
 
-	while (NetString::ReadStringFromStream(sfp, &message, src) == StatusNewItem) {
+	String message;
+	StreamReadContext src;
+	for (;;) {
+		StreamReadStatus srs = NetString::ReadStringFromStream(sfp, &message, src);
+
+		if (srs == StatusEof)
+			break;
+
+		if (srs != StatusNewItem)
+			continue;
+
 		PrintObject(std::cout, first, message, type_count, name_filter, type_filter);
 		objects_count++;
 	}