ntop / n2n

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

compile fixes #1022

Closed Logan007 closed 2 years ago

Logan007 commented 2 years ago

Fixes #1018 Fixes #1019 Fixes #1020 Fixes #1021

codecov-commenter commented 2 years ago

Codecov Report

Merging #1022 (dec3fd1) into dev (7927bd7) will not change coverage. The diff coverage is 0.00%.

@@           Coverage Diff           @@
##              dev    #1022   +/-   ##
=======================================
  Coverage   20.86%   20.86%           
=======================================
  Files          47       47           
  Lines        8398     8398           
=======================================
  Hits         1752     1752           
  Misses       6646     6646           
Impacted Files Coverage Δ
tools/n2n-portfwd.c 0.00% <0.00%> (ø)
tools/n2n-route.c 0.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 7927bd7...dec3fd1. Read the comment docs.

hamishcoleman commented 2 years ago

We should also consider compiling in C99 mode (or C11 event)

However, this also raises the question of what compiler version the user who reported this error is actually using - since none of our build tests detected this, it is likely that they are not using a supported compiler version