On Fri, 22 Apr 2022 06:26:01 GMT, Xue-Lei Andrew Fan <xue...@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 <xue...@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
