argus-3.0.6.1 bug report

[David Binderman]

Hello there,

I just ran static analysis checker "cppcheck" over the
source code of argus-3.0.6.1

It said

[argus_client.c:2390] -> [argus_client.c:2398]: (warning) Variable 'pkts' is reassigned a value before the old one has been used. 'break;' missing?
[argus_client.c:2391] -> [argus_client.c:2399]: (warning) Variable 'bytes' is reassigned a value before the old one has been used. 'break;' missing?
[argus_client.c:2392] -> [argus_client.c:2400]: (warning) Variable 'appbytes' is reassigned a value before the old one has been used. 'break;' missing?

Suggest add missing break.

It also said

[argus_client.c:11797]: (error) Resource leak: fd
[argus_label.c:339]: (error) Resource leak: fd
[argus_label.c:1800]: (error) Resource leak: fd
[radium.c:3048]: (error) Resource leak: ffd
[ranonymize.c:1882]: (error) Resource leak: fd

which might be worth fixing.

Regards

David Binderman