|
The behavior is OK. I think its only error message diff needs to be sed out.
It would be great if someone who has access to the solaris machine/env where the diff occurred can test out by modifying the aes_sed.properties as Myrna suggested. I too think this behavior is OK. The message text returned with the exception basically gives the
same explaination for why the operation failed. There is no reason for why we can expect message strings like this be the same between platforms. I will test out the sed filter proposed by Myrna on a few Solaris 10 machines. I have tried to modify the aes_sed.properties file as suggested by Myrna. I had to trim down the size of the pattern/substitute parts to get rid of all the commas in the message strings. Unfortunately, I still have a problem with one comma. The original error message seen on most platforms contains the following sub-string:
128, 192 or 256 while the new message string seen on Solaris 10 contains an extra comma: 128, 192, or 256 Since neither the pattern nor the substitute part of the sed script are allowed to contain a comma, I am unable to get rid of the comma after 192. Any suggestions on how I can use the aes_sed.properties script to get rid of this or alternative ways to make the output from this test match the master file? This patch fixes a problem in the AES test when running it on Solaris 10. The problem occurred due to a exception message text has been changed on Solaris 10. The problem is fixed by extending the substitute section of the aes_sed.properties file.
derbyall has been run on Solaris10 (sparc and x86) to verify that the problem is gone and on Solaris9 (sparc and x86) and Linux to verify that the change does not influence on JVM producing the original error message text. Although some tests failed none of these seems related to this fix. The AES encryption test runs without errors on Solaris 10.
Committed revision 406114 on 10.1 branch
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
If someone from Sun can pull some strings & find out it would help.
If the behavior is OK, then we are seeing a false diff, and it could be worked around by modifying the store/aes_sed.properties file.
Something like:
currently:
substitute=ERROR XJ001: Java exception: 'Keysize must;ERROR XJ001: Java exception: 'Wrong keysize: must
change it to:
substitute=ERROR XJ001: Java exception: 'Keysize must;ERROR XJ001: Java exception: 'Wrong keysize: must,ERROR XJ001: Java exception: 'AES key length must be 128, 192, or 256 bits: java.security.InvalidParameterException,ERROR XJ001: Java exception: 'Wrong keysize: must be equal to 128, 192 or 256: java.security.InvalidParameterException