CODEC-315: Fix possible IndexOutOfBoundException

Signed-off-by: Arthur Chan <arthur.chan@adalogics.com>

Author: arthurscchan

src/main/java/org/apache/commons/codec/language/bm/PhoneticEngine.java

@@ -409,7 +409,7 @@ public String encode(String input, final Languages.LanguageSet languageSet) {
         switch (this.nameType) {
         case SEPHARDIC:
             words.forEach(aWord -> {
-                final String[] parts = aWord.split("'");
+                final String[] parts = aWord.split("'", -1);
                 words2.add(parts[parts.length - 1]);
             });
             words2.removeAll(NAME_PREFIXES.get(this.nameType));
@@ -431,7 +431,7 @@ public String encode(String input, final Languages.LanguageSet languageSet) {
         } else if (words2.size() == 1) {
             // not a multi-word name
             input = words.iterator().next();
-        } else {
+        } else if (!words2.isEmpty()) {
             // encode each word in a multi-word name separately (normally used for approx matches)
             final StringBuilder result = new StringBuilder();
             words2.forEach(word -> result.append("-").append(encode(word)));

src/test/java/org/apache/commons/codec/language/bm/PhoneticEngineTest.java

@@ -17,6 +17,7 @@
 
 package org.apache.commons.codec.language.bm;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
@@ -48,6 +49,15 @@ public static Stream<Arguments> data() {
                 );
     }
 
+    public static Stream<Arguments> invalidData() {
+        return Stream.of(
+                        Arguments.of("bar", "bar|bor|var|vor", NameType.ASHKENAZI, RuleType.APPROX, Boolean.FALSE, TEN),
+                        Arguments.of("al", "|al", NameType.SEPHARDIC, RuleType.APPROX, Boolean.FALSE, TEN),
+                        Arguments.of("da", "da|di", NameType.GENERIC, RuleType.EXACT, Boolean.FALSE, TEN),
+                        Arguments.of("'''", "", NameType.SEPHARDIC, RuleType.APPROX, Boolean.FALSE, TEN)
+                );
+    }
+
     // TODO Identify if there is a need to an assertTimeout(Duration.ofMillis(10000L) in some point, since this method was marked as @Test(timeout = 10000L)
     @ParameterizedTest
     @MethodSource("data")
@@ -70,4 +80,15 @@ public void testEncode(final String name, final String phoneticExpected, final N
             }
         }
     }
+
+    @ParameterizedTest
+    @MethodSource("invalidData")
+    public void testInvalidEncode(final String input, final String phoneticExpected, final NameType nameType,
+                                  final RuleType ruleType, final boolean concat, final int maxPhonemes) {
+        final PhoneticEngine engine = new PhoneticEngine(nameType, ruleType, concat, maxPhonemes);
+
+        assertDoesNotThrow(() -> {
+            assertEquals(engine.encode(input), phoneticExpected);
+        });
+    }
 }