Add a fix and a test for DIRAPI-400

Author: elecharny

ldap/model/src/main/java/org/apache/directory/api/ldap/model/url/LdapUrl.java

@@ -638,7 +638,7 @@ private int parseIPV6( char[] chars, int pos )
         // Search for the closing ']'
         int start = pos;
 
-        while ( !Chars.isCharASCII( chars, pos, ']' ) )
+        while ( !Chars.isCharASCII( chars, pos, ']' ) && ( pos < chars.length ) )
         {
             pos++;
         }
@@ -699,7 +699,7 @@ private int parseIPvFuture( char[] chars, int pos )
         // Now, we should have at least one char in unreserved / sub-delims / ":"
         boolean valueFound = false;
 
-        while ( !Chars.isCharASCII( chars, pos, ']' ) )
+        while ( !Chars.isCharASCII( chars, pos, ']' ) && ( pos < chars.length ) )
         {
             switch ( chars[pos] )
             {
@@ -901,7 +901,7 @@ private int parseDecOctet( char[] chars, int pos, int[] ipElem, int octetNb )
         boolean ipElemSeen = false;
         boolean hasHeadingZeroes = false;
 
-        while ( Chars.isDigit( chars, pos ) )
+        while ( Chars.isDigit( chars, pos ) && ( pos < chars.length ) )
         {
             ipElemSeen = true;
 
@@ -974,7 +974,7 @@ private int parsePort( char[] chars, int pos )
 
         pos++;
 
-        while ( Chars.isDigit( chars, pos ) )
+        while ( Chars.isDigit( chars, pos ) && ( pos < chars.length ) )
         {
             port = ( port * 10 ) + ( chars[pos] - '0' );
 

ldap/model/src/test/java/org/apache/directory/api/ldap/model/url/LdapUrlTest.java

@@ -1683,4 +1683,11 @@ public void testLdapURLExtensionWithRFC3986UnreservedChars() throws Exception
             "ldap://localhost:123/????X-CONNECTION-NAME=abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789-._~",
             url2.toString() );
     }
+    
+    
+    @Test
+    public void hangingLdapUrl()
+    {
+    	assertThrows( LdapURLEncodingException.class, () -> new LdapUrl( "ldap://[1:2:ldap:///o" ) );
+    }
 }