Re: RFR 8167647: Copy-and-paste bug in javax.security.auth.kerberos.KerberosTicket.toString()

2016-10-18 Thread Xuelei Fan 
Looks fine to me.

Xuelei

> On 19 Oct 2016, at 9:26 AM, Wang Weijun  wrote:
> 
> Please take a review on this change:
> 
> diff --git 
> a/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosTicket.java
>  
> b/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosTicket.java
> --- 
> a/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosTicket.java
> +++ 
> b/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosTicket.java
> @@ -694,7 +694,7 @@
> "Proxy Ticket " + flags[PROXY_TICKET_FLAG] + "\n" +
> "Postdated Ticket " + flags[POSTDATED_TICKET_FLAG] + "\n" +
> "Renewable Ticket " + flags[RENEWABLE_TICKET_FLAG] + "\n" +
> -"Initial Ticket " + flags[RENEWABLE_TICKET_FLAG] + "\n" +
> +"Initial Ticket " + flags[INITIAL_TICKET_FLAG] + "\n" +
> "Auth Time = " + String.valueOf(authTime) + "\n" +
> "Start Time = " + String.valueOf(startTime) + "\n" +
> "End Time = " + endTime.toString() + "\n" +
> 
> Noreg-trivial.
> 
> Thanks
> Max
>

RFR 8167647: Copy-and-paste bug in javax.security.auth.kerberos.KerberosTicket.toString()

2016-10-18 Thread Wang Weijun 

Please take a review on this change:

diff --git 
a/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosTicket.java 
b/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosTicket.java
--- 
a/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosTicket.java
+++ 
b/src/java.security.jgss/share/classes/javax/security/auth/kerberos/KerberosTicket.java

@@ -694,7 +694,7 @@
 "Proxy Ticket " + flags[PROXY_TICKET_FLAG] + "\n" +
 "Postdated Ticket " + flags[POSTDATED_TICKET_FLAG] + 
"\n" +
 "Renewable Ticket " + flags[RENEWABLE_TICKET_FLAG] + 
"\n" +

-"Initial Ticket " + flags[RENEWABLE_TICKET_FLAG] + "\n" +
+"Initial Ticket " + flags[INITIAL_TICKET_FLAG] + "\n" +
 "Auth Time = " + String.valueOf(authTime) + "\n" +
 "Start Time = " + String.valueOf(startTime) + "\n" +
 "End Time = " + endTime.toString() + "\n" +

Noreg-trivial.

Thanks
Max