[#DIRSERVER-192] Error encoding PRESENCE filters

Directory ApacheDS

Error encoding PRESENCE filters

Created: 15/Dec/05 12:06 PM Updated: 07/Feb/06 02:41 PM

Return to search

Component/s:

ldap

Affects Version/s:

pre-1.0

Fix Version/s:

pre-1.0

Time Tracking:

Not Specified

File Attachments:

	File Name	Date Attached	Attached By	Size
	patch_missing_break_after_case_PRESENCE.patch	2005-12-15 12:08 PM	Van Nhu	3 kB

Environment:

« Hide

Module: https://svn.apache.org/repos/asf/directory/shared/ldap/trunk
Subversion revision number: 356972

Resolution Date:

15/Dec/05 06:31 PM

Description

« Hide

Missing a break statement after case( LeafNode.PRESENCE ): in the SearchRequestEncoder.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Repository	Revision	Date	User	Message
ASF	#356993	Thu Dec 15 09:26:51 UTC 2005	elecharny	Fixed DIRLDAP-80, a bug found by Van-Nhu : missing a break in a switch
				Files Changed
				MODIFY /directory/shared/ldap/trunk/apache-provider/src/main/java/org/apache/ldap/common/berlib/asn1/encoder/search/SearchRequestEncoder.java

[ Permlink | « Hide ]

Van Nhu added a comment - 15/Dec/05 12:08 PM

The attached patch adds a missing break statement that causes errors when encoding PRESENCE filters.

Van Nhu made changes - 15/Dec/05 12:08 PM

Field	Original Value	New Value
Attachment		patch_missing_break_after_case_PRESENCE.patch [ 12321336 ]

[ Permlink | « Hide ]

Emmanuel Lecharny added a comment - 15/Dec/05 06:31 PM

Thanks a lot for the accurate report and patch ! I tested it an applied it immediatly.

I don't know if this is you or me who have to close the bug, but as the testcase is OK and as the patch is obviously correct, I have marked the bug as fixed.

The jar file will be deployed very soon.

Good work, man !

Emmanuel Lecharny made changes - 15/Dec/05 06:31 PM

Resolution		Fixed [ 1 ]
Status	Open [ 1 ]	Resolved [ 5 ]

[ Permlink | « Hide ]

Van Nhu added a comment - 16/Dec/05 11:19 AM

tested and it works.

Van Nhu made changes - 16/Dec/05 11:19 AM

Status

Resolved [ 5 ]

Closed [ 6 ]

Alex Karasulu made changes - 07/Feb/06 02:41 PM

Component/s	Snickers Codec Provider [ 11083 ]
Project	Directory LDAP [ 10514 ]	ApacheDS [ 12310260 ]
Key	DIRLDAP-80	~~DIRSERVER-192~~
Affects Version/s		pre-1.0 [ 12310782 ]
Component/s		ldap [ 12310715 ]
Fix Version/s		pre-1.0 [ 12310782 ]
Affects Version/s	0.9.3 [ 12310221 ]