On Tue, 10 Nov 2020 11:37:04 GMT, Prajwal Kumaraswamy
<[email protected]> wrote:
> …ineResolveURI().
>
> Actual fix looks like this, due to git diff there are lot of changes( mostly
> because of the spaces) being displayed.
>
> --- a/src/share/classes/org/jcp/xml/dsig/internal/dom/DOMReference.java
> +++ b/src/share/classes/org/jcp/xml/dsig/internal/dom/DOMReference.java
> @@ -500,6 +500,7 @@
> }
>
> boolean secVal = Utils.secureValidation(context);
> + try {
> xi.setSecureValidation(secVal);
> if (context instanceof XMLSignContext && c14n11
> && !xi.isOctetStream() && !xi.isOutputStreamSet()) {
> @@ -533,6 +534,11 @@
> } else {
> xi.updateOutputStream(os);
> }
> + } finally {
> + if(xi.getOctetStreamReal() != null) {
> + xi.getOctetStreamReal().close();
> + }
> + }
> }
This pull request has now been integrated.
Changeset: 6e8b8628
Author: prajwal_kumaraswamy <[email protected]>
Committer: Sean Coffey <[email protected]>
URL: https://git.openjdk.java.net/jdk/commit/6e8b8628
Stats: 44 lines in 1 file changed: 17 ins; 11 del; 16 mod
8255559: Leak File Descriptors Because of
ResolverLocalFilesystem#engineResolveURI()
Reviewed-by: weijun
-------------
PR: https://git.openjdk.java.net/jdk/pull/1142
