diff options
| author | Stefano Brivio <sbrivio@redhat.com> | 2026-01-22 14:25:24 +0100 |
|---|---|---|
| committer | Stefano Brivio <sbrivio@redhat.com> | 2026-01-27 12:40:02 +0100 |
| commit | 69ce8ee07df57548faaa6ba48f379dc553321a0a (patch) | |
| tree | ccda9104bb59460a0365cf7736d4a5890b1b2bf8 /test/tcp | |
| parent | 386b5f5472b89769c025f5d5056348532a823b93 (diff) | |
| download | passt-69ce8ee07df57548faaa6ba48f379dc553321a0a.tar passt-69ce8ee07df57548faaa6ba48f379dc553321a0a.tar.gz passt-69ce8ee07df57548faaa6ba48f379dc553321a0a.tar.bz2 passt-69ce8ee07df57548faaa6ba48f379dc553321a0a.tar.lz passt-69ce8ee07df57548faaa6ba48f379dc553321a0a.tar.xz passt-69ce8ee07df57548faaa6ba48f379dc553321a0a.tar.zst passt-69ce8ee07df57548faaa6ba48f379dc553321a0a.zip | |
tcp: Add error checking for flow_epoll_set() in tcp_flow_migrate_target()
Commit 90287c2a774e ("tcp: Register fds with epoll at flow creation")
added a call to flow_epoll_set() in tcp_flow_migrate_target() without
checking for the error code.
As Laurent explains, that's harmless, as tcp_flow_migrate_target_ext()
will clean up the flow anyway, and we'll just uselessly call
flow_hash_insert() before removing it again.
But Coverity Scan complains about the unchecked return code, so check
it, skip flow_hash_insert() in that case, and explain in comments why
we don't have to do anything else.
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Reviewed-by: Laurent Vivier <lvivier@redhat.com>
Diffstat (limited to 'test/tcp')
0 files changed, 0 insertions, 0 deletions