Gert Doering has submitted this change. ( https://gerrit.openvpn.net/c/openvpn/+/2146?usp=email )
Change subject: reflect_filter: initialise warning_displayed in the rate limiter ...................................................................... reflect_filter: initialise warning_displayed in the rate limiter initial_rate_limit_init() allocates with ALLOC_OBJ, which does not clear, and assigns every field but warning_displayed. The first period reset assigns it before anything reads it, so nothing misbehaves today, but there is no reason to leave a field undefined. Change-Id: Ied9e36d45b30a88610cafee668ecefa98b29eddc Signed-off-by: Lev Stipakov <[email protected]> Acked-by: Razvan Cojocaru <[email protected]> Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/2146 Message-Id: <[email protected]> URL: https://www.mail-archive.com/[email protected]/msg40963.html Signed-off-by: Gert Doering <[email protected]> --- M src/openvpn/reflect_filter.c 1 file changed, 1 insertion(+), 0 deletions(-) diff --git a/src/openvpn/reflect_filter.c b/src/openvpn/reflect_filter.c index e5226e4..953fff5 100644 --- a/src/openvpn/reflect_filter.c +++ b/src/openvpn/reflect_filter.c @@ -95,6 +95,7 @@ irl->period_length = period_length; irl->curr_period_counter = 0; irl->last_period_reset = 0; + irl->warning_displayed = false; return irl; } -- To view, visit https://gerrit.openvpn.net/c/openvpn/+/2146?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.openvpn.net/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: openvpn Gerrit-Branch: master Gerrit-Change-Id: Ied9e36d45b30a88610cafee668ecefa98b29eddc Gerrit-Change-Number: 2146 Gerrit-PatchSet: 3 Gerrit-Owner: Lev Stipakov <[email protected]> Gerrit-Reviewer: Arne Schwabe <[email protected]> Gerrit-Reviewer: Răzvan Cojocaru <[email protected]> Gerrit-CC: openvpn-devel <[email protected]>
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
