diff options
| author | Petr Viktorin <pviktori@redhat.com> | 2012-04-17 12:48:33 -0400 |
|---|---|---|
| committer | Martin Kosek <mkosek@redhat.com> | 2012-05-14 10:38:07 +0200 |
| commit | c5689e7fafc688165945e83dd4bf63dd568b3650 (patch) | |
| tree | d1bd88acdf92fabde68c0fbfd3d25f089113547d /ipalib/plugins/selfservice.py | |
| parent | 95bb8d0f45c3061035633c7b9c3b1609b11a158d (diff) | |
| download | freeipa-c5689e7fafc688165945e83dd4bf63dd568b3650.tar.gz freeipa-c5689e7fafc688165945e83dd4bf63dd568b3650.tar.xz freeipa-c5689e7fafc688165945e83dd4bf63dd568b3650.zip | |
Do not use extra command options in ACI, permission, selfservice
Allowing Commands to be called with ignored unknown options opens the
door to problems, for example with misspelled option names.
Before we start rejecting them, we need to make sure IPA itself does
not use them when it calls commands internally.
This patch does that for ACI-related plugins.
Part of the work for https://fedorahosted.org/freeipa/ticket/2509
Diffstat (limited to 'ipalib/plugins/selfservice.py')
| -rw-r--r-- | ipalib/plugins/selfservice.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/ipalib/plugins/selfservice.py b/ipalib/plugins/selfservice.py index a60475b7c..82f2a0cc0 100644 --- a/ipalib/plugins/selfservice.py +++ b/ipalib/plugins/selfservice.py @@ -149,8 +149,7 @@ class selfservice_del(crud.Delete): msg_summary = _('Deleted selfservice "%(value)s"') def execute(self, aciname, **kw): - kw['aciprefix'] = ACI_PREFIX - result = api.Command['aci_del'](aciname, **kw) + result = api.Command['aci_del'](aciname, aciprefix=ACI_PREFIX) self.obj.postprocess_result(result) return dict( |