Issue Details (XML | Word | Printable)

Key: DIRSERVER-252
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Blocker Blocker
Assignee: Alex Karasulu
Reporter: Emmanuel Lecharny
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Directory ApacheDS

Stack trace overflow when trying to connect with a bad DN

Created: 05/Dec/05 05:01 PM   Updated: 24/Jun/06 08:43 PM
Return to search
Component/s: None
Affects Version/s: pre-1.0
Fix Version/s: 1.0-RC1

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works stackTrace.log 2005-12-05 05:02 PM Emmanuel Lecharny 102 kB
Issue Links:
Cloners
 

Resolution Date: 03/Feb/06 09:14 PM


 Description  « Hide
When trying to Bind with a bad DN, I get a StackTraceOverflow exception.

I use uid=adminBad, for instance

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Emmanuel Lecharny added a comment - 07/Dec/05 12:09 PM
Sorry, Alex, but you are the only one who can easily fix this one ;)

Emmanuel Lecharny added a comment - 07/Dec/05 12:12 PM
The problem arise when the server tries to create the matchedName that will be returned in the error message. Yhis is due to the fact that the matchedName method will do a authenticate(), and as the bad DN is the principal, this authenticate() will generate an error, so a matchedName will be created, and this is of course an infinite loop ;=)

Note : I just try to do a Bind with a non existing uid. Fortunatly, we get a StackOverflowException, but the server is still alive !

Emmanuel Lecharny added a comment - 09/Dec/05 04:54 AM
This problem is known, and is actually under heavy investigation.

Thanks for the report !

Emmanuel Lecharny added a comment - 21/Jan/06 01:36 AM
Raised to blocker

Alex Karasulu added a comment - 03/Feb/06 09:14 PM
I think I fixed this one pretty much when doing cleanups. I cannot reproduce it anymore.

Committed revision 374649.

Please confirm otherwise we'll have to reopen this one.

Emmanuel Lecharny added a comment - 24/Jun/06 08:43 PM
I think it's fixed.