On Thu, 24 Sep 2026 06:29:10 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

>> `Win32PrintService.Win32ServiceUIFactory.getUIClassNamesForRole()` 
>> incorrectly creates a zero-length String array when handling 
>> `DocumentPropertiesUI.DOCUMENTPROPERTIES_ROLE`, then writes the Document 
>> Properties UI class name at index zero, so as
>> a result, calling 
>> `Win32PrintService.getServiceUIFactory().getUIClassNamesForRole(DocumentPropertiesUI.DOCUMENTPROPERTIES_ROLE)`
>> throws `ArrayIndexOutOfBoundsException `instead of reporting the supported 
>> UI class.
>> 
>> The fix returns a single-element array containing 
>> `DocumentPropertiesUI.DOCPROPERTIESCLASSNAME`
>> 
>> 
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Prasanta Sadhukhan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   null check

test/jdk/javax/print/GetUIClassNameForRole.java line 36:

> 34: import javax.print.ServiceUIFactory;
> 35: 
> 36: public class GetUIClassNameForRole {

Just a small nit, the name of the test probably should be 
`GetUIClassNamesForRole`

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/33009#discussion_r4131440931

Reply via email to