Commit b10b3adac9 for openssl.org
commit b10b3adac96b50b1a70194d2df832105f52124a8
Author: Andrew Dinh <andrewd@openssl.org>
Date: Sat Sep 5 09:44:35 2026 -0400
Port script_64
Assisted-by: Claude:claude-sonnet-5
Reviewed-by: Neil Horman <nhorman@openssl.org>
Reviewed-by: Saša NedvÄ›dický <sashan@openssl.org>
Merge-date: Tue Sep 15 09:39:49 2026
Merged-from: https://github.com/openssl/openssl/pull/32734
diff --git a/test/quic_multistream_test.c b/test/quic_multistream_test.c
index 0fed55749e..2e6ae86f4d 100644
--- a/test/quic_multistream_test.c
+++ b/test/quic_multistream_test.c
@@ -2304,71 +2304,6 @@ static const struct script_op script_32[] = {
OP_END
};
-static int script_32_inject_plain(struct helper *h, QUIC_PKT_HDR *hdr,
- unsigned char *buf, size_t len)
-{
- int ok = 0;
- WPACKET wpkt;
- unsigned char frame_buf[64];
- size_t written;
- uint64_t type = OSSL_QUIC_FRAME_TYPE_STREAM_OFF_LEN, offset, flen, i;
-
- if (hdr->type != QUIC_PKT_TYPE_1RTT)
- return 1;
-
- switch (h->inject_word1) {
- default:
- return 0;
- case 0:
- return 1;
- case 1:
- offset = 0;
- flen = 0;
- break;
- case 2:
- offset = (((uint64_t)1) << 62) - 1;
- flen = 5;
- break;
- case 3:
- offset = 1 * 1024 * 1024 * 1024; /* 1G */
- flen = 5;
- break;
- case 4:
- offset = 0;
- flen = 1;
- break;
- }
-
- if (!TEST_true(WPACKET_init_static_len(&wpkt, frame_buf,
- sizeof(frame_buf), 0)))
- return 0;
-
- if (!TEST_true(WPACKET_quic_write_vlint(&wpkt, type))
- || !TEST_true(WPACKET_quic_write_vlint(&wpkt, /* stream ID */
- h->inject_word0 - 1))
- || !TEST_true(WPACKET_quic_write_vlint(&wpkt, offset))
- || !TEST_true(WPACKET_quic_write_vlint(&wpkt, flen)))
- goto err;
-
- for (i = 0; i < flen; ++i)
- if (!TEST_true(WPACKET_put_bytes_u8(&wpkt, 0x42)))
- goto err;
-
- if (!TEST_true(WPACKET_get_total_written(&wpkt, &written)))
- goto err;
-
- if (!qtest_fault_prepend_frame(h->qtf, frame_buf, written))
- goto err;
-
- ok = 1;
-err:
- if (ok)
- WPACKET_finish(&wpkt);
- else
- WPACKET_cleanup(&wpkt);
- return ok;
-}
-
/* 33. Fault injection - STREAM frame with illegal offset */
static const struct script_op script_33[] = {
/* test moved to test/radix/quic_tests.c */
@@ -2557,21 +2492,7 @@ static const struct script_op script_63[] = {
/* 64. Fault injection - STREAM - zero-length no-FIN is accepted */
static const struct script_op script_64[] = {
- OP_S_SET_INJECT_PLAIN(script_32_inject_plain),
- OP_C_SET_ALPN("ossltest"),
- OP_C_CONNECT_WAIT(),
- OP_C_SET_DEFAULT_STREAM_MODE(SSL_DEFAULT_STREAM_MODE_NONE),
-
- OP_S_NEW_STREAM_UNI(a, S_UNI_ID(0)),
- OP_S_WRITE(a, "apple", 5),
-
- OP_C_ACCEPT_STREAM_WAIT(a),
- OP_C_READ_EXPECT(a, "apple", 5),
-
- OP_SET_INJECT_WORD(S_BIDI_ID(20) + 1, 1),
- OP_S_WRITE(a, "orange", 6),
- OP_C_READ_EXPECT(a, "orange", 6),
-
+ /* test moved to test/radix/quic_tests.c */
OP_END
};
diff --git a/test/radix/quic_tests.c b/test/radix/quic_tests.c
index 023676d7bd..b6eaa5273e 100644
--- a/test/radix/quic_tests.c
+++ b/test/radix/quic_tests.c
@@ -3685,8 +3685,25 @@ DEF_SCRIPT(script_63, "Fault injection - STREAM frame exceeding stream limit")
OP_EXPECT_CONN_CLOSE_INFO(C, OSSL_QUIC_ERR_STREAM_LIMIT_ERROR, 0, 0);
}
-DEF_SCRIPT(script_64, "place holder for multistrem script_64")
+/* 64. Fault injection - STREAM - zero-length no-FIN is accepted */
+DEF_SCRIPT(script_64, "Fault injection - STREAM - zero-length no-FIN is accepted")
{
+ OP_SIMPLE_PAIR_CONN_ND();
+ OP_ACCEPT_CONN_WAIT_ND(L, S, 0);
+
+ OP_SET_INJECT_PLAIN(S, inject_stream_data_frame_plain);
+
+ OP_NEW_STREAM(S, Sa, SSL_STREAM_FLAG_UNI);
+ OP_WRITE(Sa, "apple", 5);
+
+ OP_ACCEPT_STREAM_WAIT(C, Ca, 0);
+ OP_READ_EXPECT(Ca, "apple", 5);
+
+ OP_ENGINE_TICK_DISABLE(S);
+ OP_SET_INJECT_WORD(S_BIDI_ID(20) + 1, 1);
+ OP_WRITE(Sa, "orange", 6);
+ OP_ENGINE_TICK_ENABLE(S);
+ OP_READ_EXPECT(Ca, "orange", 6);
}
DEF_SCRIPT(script_65, "place holder for multistrem script_65")