aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLibravatar Kelvin M. Klann <kmk3.code@protonmail.com>2023-07-27 08:46:09 -0300
committerLibravatar Kelvin M. Klann <kmk3.code@protonmail.com>2023-07-27 08:49:36 -0300
commit16dc7448ca0682c73ddd3150d93bcb62c60f7933 (patch)
treeca39c0d99a56e1b99aa00352c4e5166b9d15aa27
parentMerge branch 'master' of ssh://github.com/netblue30/firejail (diff)
downloadfirejail-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.c6
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);