On Mon, 26 Apr 2021 01:17:43 GMT, Hamlin Li <m...@openjdk.org> wrote:

> code like below will create Deflater before null check, although it's not a 
> real mem leak, but it's better to do null check before new Deflater.
> 
>         try {
>             DeflaterOutputStream dos = new DeflaterOutputStream(null);
>         } catch (NullPointerException e) {
>             passed = true;
>         }
> Similar issues exist in several other classes.

This pull request has now been integrated.

Changeset: 15d47877
Author:    Hamlin Li <m...@openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/15d47877
Stats:     15 lines in 8 files changed: 0 ins; 0 del; 15 mod

8265496: improve null check in DeflaterOutputStream/InflaterInputStream

Reviewed-by: lancea, naoto

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

PR: https://git.openjdk.java.net/jdk/pull/3681

Reply via email to