Integrated: 8293815: P11PSSSignature.engineUpdate should not print debug messages during normal operation

zzambers duke at openjdk.org
Thu Sep 15 01:09:18 UTC 2022


On Wed, 14 Sep 2022 17:03:41 GMT, zzambers <duke at openjdk.org> wrote:

> During testing I have found messages such as this are printed to stdout:
> 
> sun.security.pkcs11.P11PSSSignature at 6e1567f1: Calling C_SignUpdate
> 
> 
> **Problem:**
> Problem is in P11PSSSignature.engineUpdate method [1], which unconditionally prints some debug information to stdout. Other prints in that class are conditional based on value of DEBUG field.
> 
> **Solution:**
> I have made fix to make these conditional based on value of DEBUG field. As it only affects debug messages, there is no test added.
> 
> **Testing:**
> Passed jdk_security locally with my fix.
> 
> [1] https://github.com/openjdk/jdk/blob/211fab8d361822bbd1a34a88626853bf4a029af5/src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11PSSSignature.java#L578

This pull request has now been integrated.

Changeset: 2057070e
Author:    Zdenek Zambersky <zzambers at redhat.com>
Committer: Valerie Peng <valeriep at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/2057070eb069ccee20760e47bd5e09590850d2ce
Stats:     4 lines in 1 file changed: 0 ins; 0 del; 4 mod

8293815: P11PSSSignature.engineUpdate should not print debug messages during normal operation

Reviewed-by: valeriep

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

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



More information about the security-dev mailing list