<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>