diff options
author | Stefano Brivio <sbrivio@redhat.com> | 2022-11-19 09:29:54 +0100 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2022-11-25 01:37:36 +0100 |
commit | b06014a6b2f35bb3ca58e94a6ef84e3644e49fb0 (patch) | |
tree | 658deedb11e2f17417339ec16ad4f577d2338286 /tcp_conn.h | |
parent | 8e914238b6def7abc61d97482dd618cfc3a061bd (diff) | |
download | passt-b06014a6b2f35bb3ca58e94a6ef84e3644e49fb0.tar passt-b06014a6b2f35bb3ca58e94a6ef84e3644e49fb0.tar.gz passt-b06014a6b2f35bb3ca58e94a6ef84e3644e49fb0.tar.bz2 passt-b06014a6b2f35bb3ca58e94a6ef84e3644e49fb0.tar.lz passt-b06014a6b2f35bb3ca58e94a6ef84e3644e49fb0.tar.xz passt-b06014a6b2f35bb3ca58e94a6ef84e3644e49fb0.tar.zst passt-b06014a6b2f35bb3ca58e94a6ef84e3644e49fb0.zip |
tcp: Pass union tcp_conn pointer to destroy and splice timer functions
The pointers are actually the same, but we later pass the container
union to tcp_table_compact(), which might zero the size of the whole
union, and this confuses Coverity Scan.
Given that we have pointers to the container union to start with,
just pass those instead, all the way down to tcp_table_compact().
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'tcp_conn.h')
-rw-r--r-- | tcp_conn.h | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -184,8 +184,8 @@ extern union tcp_conn tc[]; void tcp_splice_conn_update(struct ctx *c, struct tcp_splice_conn *new); void tcp_table_compact(struct ctx *c, union tcp_conn *hole); -void tcp_splice_destroy(struct ctx *c, struct tcp_splice_conn *conn); -void tcp_splice_timer(struct ctx *c, struct tcp_splice_conn *conn); +void tcp_splice_destroy(struct ctx *c, union tcp_conn *conn_union); +void tcp_splice_timer(struct ctx *c, union tcp_conn *conn_union); void tcp_splice_pipe_refill(const struct ctx *c); |