Merge pull request #28 from DBCDK/fix-27

Make sure the limit operator doesn't cause out-of-bounds errors
DBCDK · Nov 15, 2018 · 16861df · 16861df
2 parents 0794c50 + 2d111df
commit 16861df
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/filter/filter.go b/filter/filter.go
@@ -29,8 +29,8 @@ func FilterHosts(allHosts []nix.Host, skip int, every int, limit int) (hosts []n
 		}
 	}
 
-	// limit to $limit hosts
-	if limit > 0 {
+	// limit to $limit hosts, making sure not to go out of bounds either
+	if limit > 0 && limit < len(hosts) {
 		return hosts[:limit]
 	} else {
 		return hosts