Bug 817401

Summary: ipa permission-add does not fail if using invalid attribute
Product: Red Hat Enterprise Linux 7 Reporter: Dmitri Pal <dpal>
Component: ipaAssignee: Martin Kosek <mkosek>
Status: CLOSED WONTFIX QA Contact: IDM QE LIST <seceng-idm-qe-list>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 7.0CC: dpal, jgalipea, mkosek, nsoman
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 783502 Environment:
Last Closed: 2015-01-16 13:46:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 783502    
Bug Blocks:    

Comment 1 Martin Kosek 2012-05-03 06:56:45 UTC
Upstream ticket:
https://fedorahosted.org/freeipa/ticket/2714

Comment 4 Martin Kosek 2015-01-16 13:46:29 UTC
permission-add should fail when permission is added on an *unknown* attribute, which it does, in IdM/FreeIPA 4.1 and later:

# ipa permission-add ManageUser --permissions=write --type=user --attr=foo
ipa: ERROR: targetattr "foo" does not exist in schema. Please add attributeTypes "foo" to schema if necessary. ACL Syntax Error(-5):(targetattr = \22foo\22)(targetfilter = \22(objectclass=posixaccount)\22)(version 3.0;acl \22permission:ManageUser\22;allow (write) groupdn = \22ldap:///cn=ManageUser,cn=permissions,cn=pbac,dc=mkosek-f21,dc=test\22;): Invalid syntax.

Additional checks on top of that would lower the flexibility of the command (default list of user objectclasses can be extended by plugins or other means).