This is an automated email from the ASF dual-hosted git repository.
bneradt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/master by this push:
new 5d878a4be2 Guard quiche-only QUIC connection state (#13478)
5d878a4be2 is described below
commit 5d878a4be2d6d8a22d11977e3811f5a4db948945
Author: Brian Neradt <[email protected]>
AuthorDate: Mon Aug 3 14:08:56 2026 -0500
Guard quiche-only QUIC connection state (#13478)
Native OpenSSL QUIC builds do not use the connection table or quiche
timeout event stored by QUICNetVConnection. Clang diagnoses those
quiche-only private fields as unused, causing warnings-as-errors builds
to fail.
This guards the members with TS_HAS_QUICHE so native OpenSSL builds do
not declare them while quiche builds retain the required state.
---
src/iocore/net/P_QUICNetVConnection.h | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/src/iocore/net/P_QUICNetVConnection.h
b/src/iocore/net/P_QUICNetVConnection.h
index bb8064ecb4..a43ed31c34 100644
--- a/src/iocore/net/P_QUICNetVConnection.h
+++ b/src/iocore/net/P_QUICNetVConnection.h
@@ -221,7 +221,9 @@ private:
QUICConnectionId _initial_source_connection_id; // src cid used for Initial
packet
QUICConnectionId _quic_connection_id; // src cid in local
+#if TS_HAS_QUICHE
QUICConnectionTable *_ctable = nullptr;
+#endif
#if TS_HAS_OPENSSL_QUIC
std::unordered_map<QUICStreamId, SSL *> _openssl_streams;
@@ -240,10 +242,12 @@ private:
void _close_packet_write_ready(Event *data);
Event *_packet_write_ready = nullptr;
- void _schedule_quiche_timeout();
- void _unschedule_quiche_timeout();
- void _close_quiche_timeout(Event *data);
+ void _schedule_quiche_timeout();
+ void _unschedule_quiche_timeout();
+ void _close_quiche_timeout(Event *data);
+#if TS_HAS_QUICHE
Event *_quiche_timeout = nullptr;
+#endif
void _schedule_closing_event();