RFR: 8368606: Printer lookup returns empty on AIX platform due to uninitialized results list [v2]

Ravi-Patel8 duke at openjdk.org
Thu Sep 25 11:23:18 UTC 2025


> 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 two additional commits since the last revision:

 - Updated Comments
   
   Signed-off-by: Ravi.Patel8 <Ravi.Patel8 at ibm.com>
 - Add catch block to log exception details for troubleshooting printer
   command
   
   Signed-off-by: Ravi.Patel8 <Ravi.Patel8 at ibm.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/27482/files
  - new: https://git.openjdk.org/jdk/pull/27482/files/92c3ec8e..80fcec26

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=27482&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=27482&range=00-01

  Stats: 4 lines in 1 file changed: 3 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/27482.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/27482/head:pull/27482

PR: https://git.openjdk.org/jdk/pull/27482


More information about the client-libs-dev mailing list