RFR: 8368606: Printer lookup returns empty on AIX platform due to uninitialized results list [v3]
Harshitha Onkar
honkar at openjdk.org
Mon Oct 6 23:58:47 UTC 2025
On Mon, 29 Sep 2025 04:17:34 GMT, Ravi-Patel8 <duke at openjdk.org> wrote:
>> Bug Ref: https://bugs.openjdk.org/browse/JDK-8368606
>>
>> As part of [JDK-8344057](https://bugs.openjdk.org/browse/JDK-8344057) ("Remove doPrivileged calls from unix platform sources in the java.desktop module"), changes were made to execCmd() in
>> `src/java.desktop/unix/classes/sun/print/PrintServiceLookupProvider.java`.
>> In the updated implementation, execCmd(...) code is missing to initialize the results variable. It declares it as null and then tries to results.add(line), which prevents the results list from being populated with printer names.
>>
>> Initializing the results variable will resolve the issue.
>>
>>
>> Signed-off-by: Ravi.Patel8 <Ravi.Patel8 at ibm.com>
>
> Ravi-Patel8 has updated the pull request incrementally with one additional commit since the last revision:
>
> Add debug_println instead of System.err.println
>
> Signed-off-by: Ravi.Patel8 <Ravi.Patel8 at ibm.com>
Changes LGTM
-------------
Marked as reviewed by honkar (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/27482#pullrequestreview-3307877138
More information about the client-libs-dev
mailing list