diff options
author | Ana Krivokapic <akrivoka@redhat.com> | 2013-01-03 08:40:40 -0500 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2013-01-14 14:09:54 +0100 |
commit | 38dded7db6529be096b92c9d63182a980e0b3a93 (patch) | |
tree | 30ea1e6c2bbffd8b521f4dd41aa3a6429708a260 | |
parent | 1d2d1e1af1660d022adca447f900403d171cba00 (diff) | |
download | freeipa-38dded7db6529be096b92c9d63182a980e0b3a93.tar.gz freeipa-38dded7db6529be096b92c9d63182a980e0b3a93.tar.xz freeipa-38dded7db6529be096b92c9d63182a980e0b3a93.zip |
Raise ValidationError for incorrect subtree option.
Ticket: https://fedorahosted.org/freeipa/ticket/3233
-rw-r--r-- | ipalib/plugins/aci.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/ipalib/plugins/aci.py b/ipalib/plugins/aci.py index 24be17458..7c4e8a549 100644 --- a/ipalib/plugins/aci.py +++ b/ipalib/plugins/aci.py @@ -341,7 +341,10 @@ def _aci_to_kw(ldap, a, test=False, pkey_only=False): else: # See if the target is a group. If so we set the # targetgroup attr, otherwise we consider it a subtree - targetdn = DN(target.replace('ldap:///','')) + try: + targetdn = DN(target.replace('ldap:///','')) + except ValueError as e: + raise errors.ValidationError(name='subtree', error=_("invalid DN (%s)") % e.message) if targetdn.endswith(DN(api.env.container_group, api.env.basedn)): kw['targetgroup'] = targetdn[0]['cn'] else: |