From e3b887f770db983393cd920ce8699a9aeb6630e2 Mon Sep 17 00:00:00 2001 From: James Cammarata Date: Fri, 7 Mar 2014 16:34:04 -0600 Subject: [PATCH] Fix range issue in inventory and add additional error checking Fixes #6331 --- lib/ansible/inventory/__init__.py | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/lib/ansible/inventory/__init__.py b/lib/ansible/inventory/__init__.py index 8f74d5ea9e..171a4f2a04 100644 --- a/lib/ansible/inventory/__init__.py +++ b/lib/ansible/inventory/__init__.py @@ -208,12 +208,14 @@ class Inventory(object): """ # The regex used to match on the range, which can be [x] or [x-y]. - pattern_re = re.compile("^(.*)\[([0-9]+)(?:(?:-)([0-9]+))?\](.*)$") + pattern_re = re.compile("^(.*)\[([-]?[0-9]+)(?:(?:-)([0-9]+))?\](.*)$") m = pattern_re.match(pattern) if m: (target, first, last, rest) = m.groups() first = int(first) if last: + if first < 0: + raise errors.AnsibleError("invalid range: negative indices cannot be used as the first item in a range") last = int(last) else: last = first @@ -245,10 +247,13 @@ class Inventory(object): right = 0 left=int(left) right=int(right) - if left != right: - return hosts[left:right] - else: - return [ hosts[left] ] + try: + if left != right: + return hosts[left:right] + else: + return [ hosts[left] ] + except IndexError: + raise errors.AnsibleError("no hosts matching the pattern '%s' were found" % pat) def _create_implicit_localhost(self, pattern): new_host = Host(pattern)