Code review request: 8017453: ReplayCache tests fail on multiple platforms (was Re: Please be prepared for possible test failures (was Fwd: [JBS] (JDK-8001326) Improve Kerberos replay caching))

Xuelei Fan xuelei.fan at oracle.com
Mon Jun 24 03:28:03 UTC 2013


On 6/24/2013 11:04 AM, Weijun Wang wrote:
> Oh, no
> 
>  if (!os.startsWith("Solaris") && !os.startsWith("Linux")) {
> 
Good.

Xuelei

> --Max
> 
> On 6/24/13 11:01 AM, Weijun Wang wrote:
>> How about
>>
>>              if (!os.startsWith("Solaris") || !os.startsWith("Linux")) {
>>                  mode = -1;
>>              }
>>
>> Thanks
>> Max
>>
>>
>> On 6/24/13 10:35 AM, Xuelei Fan wrote:
>>> On 6/24/2013 10:01 AM, Weijun Wang wrote:
>>>>
>>>>
>>>> On 6/24/13 9:49 AM, Xuelei Fan wrote:
>>>>> ReplayCacheTestProc.java
>>>>> ========================
>>>>> 75       String os = System.getProperty("os.name", "???");
>>>>>
>>>>> What happens if "os" is "???"?
>>>>
>>>> Then it's neither Windows nor Mac OS X. :)
>>>>
>>> The code means except Windows and Mac OS X, all other platform supports
>>> native mode, even the OS name is unknown.
>>>
>>>> I gave a default value so that the result will not be null. Although the
>>>> system property "os.name" should always have a value, it makes me
>>>> comfort not thinking of possible NPE.
>>>>
>>> ;-) I guess you may also worry that if the OS does not support native
>>> mode other than Windows and Mac OS X, the test will fail again.
>>>
>>> As you are here already, I would suggest to make more improvement. When
>>> the OS is unknown, java mode should be used .
>>>
>>> Xuelei
>>>
>>>> Thanks
>>>> Max
>>>>
>>>>>
>>>>> Xuelei
>>>>>
>>>>> On 6/24/2013 12:36 AM, Wang Weijun wrote:
>>>>>> Send again. And BTW, JPRT runs fine.
>>>>>>
>>>>>> 在 Jun 23, 2013,6:29 PM,Weijun Wang <weijun.wang at oracle.com> 写道:
>>>>>>
>>>>>>> The macosx problem found, the machine's native GSS does not support shared replaycache.
>>>>>>>
>>>>>>> *Valerie* and/or *Xuelei*, can you please review the fix?
>>>>>>>
>>>>>>>      http://cr.openjdk.java.net/~weijun/8017453/webrev.00/
>>>>>>>
>>>>>>>




More information about the security-dev mailing list