From 9e5df350d63b0819f04b44bb57ea146274a6b42f Mon Sep 17 00:00:00 2001 From: David Gibson Date: Mon, 21 Oct 2024 18:40:29 +1100 Subject: tcp: Use structures to construct initial TCP options As a rule, we prefer constructing packets with matching C structures, rather than building them byte by byte. However, one case we still build byte by byte is the TCP options we include in SYN packets (in fact the only time we generate TCP options on the tap interface). Rework this to use a structure and initialisers which make it a bit clearer what's going on. Signed-off-by: David Gibson Signed-off-by; Stefano Brivio --- tcp_buf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tcp_buf.c') diff --git a/tcp_buf.c b/tcp_buf.c index 238827b..44df0e4 100644 --- a/tcp_buf.c +++ b/tcp_buf.c @@ -282,7 +282,7 @@ int tcp_buf_send_flag(const struct ctx *c, struct tcp_tap_conn *conn, int flags) seq = conn->seq_to_tap; ret = tcp_prepare_flags(c, conn, flags, &payload->th, - payload->opts, &optlen); + &payload->opts, &optlen); if (ret <= 0) { if (CONN_V4(conn)) tcp4_flags_used--; -- cgit v1.2.3