Integrated: 8285431: Assertion in NativeGSSContext constructor

Xue-Lei Andrew Fan xuelei at openjdk.java.net
Sat Apr 23 14:32:25 UTC 2022


On Fri, 22 Apr 2022 06:26:01 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:

> Hi,
> 
> May I have the simple update reviewed.
> 
> In the NativeGSSContext constructor for imported context, the assert is use on the object field, instead of the input parameters. As in a constructor, `'this'` object does not exist yet, this looks like an obvious issue.  The fix is straightforward as well. 
> 
> 
>     NativeGSSContext(long pCtxt, GSSLibStub stub) throws GSSException {
> -       assert(pContext != 0);
> +       assert(pCtxt != 0);
>         pContext = pCtxt;
>         ...

This pull request has now been integrated.

Changeset: e9d604a3
Author:    Xue-Lei Andrew Fan <xuelei at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/e9d604a3e5dcd26eed21eda4b5251a07dafcb92a
Stats:     2 lines in 1 file changed: 0 ins; 0 del; 2 mod

8285431: Assertion in NativeGSSContext constructor

Reviewed-by: djelinski, valeriep

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

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



More information about the security-dev mailing list