On Tue, 10 Jun 2025 08:17:29 GMT, Varada M wrote:
>> Varada M has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8358159: Empty mode/padding in cipher transformations
>
> Thanks all
@varada1110 After checking various javadoc, security docu
On Sat, 7 Jun 2025 20:07:45 GMT, Varada M wrote:
>> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
>> throws NoSuchAlgorithmException.
>> This patch restores the behavior by ensuring that empty mode or padding
>> strings are interpreted as null.
>>
>> Testing done f
On Sat, 7 Jun 2025 20:07:45 GMT, Varada M wrote:
>> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
>> throws NoSuchAlgorithmException.
>> This patch restores the behavior by ensuring that empty mode or padding
>> strings are interpreted as null.
>>
>> Testing done f
On Sat, 7 Jun 2025 20:07:45 GMT, Varada M wrote:
>> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
>> throws NoSuchAlgorithmException.
>> This patch restores the behavior by ensuring that empty mode or padding
>> strings are interpreted as null.
>>
>> Testing done f
On Thu, 5 Jun 2025 21:20:58 GMT, Valerie Peng wrote:
>> Varada M has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8358159: Empty mode/padding in cipher transformations
>
> src/java.base/share/classes/javax/crypto/Cipher.java line 460:
>
> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
> throws NoSuchAlgorithmException.
> This patch restores the behavior by ensuring that empty mode or padding
> strings are interpreted as null.
>
> Testing done for : tier1 - fastdebug level (AIX)
>
> JBS: [JDK-8358159
On Thu, 5 Jun 2025 11:30:42 GMT, Varada M wrote:
>> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
>> throws NoSuchAlgorithmException.
>> This patch restores the behavior by ensuring that empty mode or padding
>> strings are interpreted as null.
>>
>> Testing done f
> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
> throws NoSuchAlgorithmException.
> This patch restores the behavior by ensuring that empty mode or padding
> strings are interpreted as null.
>
> Testing done for : tier1 - fastdebug level (AIX)
>
> JBS: [JDK-8358159
On Wed, 4 Jun 2025 18:13:59 GMT, Valerie Peng wrote:
>> src/java.base/share/classes/javax/crypto/Cipher.java line 393:
>>
>>> 391: this.suffix = suffix.toUpperCase(Locale.ENGLISH);
>>> 392: this.mode = ((mode == null) || mode.isEmpty()) ? null :
>>> mode;
>>> 393:
On Wed, 4 Jun 2025 18:06:33 GMT, Valerie Peng wrote:
>> Varada M has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> whitespace error fix
>
> src/java.base/share/classes/javax/crypto/Cipher.java line 393:
>
>> 391: this.suffix =
On Wed, 4 Jun 2025 10:37:28 GMT, Varada M wrote:
>> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
>> throws NoSuchAlgorithmException.
>> This patch restores the behavior by ensuring that empty mode or padding
>> strings are interpreted as null.
>>
>> Testing done f
On Wed, 4 Jun 2025 10:37:28 GMT, Varada M wrote:
>> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
>> throws NoSuchAlgorithmException.
>> This patch restores the behavior by ensuring that empty mode or padding
>> strings are interpreted as null.
>>
>> Testing done f
On Wed, 4 Jun 2025 13:39:50 GMT, Sean Mullan wrote:
> @valeriepeng Can you also review this? Thanks.
Yes, I have started looking at it. Was about to ask for a regression test, and
you beat me to it.
-
PR Comment: https://git.openjdk.org/jdk/pull/25547#issuecomment-2940866076
On Wed, 4 Jun 2025 10:37:28 GMT, Varada M wrote:
>> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
>> throws NoSuchAlgorithmException.
>> This patch restores the behavior by ensuring that empty mode or padding
>> strings are interpreted as null.
>>
>> Testing done f
On Wed, 4 Jun 2025 10:37:28 GMT, Varada M wrote:
>> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
>> throws NoSuchAlgorithmException.
>> This patch restores the behavior by ensuring that empty mode or padding
>> strings are interpreted as null.
>>
>> Testing done f
> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
> throws NoSuchAlgorithmException.
> This patch restores the behavior by ensuring that empty mode or padding
> strings are interpreted as null.
>
> Testing done for : tier1 - fastdebug level (AIX)
>
> JBS: [JDK-8358159
> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
> throws NoSuchAlgorithmException.
> This patch restores the behavior by ensuring that empty mode or padding
> strings are interpreted as null.
>
> Testing done for : tier1 - fastdebug level (AIX)
>
> JBS: [JDK-8358159
On Fri, 30 May 2025 12:40:51 GMT, Sean Mullan wrote:
>> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
>> throws NoSuchAlgorithmException.
>> This patch restores the behavior by ensuring that empty mode or padding
>> strings are interpreted as null.
>>
>> Testing do
On Fri, 30 May 2025 12:40:51 GMT, Sean Mullan wrote:
>> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
>> throws NoSuchAlgorithmException.
>> This patch restores the behavior by ensuring that empty mode or padding
>> strings are interpreted as null.
>>
>> Testing do
Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
throws NoSuchAlgorithmException.
This patch restores the behavior by ensuring that empty mode or padding strings
are interpreted as null.
Testing done for : tier1 - fastdebug level
JBS: [JDK-8358159](https://bugs.openjdk
On Fri, 30 May 2025 12:06:21 GMT, Varada M wrote:
> Omitting the mode/padding in a transformation string eg: "AES/ /NoPadding"
> throws NoSuchAlgorithmException.
> This patch restores the behavior by ensuring that empty mode or padding
> strings are interpreted as null.
>
> Testing done for :
21 matches
Mail list logo
