RFR 8039853: Provider.Service.newInstance() does not work with current JDK JGSS Mechanisms
Sean Mullan
sean.mullan at oracle.com
Tue Apr 15 15:01:01 UTC 2014
Looks fine to me.
--Sean
On 04/15/2014 04:03 AM, Wang Weijun wrote:
> Please review the code changes at
>
> http://cr.openjdk.java.net/~weijun/8039853/webrev.00/
>
> If you find it confused, I have mistakenly pushed some code changes in
>
> http://hg.openjdk.java.net/jdk9/dev/jdk/rev/ba6e2fcdfa15
>
> and the current code change is trying to fix/enhance it. Altogether, the actual change I want to make is:
>
> diff --git a/src/share/classes/sun/security/jgss/krb5/Krb5MechFactory.java b/src/share/classes/sun/security/jgss/krb5/Krb5MechFactory.java
> --- a/src/share/classes/sun/security/jgss/krb5/Krb5MechFactory.java
> +++ b/src/share/classes/sun/security/jgss/krb5/Krb5MechFactory.java
> @@ -86,6 +86,10 @@
> return result;
> }
>
> + public Krb5MechFactory() {
> + this(GSSCaller.CALLER_UNKNOWN);
> + }
> +
> public Krb5MechFactory(GSSCaller caller) {
> this.caller = caller;
> }
> diff --git a/src/share/classes/sun/security/jgss/spnego/SpNegoMechFactory.java b/src/share/classes/sun/security/jgss/spnego/SpNegoMechFactory.java
> --- a/src/share/classes/sun/security/jgss/spnego/SpNegoMechFactory.java
> +++ b/src/share/classes/sun/security/jgss/spnego/SpNegoMechFactory.java
> @@ -96,6 +96,10 @@
> return result;
> }
>
> + public SpNegoMechFactory() {
> + this(GSSCaller.CALLER_UNKNOWN);
> + }
> +
> public SpNegoMechFactory(GSSCaller caller) {
> manager = new GSSManagerImpl(caller, false);
> Oid[] mechs = manager.getMechs();
>
> Please note that the previous change made in src/share/classes/java/security/Provider.java is now backed out.
>
> Thanks
> Max
>
More information about the security-dev
mailing list