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))

Weijun Wang weijun.wang at oracle.com
Mon Jun 24 03:04:16 UTC 2013


Oh, no

 if (!os.startsWith("Solaris") && !os.startsWith("Linux")) {

--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