<div dir="ltr">Have you considered zeroizing the whole "gss_channel_bindings_t" structure with memset? So we don't have problem if new fields are eventually added.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 21, 2018 at 1:39 PM, Jan Kalina <span dir="ltr"><<a href="mailto:jkalina@redhat.com" target="_blank">jkalina@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I has prepared fix for bug related to using native GSS API.<br><div>Uninitialized fields causes JVM crash or authentication failing.<br></div><div><br>Bug consequences are more described in bugreport:<br><a href="https://bugs.openjdk.java.net/browse/JDK-8194630" target="_blank">https://bugs.openjdk.java.net/<wbr>browse/JDK-8194630</a><br></div><div>Reproducer is attached to bugreport too.<br></div><div></div><div><br>Would anyone be interested in reviewing/sponsoring this change?<br></div><div>It would be really great to get this into JDK 9 and above.<br></div><div>(I am covered by Red Hat OCA.)<br></div><div><br></div><div>Thanks for your response!<br></div><div><br></div><div>PATCH:<br>------------------------------<wbr>----------------<br>diff --git a/src/java.security.jgss/<wbr>share/native/libj2gss/<wbr>GSSLibStub.c b/src/java.security.jgss/<wbr>share/native/libj2gss/<wbr>GSSLibStub.c<br>--- a/src/java.security.jgss/<wbr>share/native/libj2gss/<wbr>GSSLibStub.c<br>+++ b/src/java.security.jgss/<wbr>share/native/libj2gss/<wbr>GSSLibStub.c<br>@@ -169,6 +169,11 @@<br>   // initialize addrtype in CB first<br>   cb->initiator_addrtype = GSS_C_AF_NULLADDR;<br>   cb->acceptor_addrtype = GSS_C_AF_NULLADDR;<br>+  // addresses needs to be initialized to empty<br>+  cb->initiator_address.length = 0;<br>+  cb->initiator_address.value = NULL;<br>+  cb->acceptor_address.length = 0;<br>+  cb->acceptor_address.value = NULL;<br> <br>   /* set up initiator address */<br>   jinetAddr = (*env)->CallObjectMethod(env, jcb,<br>------------------------------<wbr>----------------<br><br></div></div>
</blockquote></div><br></div>