FuzzingTeam commented on code in PR #4957: URL: https://github.com/apache/hadoop/pull/4957#discussion_r985709429
########## hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/TestDefaultStringifier.java: ########## @@ -107,6 +108,12 @@ public void testStoreLoadArray() throws IOException { Integer[] array = new Integer[] {1,2,3,4,5}; + try { Review Comment: we have updated the code, please review. ########## hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/DefaultStringifier.java: ########## @@ -158,6 +158,9 @@ public static <K> K load(Configuration conf, String keyName, public static <K> void storeArray(Configuration conf, K[] items, String keyName) throws IOException { + if (items.length == 0) { + throw new IndexOutOfBoundsException(); + } DefaultStringifier<K> stringifier = new DefaultStringifier<K>(conf, GenericsUtil.getClass(items[0])); Review Comment: Thanks @steveloughran for the review. Yes, the original code would trigger ArrayIndexOutOfBoundsException automatically, and the new test passes without the new lines. But we found the same exception has been handled in other places in the same module and hence, request to handle it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org