diff options
author | Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-07-27 08:46:09 -0300 |
---|---|---|
committer | Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-07-27 08:49:36 -0300 |
commit | 16dc7448ca0682c73ddd3150d93bcb62c60f7933 (patch) | |
tree | ca39c0d99a56e1b99aa00352c4e5166b9d15aa27 | |
parent | Merge branch 'master' of ssh://github.com/netblue30/firejail (diff) | |
download | firejail-16dc7448ca0682c73ddd3150d93bcb62c60f7933.tar.gz firejail-16dc7448ca0682c73ddd3150d93bcb62c60f7933.tar.zst firejail-16dc7448ca0682c73ddd3150d93bcb62c60f7933.zip |
fnetlock: fix unused bw variable warning
This is breaking build-clang on CI[1]:
make -C src/fnetlock/
make[1]: Entering directory '/home/runner/work/firejail/firejail/src/fnetlock'
clang-14 [...] -c main.c -o main.o
main.c:97:11: error: variable 'bw' set but not used [-Werror,-Wunused-but-set-variable]
unsigned bw = 0; // bandwidth calculations
^
1 error generated.
make[1]: *** [../../src/prog.mk:16: main.o] Error 1
make: *** [Makefile:58: src/fnetlock/fnetlock] Error 2
Added on commit 8e4b847cd ("split nettrace executable ^Cto netrace and
netlock", 2023-07-25).
[1]: https://github.com/netblue30/firejail/actions/runs/5669072674/job/15361026508
Cc: @netblue30
-rw-r--r-- | src/fnetlock/main.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/fnetlock/main.c b/src/fnetlock/main.c index b76be029d..d4169b7e1 100644 --- a/src/fnetlock/main.c +++ b/src/fnetlock/main.c | |||
@@ -94,7 +94,8 @@ static void run_trace(void) { | |||
94 | 94 | ||
95 | unsigned start = time(NULL); | 95 | unsigned start = time(NULL); |
96 | unsigned char buf[MAX_BUF_SIZE]; | 96 | unsigned char buf[MAX_BUF_SIZE]; |
97 | unsigned bw = 0; // bandwidth calculations | 97 | // FIXME: error: variable 'bw' set but not used [-Werror,-Wunused-but-set-variable] |
98 | //unsigned bw = 0; // bandwidth calculations | ||
98 | 99 | ||
99 | int printed = 0; | 100 | int printed = 0; |
100 | while (1) { | 101 | while (1) { |
@@ -149,7 +150,8 @@ static void run_trace(void) { | |||
149 | #endif | 150 | #endif |
150 | // filter out loopback traffic | 151 | // filter out loopback traffic |
151 | if (buf[12] != 127 && buf[16] != 127) { | 152 | if (buf[12] != 127 && buf[16] != 127) { |
152 | bw += bytes + 14; // assume a 14 byte Ethernet layer | 153 | // FIXME: error: variable 'bw' set but not used [-Werror,-Wunused-but-set-variable] |
154 | //bw += bytes + 14; // assume a 14 byte Ethernet layer | ||
153 | 155 | ||
154 | uint32_t ip_src; | 156 | uint32_t ip_src; |
155 | memcpy(&ip_src, buf + 12, 4); | 157 | memcpy(&ip_src, buf + 12, 4); |