Issue Details (XML | Word | Printable)

Key: DIRSERVER-177
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Alex Karasulu
Reporter: Alex Karasulu
Votes: 0
Watchers: 0
Operations

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

Bad filter makes filter parser hang rather than throw an exception

Created: 07/Feb/06 06:36 AM   Updated: 27/Aug/06 07:55 AM
Return to search
Component/s: ldap
Affects Version/s: None
Fix Version/s: None

Time Tracking:
Not Specified

Resolution Date: 27/Aug/06 07:55 AM


 Description  « Hide
try simple malformed filter like 'blah' and see what happends:

line 1:1: expecting LPAREN, found 'blah'

This is printed out to system.err and no exception results. This is not good because it can hang indefinately.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Alex Karasulu made changes - 07/Feb/06 02:39 PM
Field Original Value New Value
Key DIRLDAP-88 DIRSERVER-177
Project Directory LDAP [ 10514 ] ApacheDS [ 12310260 ]
Alex Karasulu made changes - 10/Feb/06 02:16 PM
Component/s ldap [ 12310715 ]
Stefan Zoerner added a comment - 15/Apr/06 08:20 PM
Is it possible to get more info on this?

Within my test cases, I have some search operations here:
https://svn.apache.org/repos/asf/directory/trunks/testsuites/src/main/java/org/apache/ldap/testsuite/ldaptests/jndi/ops/search/IllegalSearchFilterTest.java
The class contains a test method testFiltersWithSyntaxErrors which expects an InvalidSearchFilterException in case of a filter like "This is no filter" or "blah". It works as expected for RC1, but it seems to me that it is the JNDI provider for LDAP on the client side causing the exception (no ApacheDS involved).

How was the error achieved in detail? What does "try simple malformed filter" exactly means? command line tool like ldapsearch?

Emmanuel Lecharny added a comment - 22/Aug/06 11:51 PM
ok, the issue is 4 months old, and we asked for more info, but didn't get some.

We may close this issue and reopen it if needed. wdyt ?

Alex Karasulu made changes - 27/Aug/06 07:54 AM
Assignee Alex Karasulu [ akarasulu ]
Alex Karasulu added a comment - 27/Aug/06 07:55 AM
I think this was fixed a while back. Sorry guys this escaped me. I did not know it was out there. I allways like to provide status :).

Alex Karasulu made changes - 27/Aug/06 07:55 AM
Resolution Fixed [ 1 ]
Status Open [ 1 ] Closed [ 6 ]