search

ntop / n2n

Peer-to-peer VPN
GNU General Public License v3.0
6.18k stars 930 forks source link

Add CI for Code Analysing #992

Closed hamishcoleman closed 2 years ago

hamishcoleman commented 2 years ago

This adds a number of code analysis steps to the automated CI. They will try to catch memory boundary access errors and memory leaks as well as provide a number of warnings for suspicious memory manipulation.

All the fatal errors have been addressed in this branch, but there are still a number of warnings that should be looked at.

The tests done here will generally only trigger for code paths covered by the test suite.

codecov-commenter commented 2 years ago

Codecov Report

Merging #992 (a00aa63) into dev (23e168b) will decrease coverage by 0.00%. The diff coverage is 73.68%.

@@            Coverage Diff             @@
##              dev     #992      +/-   ##
==========================================
- Coverage   21.05%   21.05%   -0.01%     
==========================================
  Files          43       43              
  Lines        8315     8320       +5     
==========================================
+ Hits         1751     1752       +1     
- Misses       6564     6568       +4
Impacted Files Coverage Δ
src/edge_utils.c 1.60% <0.00%> (-0.01%) :arrow_down:
src/n2n.c 0.97% <0.00%> (-0.01%) :arrow_down:
src/sn_utils.c 0.00% <0.00%> (ø)
src/auth.c 65.07% <100.00%> (ø)
src/cc20.c 77.77% <100.00%> (+0.31%) :arrow_up:
src/transform_aes.c 71.76% <100.00%> (ø)
src/transform_cc20.c 89.23% <100.00%> (ø)
src/transform_speck.c 92.30% <100.00%> (ø)
src/transform_tf.c 73.41% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 23e168b...a00aa63. Read the comment docs.