Skip to content

Commit 32b00fd

Browse files
Address code review feedback for PR 9391
1 parent 3af60ff commit 32b00fd

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

.github/workflows/os-check.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,7 @@ jobs:
6363
'--enable-coding=no',
6464
'--enable-dtls --enable-dtls13 --enable-ocspstapling --enable-ocspstapling2
6565
--enable-cert-setup-cb --enable-sessioncerts',
66+
'--disable-sni --disable-ecc --disable-tls13 --disable-secure-renegotiation-info',
6667
]
6768
name: make check
6869
if: github.repository_owner == 'wolfssl'

src/internal.c

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -38310,7 +38310,7 @@ static int DoSessionTicket(WOLFSSL* ssl, const byte* input, word32* inOutIdx,
3831038310
if ((ret = TLSX_PopulateExtensions(ssl, 1)) != 0)
3831138311
goto out;
3831238312
#else
38313-
word64 extensions_seen = 0u;
38313+
word32 extensions_seen = 0U;
3831438314
#endif
3831538315

3831638316
if ((i - begin) + OPAQUE16_LEN > helloSz) {
@@ -38364,12 +38364,11 @@ static int DoSessionTicket(WOLFSSL* ssl, const byte* input, word32* inOutIdx,
3836438364
ato16(&input[i], &extSz);
3836538365
i += OPAQUE16_LEN;
3836638366

38367-
if (extId < (sizeof(extensions_seen) * 8u))
38368-
{
38369-
word64 mask = 1u << extId;
38370-
if ((extensions_seen & mask) != 0u)
38371-
{
38372-
WOLFSSL_MSG("DoClientHello: duplicate extension found");
38367+
if (extId < (sizeof(extensions_seen) * 8U)) {
38368+
word32 mask = 1u << extId;
38369+
if ((extensions_seen & mask) != 0U) {
38370+
WOLFSSL_MSG(
38371+
"DoClientHello: duplicate extension found");
3837338372
ret = DUPLICATE_TLS_EXT_E;
3837438373
goto out;
3837538374
}

0 commit comments

Comments
 (0)