On Tue, 2 Nov 2021 23:43:42 GMT, Xue-Lei Andrew Fan <xue...@openjdk.org> wrote:

>> Daniel Jeliński has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   avoid modifying DTLSOutputRecord
>
> src/java.base/share/classes/sun/security/ssl/SSLEngineOutputRecord.java line 
> 436:
> 
>> 434: 
>> 435:         void queueUpCipherDispose() {
>> 436:             RecordMemo lastMemo = handshakeMemos.getLast();
> 
> Sorry, I missed that the getLast could throw exception if it is empty.  I may 
> check it before the call to getLast.
> 
> +           if (handshakeMemos.isEmpty()) {
> +              return;
> +          }

my mistake. Replaced with `peekLast`, should be better now.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6084

Reply via email to