Skip to content

Commit 06e753e

Browse files
committed
Fix TP value for draft 13.
1 parent b2c1dcf commit 06e753e

File tree

4 files changed

+4
-5
lines changed

4 files changed

+4
-5
lines changed

picohttp/quicperf.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1244,7 +1244,7 @@ size_t quicperf_prepare_time_stamp(quicperf_stream_ctx_t* stream_ctx, uint8_t *
12441244
{
12451245
size_t byte_index = 0;
12461246
if (stream_ctx->frame_bytes_sent < 8) {
1247-
uint8_t time_stamp[8];
1247+
uint8_t time_stamp[8] = { 0, 0, 0, 0, 0, 0, 0, 0 };
12481248
if (picoquic_frames_uint64_encode(buffer, buffer + 8, stream_ctx->frame_start_stamp) != NULL) {
12491249
while (stream_ctx->frame_bytes_sent < 8 && byte_index < available) {
12501250
buffer[byte_index] = time_stamp[stream_ctx->frame_bytes_sent];

picoquic/picoquic_internal.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -588,7 +588,7 @@ typedef uint64_t picoquic_tp_enum;
588588
#define picoquic_tp_grease_quic_bit 0x2ab2
589589
#define picoquic_tp_version_negotiation 0x11
590590
#define picoquic_tp_enable_bdp_frame 0xebd9 /* per draft-kuhn-quic-0rtt-bdp-09 */
591-
#define picoquic_tp_initial_max_path_id 0x0f739bbc1b666d0cull /* per draft quic multipath 12 */
591+
#define picoquic_tp_initial_max_path_id 0x0f739bbc1b666d0dull /* per draft quic multipath 13 */
592592
#define picoquic_tp_address_discovery 0x9f81a176 /* per draft-seemann-quic-address-discovery */
593593

594594
/* Callback for converting binary log to quic log at the end of a connection.

picoquictest/log_tp_test_ref.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@
5656
0102030405060708: Extension type: 1 (idle_timeout), length 1, 1e
5757
0102030405060708: Extension type: 3 (max_packet_size), length 2, 45c8
5858
0102030405060708: Extension type: 15 (hcid), length 8, 0203040506070809
59-
0102030405060708: Extension type: 1113404765106498828 (initial_max_path_id), length 1, 04
59+
0102030405060708: Extension type: 1113404765106498829 (initial_max_path_id), length 1, 04
6060

6161
0102030405060708: Extension list (36 bytes):
6262
0102030405060708: Extension type: 1 (idle_timeout), length 2, 400a

picoquictest/tls_api_test.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8254,7 +8254,7 @@ int padding_test_one(uint32_t padding_multiple, uint32_t padding_min_size)
82548254

82558255
while (ret == 0 && nb_trials < 256 && nb_inactive < 256 && TEST_CLIENT_READY && TEST_SERVER_READY) {
82568256
int was_active = 0;
8257-
int ack_is_queued = 0;
8257+
82588258
nb_trials++;
82598259
if ((picoquic_is_cnx_backlog_empty(test_ctx->cnx_client) && picoquic_is_cnx_backlog_empty(test_ctx->cnx_server) &&
82608260
test_ctx->c_to_s_link->first_packet == NULL && test_ctx->s_to_c_link->first_packet == NULL)) {
@@ -8271,7 +8271,6 @@ int padding_test_one(uint32_t padding_multiple, uint32_t padding_min_size)
82718271

82728272
if (ret == 0 && is_queued && test_ctx->c_to_s_link->first_packet != NULL) {
82738273
size_t length = test_ctx->c_to_s_link->first_packet->length;
8274-
uint8_t* bytes = test_ctx->c_to_s_link->first_packet->bytes;
82758274
size_t pn_offset = 1 + test_ctx->cnx_client->path[0]->p_remote_cnxid->cnx_id.id_len;
82768275
size_t pn_length = padding_test_predict_pn_length(&test_ctx->cnx_client->pkt_ctx[picoquic_packet_context_application]);
82778276
size_t raw_length = pn_offset + pn_length + test_sizes[i];

0 commit comments

Comments
 (0)