db/upnp: break when count==0

Make this non-fatal. Just stop reading more objects.

db/upnp: break when count==0
7573d4e1 · Max Kellermann · bfb48389 · 7573d4e1
Commit 7573d4e1 authored Jan 22, 2014 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 6 deletions

ContentDirectoryService.cxx src/db/upnp/ContentDirectoryService.cxx +3 -6

No files found.
--- a/src/db/upnp/ContentDirectoryService.cxx
+++ b/src/db/upnp/ContentDirectoryService.cxx
@@ -191,12 +191,6 @@ ContentDirectoryService::search(UpnpClient_Handle hdl,
 			? ParseUnsigned(value)
 			: 0;

-		if (count == 0) {
-			// TODO: what's this?
-			error.Set(upnp_domain, "got -1 or 0 entries");
-			return false;
-		}
-
 		offset += count;

 		value = ixmlwrap::getFirstElementValue(response, "TotalMatches");
@@ -205,6 +199,9 @@ ContentDirectoryService::search(UpnpClient_Handle hdl,

 		if (!ReadResultTag(dirbuf, response, error))
 			return false;
+
+		if (count == 0)
+			break;
 	}

 	return true;