code review request: 6330275: Rework the PaddingTest regression test. (was Re: Fwd: jdk_security2 tests)

Bradford Wetmore bradford.wetmore at
Thu Jul 7 22:05:49 PDT 2011

Hi Max,

 > Ping

Pong.  :)

My only comment is in the diff section.  I realize it's a 
FileInputStream and all the bytes should be obtained on a read, but I 
think you might want to consider the case of a short read for whatever 
reason.  FileInputStream doesn't guarantee that all bytes will be 
returned, just "some".  It might be better to read both files until EOF, 
then compare sizes/contents.

Otherwise, looks fine.


On 6/22/2011 5:48 PM, Weijun Wang wrote:
> Thanks
> Max
> On 06/23/2011 08:03 AM, Brad Wetmore wrote:
>> No, feel free to take it.
>> Brad
>> On 6/21/2011 2:24 AM, Weijun Wang wrote:
>>> Hi Brad
>>>> # Timed out, Solaris 10 64bit sparcv9
>>>> com/sun/crypto/provider/Cipher/DES/ generic-all
>>>> This test has not generated random numbers (all key material in byte[]
>>>> constants) so there is no entropy pool issue. One special thing is that
>>>> it calls the external diff command on two files but only prints out the
>>>> result with no exception thrown when the files are different (6330275).
>>> I see you're RE for . Are you working on it now?
>>> Thanks
>>> Max

More information about the security-dev mailing list