From 9147f0da69b9f0b4181696a9a716a687674ccf64 Mon Sep 17 00:00:00 2001 From: Rob Crittenden Date: Wed, 13 May 2009 16:52:30 -0400 Subject: Don't pass non-existent arguments to _handle_errors() --- ipaserver/ipaldap.py | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) (limited to 'ipaserver') diff --git a/ipaserver/ipaldap.py b/ipaserver/ipaldap.py index d60e23fd..3d0b321e 100644 --- a/ipaserver/ipaldap.py +++ b/ipaserver/ipaldap.py @@ -297,7 +297,7 @@ class IPAdmin(SimpleLDAPObject): # re-raise the error so we can handle it raise e except ldap.NO_SUCH_OBJECT, e: - args = kw.get('args', '') + args = kw.get('args', ["entry not found"]) raise errors.NotFound(reason=notfound(args)) except ldap.ALREADY_EXISTS, e: raise errors.DuplicateEntry() @@ -459,8 +459,7 @@ class IPAdmin(SimpleLDAPObject): self.set_option(ldap.OPT_SERVER_CONTROLS, sctrl) self.modrdn_s(dn, newrdn, delold=1) except ldap.LDAPError, e: - kw = {'args': args} - self.__handle_errors(e, **kw) + self.__handle_errors(e, **{}) return True def updateEntry(self,dn,oldentry,newentry): @@ -479,8 +478,7 @@ class IPAdmin(SimpleLDAPObject): self.set_option(ldap.OPT_SERVER_CONTROLS, sctrl) self.modify_s(dn, modlist) except ldap.LDAPError, e: - kw = {'args': args} - self.__handle_errors(e, **kw) + self.__handle_errors(e, **{}) return True def generateModList(self, old_entry, new_entry): @@ -538,8 +536,7 @@ class IPAdmin(SimpleLDAPObject): self.set_option(ldap.OPT_SERVER_CONTROLS, sctrl) self.modify_s(dn, modlist) except ldap.LDAPError, e: - kw = {'args': args} - self.__handle_errors(e, **kw) + self.__handle_errors(e, **{}) return True def deleteEntry(self,*args): @@ -571,8 +568,7 @@ class IPAdmin(SimpleLDAPObject): self.set_option(ldap.OPT_SERVER_CONTROLS, sctrl) self.passwd_s(dn, oldpass, newpass) except ldap.LDAPError, e: - kw = {'args': args} - self.__handle_errors(e, **kw) + self.__handle_errors(e, **{}) return True def __wrapmethods(self): -- cgit