<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <p>I'll have a look Martin, but it'll be better if we can get an NSS
      or PKCS11 expert to take a look. Any takers ? Can you expand some
      bit on the exact reason for your 8195607 changes ? Pointers to NSS
      changes etc. ?<br>
    </p>
    <pre class="moz-signature" cols="72">Regards,
Sean.</pre>
    <div class="moz-cite-prefix">On 14/03/18 16:11, Martin Balao wrote:<br>
    </div>
    <blockquote
cite="mid:CAKZz+gcu6WO0RakmzR7_mURcf1_=2=BfZV17BBuSAK5QvowD_A@mail.gmail.com"
      type="cite">
      <div dir="ltr">Hi Sean,
        <div><br>
        </div>
        <div>Thanks!</div>
        <div><br>
        </div>
        <div>Can you please review the fix [0] so we have it in? As far
          as I know, fix makes Oracle internal tests pass.</div>
        <div><br>
        </div>
        <div>Kind regards,</div>
        <div>Martin.-</div>
        <div><br>
        </div>
        <div>--</div>
        <div>[0] - <a moz-do-not-send="true"
href="http://mail.openjdk.java.net/pipermail/security-dev/2018-February/016776.html"
            target="_blank" style="font-size:12.8px">http://mail.openjdk.java.<wbr>net/pipermail/security-dev/<wbr>2018-February/016776.html</a><span
            style="font-size:12.8px"> </span></div>
      </div>
      <div class="gmail_extra"><br>
        <div class="gmail_quote">On Wed, Mar 14, 2018 at 12:05 PM, Seán
          Coffey <span dir="ltr"><<a moz-do-not-send="true"
              href="mailto:sean.coffey@oracle.com" target="_blank">sean.coffey@oracle.com</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0
            .8ex;border-left:1px #ccc solid;padding-left:1ex">
            <div bgcolor="#FFFFFF" text="#000000">
              <p>Hi Martin, <br>
              </p>
              <p>Thanks for the 8195607 pointer. I'll get this ported to
                jdk8u also. I didn't see that actual issue during
                testing but no harm to port it. Will submit a new webrev
                shortly.<br>
              </p>
              <pre class="m_9220356643261423839moz-signature" cols="72">Regards,
Sean.</pre>
              <div>
                <div class="h5">
                  <div class="m_9220356643261423839moz-cite-prefix">On
                    14/03/18 14:55, Martin Balao wrote:<br>
                  </div>
                  <blockquote type="cite">
                    <div dir="ltr">Hi Sean,
                      <div><br>
                      </div>
                      <div>Is this related to <a moz-do-not-send="true"
href="http://mail.openjdk.java.net/pipermail/security-dev/2018-February/016776.html"
                          target="_blank">http://mail.openjdk.java.<wbr>net/pipermail/security-dev/<wbr>2018-February/016776.html</a> ?</div>
                      <div><br>
                      </div>
                      <div>Kind regards,</div>
                      <div>Martin.-</div>
                    </div>
                    <div class="gmail_extra"><br>
                      <div class="gmail_quote">On Wed, Mar 14, 2018 at
                        11:48 AM, Seán Coffey <span dir="ltr"><<a
                            moz-do-not-send="true"
                            href="mailto:sean.coffey@oracle.com"
                            target="_blank">sean.coffey@oracle.com</a>></span>
                        wrote:<br>
                        <blockquote class="gmail_quote" style="margin:0
                          0 0 .8ex;border-left:1px #ccc
                          solid;padding-left:1ex">Looking to backport
                          this fix to jdk8u-dev.  Contributed to JDK
                          Project by Martin Balao.<br>
                          <br>
                          <a moz-do-not-send="true"
                            href="https://bugs.openjdk.java.net/browse/JDK-8165996"
                            rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/<wbr>browse/JDK-8165996</a><br>
                          webrev : <a moz-do-not-send="true"
                            href="http://cr.openjdk.java.net/%7Ecoffeys/webrev.8165996.8u/webrev/"
                            rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~co<wbr>ffeys/webrev.8165996.8u/webrev<wbr>/</a><br>
                          <br>
                          The test/jdk/sun/security/pkcs11/P<wbr>KCS11Test.java
                          edits didn't seem applicable to jdk8u.<br>
                          <br>
                          Also, I edited sun/security/pkcs11/Secmod/Tes<wbr>tNssDbSqlite.java
                          to exclude test where NSS Provider<br>
                          didn't register. (line 66). In particular, I
                          saw this while running 32bit JDK tests on 64
                          bit Linux.<span
                            class="m_9220356643261423839HOEnZb"><font
                              color="#888888"><br>
                              <br>
                              -- <br>
                              Regards,<br>
                              Sean.<br>
                              <br>
                            </font></span></blockquote>
                      </div>
                      <br>
                    </div>
                  </blockquote>
                  <br>
                </div>
              </div>
            </div>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <br>
  </body>
</html>