Re: RFR: 8296400: pointCrlIssuers might be null in DistributionPointFetcher::verifyURL [v2]

2023-03-09 Thread Weijun Wang
On Thu, 9 Mar 2023 20:53:44 GMT, Matthew Donovan wrote: >> Added null-checks for pointCrlIssuers before it is accessed. > > Matthew Donovan has updated the pull request incrementally with one > additional commit since the last revision: > > fixed indentation Looks good. You might need to add

Re: RFR: 8296400: pointCrlIssuers might be null in DistributionPointFetcher::verifyURL [v2]

2023-03-09 Thread Matthew Donovan
> Added null-checks for pointCrlIssuers before it is accessed. Matthew Donovan has updated the pull request incrementally with one additional commit since the last revision: fixed indentation - Changes: - all: https://git.openjdk.org/jdk/pull/12866/files - new: https://git.op

Re: RFR: 8296400: pointCrlIssuers might be null in DistributionPointFetcher::verifyURL

2023-03-09 Thread Weijun Wang
On Fri, 3 Mar 2023 19:45:22 GMT, Matthew Donovan wrote: > Added null-checks for pointCrlIssuers before it is accessed. src/java.base/share/classes/sun/security/provider/certpath/DistributionPointFetcher.java line 485: > 483: // cannot all be missing from a certificate. > 48

RFR: 8296400: pointCrlIssuers might be null in DistributionPointFetcher::verifyURL

2023-03-03 Thread Matthew Donovan
Added null-checks for pointCrlIssuers before it is accessed. - Commit messages: - 8296400: pointCrlIssuers might be null in DistributionPointFetcher::verifyURL Changes: https://git.openjdk.org/jdk/pull/12866/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=12866&range=00 Is