diff options
author | Marcus Wichelmann <marcus.wichelmann@hetzner-cloud.de> | 2025-03-06 00:34:38 +0300 |
---|---|---|
committer | Martin KaFai Lau <martin.lau@kernel.org> | 2025-03-06 23:31:08 +0300 |
commit | 49306d5bfc6a86e3a75f8d584cda65164fb3ed71 (patch) | |
tree | 73721046c859f7d8df3aa4c6b02ff5e4034d3d88 /tools/perf/scripts/python/exported-sql-viewer.py | |
parent | 73eeecc3cdfeb098f8859269b608589b8a919686 (diff) | |
download | linux-49306d5bfc6a86e3a75f8d584cda65164fb3ed71.tar.xz |
selftests/bpf: Fix file descriptor assertion in open_tuntap helper
The open_tuntap helper function uses open() to get a file descriptor for
/dev/net/tun.
The open(2) manpage writes this about its return value:
On success, open(), openat(), and creat() return the new file
descriptor (a nonnegative integer). On error, -1 is returned and
errno is set to indicate the error.
This means that the fd > 0 assertion in the open_tuntap helper is
incorrect and should rather check for fd >= 0.
When running the BPF selftests locally, this incorrect assertion was not
an issue, but the BPF kernel-patches CI failed because of this:
open_tuntap:FAIL:open(/dev/net/tun) unexpected open(/dev/net/tun):
actual 0 <= expected 0
Signed-off-by: Marcus Wichelmann <marcus.wichelmann@hetzner-cloud.de>
Signed-off-by: Martin KaFai Lau <martin.lau@kernel.org>
Reviewed-by: Willem de Bruijn <willemb@google.com>
Link: https://patch.msgid.link/20250305213438.3863922-7-marcus.wichelmann@hetzner-cloud.de
Diffstat (limited to 'tools/perf/scripts/python/exported-sql-viewer.py')
0 files changed, 0 insertions, 0 deletions