Code review request: 8000653: SPNEGO tests fail at context.getDelegCred().getRemainingInitLifetime(mechOid)

[Valerie (Yu-Ching) Peng]

Looks fine to me.
Thanks,
Valerie

On 03/08/13 01:13, Wang Weijun wrote:
> Hi Valerie
>
> Please review this code change
>
>     http://cr.openjdk.java.net/~weijun/8000653/webrev.00/
>
> The reason is that if a SPNEGO GSSContext is established with krb5 underneath, its getDelegCred() only returns one krb5 cred element. I need to add more for mechs that are "compatible" with this mech. Currently it's only spnego so I hard code it.
>
> This change seems to have made the following lines in GSSContextImpl.initSecContext(stream,stream) obsolete, but I'll keep them unchanged for safety.
>
>      try {
>          credElement = myCred.getElement(mechOid, true);
>      } catch (GSSException ge) {
>          if (GSSUtil.isSpNegoMech(mechOid)&&
>              ge.getMajor() == GSSException.NO_CRED) {
>              credElement = myCred.getElement
>                  (myCred.getMechs()[0], true);
>          } else {
>              throw ge;
>          }
>      }
>
> All regression tests run fine.
>
> Thanks
> Max
>
>
>