<html>
<head>
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
</head>
<body bgcolor="#FFFFFF" text="#000000">
On 15/10/2018 03:32, Weijun Wang wrote:<br>
<blockquote type="cite"
cite="mid:7BF58AB1-0200-431E-A7F5-0D0DAEDA6599@oracle.com">
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
Hi All
<div class=""><br class="">
</div>
<div class="">Please take a review on a change in DEF_JNI_OnLoad
of NativeUtil.c:</div>
<div class=""><br class="">
</div>
<div class="">
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; background-color:
rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class=""><b
class="">diff --git
a/src/java.security.jgss/share/native/libj2gss/NativeUtil.c
b/src/java.security.jgss/share/native/libj2gss/NativeUtil.c</b></span></div>
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; color: rgb(195, 55,
32); background-color: rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class=""><b
class="">---
a/src/java.security.jgss/share/native/libj2gss/NativeUtil.c</b></span></div>
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; color: rgb(52, 188,
38); background-color: rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class=""><b
class="">+++
b/src/java.security.jgss/share/native/libj2gss/NativeUtil.c</b></span></div>
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; color: rgb(213, 59,
211); background-color: rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class="">@@
-145,7 +145,7 @@</span></div>
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; background-color:
rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class="">
return JNI_ERR;</span></div>
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; background-color:
rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class="">
}</span></div>
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; background-color:
rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class="">
CLS_GSSNameElement = (*env)->NewGlobalRef(env, cls);</span></div>
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; color: rgb(195, 55,
32); background-color: rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class="">-
if (CLS_GSSException == NULL) {</span></div>
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; color: rgb(52, 188,
38); background-color: rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class="">+
if (CLS_GSSNameElement == NULL) {</span></div>
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; background-color:
rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class="">
return JNI_ERR;</span></div>
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; background-color:
rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class="">
}</span></div>
<div style="margin: 0px; font-stretch: normal; font-size: 11px;
line-height: normal; font-family: Menlo; background-color:
rgb(255, 255, 255);" class=""><span
style="font-variant-ligatures: no-common-ligatures" class="">
cls = (*env)->FindClass(env,
"sun/security/jgss/wrapper/GSSCredElement");</span></div>
</div>
<br>
</blockquote>
This looks okay to me.<br>
<br>
-Alan<br>
<br>
</body>
</html>