Looks good to me.
Regards
Prasanta
On 8/18/2016 9:34 PM, Jayathirth D V wrote:
Hi,
Please review the following fix in JDK9 at your convenience:
Bug : https://bugs.openjdk.java.net/browse/JDK-8163258
Webrev : http://cr.openjdk.java.net/~jdv/8163258/webrev.00/
<http://cr.openjdk.java.net/%7Ejdv/8163258/webrev.00/>
Issue : When we call ImageIO.getReaderMIMETypes() or ImageIO.
getFileSuffixes() with MIMEType or FileSuffix as null in subclass
ImageReaderSpi it throws NullPointerException.
Root cause : We allow MIMEType and FileSuffix to be null in
ImageReaderWriterSpi. But when we call getReaderMIMETypes() or
getFileSuffixes() we try populate HashSet using Collections.addALL()
which throws NPE.
Solution : Before we populate HashSet using Collections.addALL()
verify whether the input is null.
Thanks,
Jay