ntop / n2n

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

Safely remove encryption keys #1093

Closed shaxxx closed 1 year ago

shaxxx commented 1 year ago

Fixes supernode crash on Windows when no header encryption is used. This is not ideal, but these fields are initialized and used only when header encryption is used. Keep in mind, trying to reload communities while using header encryption will still crash on Windows. Think of it like a band aid until we have proper fix.

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage has no change and project coverage change: -0.01 :warning:

Comparison is base (39b9c6b) 19.92% compared to head (b3e25d1) 19.92%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #1093 +/- ## ========================================== - Coverage 19.92% 19.92% -0.01% ========================================== Files 47 47 Lines 8786 8787 +1 ========================================== Hits 1751 1751 - Misses 7035 7036 +1 ``` | [Impacted Files](https://codecov.io/gh/ntop/n2n/pull/1093?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [src/sn\_utils.c](https://codecov.io/gh/ntop/n2n/pull/1093?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL3NuX3V0aWxzLmM=) | `0.00% <0.00%> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.