Update tests/api/test_tls_ext.c

embhorn · Copilot · web-flow · commit 06a36d8971ce · 2026-02-24T11:19:40.000-06:00
Explain reason for cast in test comments

Co-authored-by: Copilot &lt;175728472+Copilot@users.noreply.github.com&gt;
diff --git a/tests/api/test_tls_ext.c b/tests/api/test_tls_ext.c
@@ -592,6 +592,8 @@ static word16 test_SNI_GetSize_old(SNI* list)
         length += ENUM_LEN + OPAQUE16_LEN;
         switch (sni->type) {
             case WOLFSSL_SNI_HOST_NAME:
+                /* Intentional truncation to word16 to mirror the original
+                 * vulnerable implementation for testing overflow behavior. */
                 length += (word16)XSTRLEN((char*)sni->data.host_name);
                 break;
         }

Original file line number	Diff line number	Diff line change
`@@ -592,6 +592,8 @@ static word16 test_SNI_GetSize_old(SNI* list)`
`592`	`592`	`length += ENUM_LEN + OPAQUE16_LEN;`
`593`	`593`	`switch (sni->type) {`
`594`	`594`	`case WOLFSSL_SNI_HOST_NAME:`
	`595`	`+ /* Intentional truncation to word16 to mirror the original`
	`596`	`+ * vulnerable implementation for testing overflow behavior. */`
`595`	`597`	`length += (word16)XSTRLEN((char*)sni->data.host_name);`
`596`	`598`	`break;`
`597`	`599`	`}`