argus-3.0.6.1 bug report

[Carter Bullard]

Hey David,
Thanks for the report… Definitely caught a bug in the new code thread,
which will be released as argus-clients-3.0.8 before the end of
the year.

You should download the latest argus threads from:

    http://qosient.com/argus/dev/argus-latest.tar.gz
    http://qosient.com/argus/dev/argus-clients-latest.tar.gz

So, cppcheck, that is a Windows app?  Any idea if there is a Linux
version ???  Are you doing a port?

Thanks again !!!!

Carter


On Sep 2, 2013, at 6:37 AM, David Binderman <dcb314 at hotmail.com> wrote:

> 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 		 	   		  

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 6837 bytes
Desc: not available
URL: <https://pairlist1.pair.net/pipermail/argus/attachments/20130903/36efc574/attachment.bin>