From: Lev Stipakov <[email protected]> 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 --- This change was reviewed on Gerrit and approved by at least one developer. I request to merge it to master. Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/2146 This mail reflects revision 2 of this Change. Acked-by according to Gerrit (reflected above): Razvan Cojocaru <[email protected]> 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; } _______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
