RFR: 8296442: EncryptedPrivateKeyInfo can be created with an uninitialized AlgorithmParameters [v3]
Weijun Wang
weijun at openjdk.org
Thu Nov 10 17:59:44 UTC 2022
> An `EncryptedPrivateKeyInfo` object can be created with an uninitialized `AlgorithmParameters`, but before you call `getEncoded` on it you need to remember to initialize the params. This is unfortunate but since this is a public API, I hesitate to make a change.
>
> Instead, this code change fixes the much more widely used internal class `AlgorithmId` so that it cannot be created with an uninitialized `AlgorithmParameters`. `EncryptedPrivateKeyInfo` now works with both initialized and uninitialized params, and it's immutable.
>
> No intention to make `AlgorithmId` immutable this time. It has a child class named `AlgIdDSA` which makes things complicated.
Weijun Wang has updated the pull request incrementally with one additional commit since the last revision:
expected
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/11067/files
- new: https://git.openjdk.org/jdk/pull/11067/files/499f50a8..619c6f44
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=11067&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=11067&range=01-02
Stats: 9 lines in 1 file changed: 0 ins; 5 del; 4 mod
Patch: https://git.openjdk.org/jdk/pull/11067.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/11067/head:pull/11067
PR: https://git.openjdk.org/jdk/pull/11067
More information about the security-dev
mailing list