[DIRSERVER-652] Avoid multiple creation of LdapDN

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0-RC3
• Fix Version/s: 1.0-RC4
• Assignee: Unassigned
• Reporter: Emmanuel Lécharny
• Created: 24/Jun/06 15:32
• Updated: 23/Aug/06 12:33
• Resolved: 23/Aug/06 12:33

Description

After having profiled the server, I found that the LdapDN<init>(Name) has been called 3000 times, because the LdapPrincipal contains a Name instead of a LdapDN element.

We should check that the passed Name is not an instance of LdapDN before calling new LdapDN( Name) in
org.apache.directory.server.core.authz.DefaultAuthorizationService.protectLookUp() method
:
...
LdapDN principalDn = new LdapDN( ( ( ServerContext ) ctx ).getPrincipal().getJndiName() );
...

could be :
Name name = ( ( ServerContext ) ctx ).getPrincipal().getJndiName() ;

LdapDN principalDn= ( name instanceof LdapDN ? name.clone() : new LdapDN( name ) );