> One `PKCS9Attribute` can be created but cannot be encoded. Since the > `SigningCertificateInfo::parse` method has not fully parsed the data > (`PolicyInformation` is left out), this code change add the encoding itself > as a field to the `SigningCertificateInfo` class so we can encode it. > > After this change, unsupported `PKCSAttribute` object simply cannot be > created. The `new(DerValue)` constructor rejects them (type 9-13, 15) in a > `switch` block, and the `new(ObjectIdentifier, Object)` constructor rejects > them because `VALUE_CLASSES` for them are null. > > In the `encode()` method, we now throw `IllegalArgumentException` for these > types and they will not happen.
Weijun Wang has updated the pull request incrementally with one additional commit since the last revision: enhance test ------------- Changes: - all: https://git.openjdk.org/jdk/pull/11070/files - new: https://git.openjdk.org/jdk/pull/11070/files/efc69f01..e8af4309 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=11070&range=03 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=11070&range=02-03 Stats: 6 lines in 1 file changed: 5 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jdk/pull/11070.diff Fetch: git fetch https://git.openjdk.org/jdk pull/11070/head:pull/11070 PR: https://git.openjdk.org/jdk/pull/11070