Hi,

On Thu, Jul 09, 2020 at 12:16:02PM +0200, Arne Schwabe wrote:
> Signed-off-by: Arne Schwabe <a...@rfc2549.org>
> ---
>  src/openvpn/ssl.c | 5 +++--
>  src/openvpn/ssl.h | 7 -------
>  2 files changed, 3 insertions(+), 9 deletions(-)
> 
> diff --git a/src/openvpn/ssl.c b/src/openvpn/ssl.c
> index 668bcbd9..4ee4c245 100644
> --- a/src/openvpn/ssl.c
> +++ b/src/openvpn/ssl.c
> @@ -1988,8 +1988,9 @@ tls_session_update_crypto_params(struct tls_session 
> *session,
>      }
>      else
>      {
> -      /* Very hacky workaround and quick fix for our calculation
> -       * not correct to avoid a regression */
> +      /* Very hacky workaround and quick fix for frame calculation
> +       * different when adjusting frame size when the original and new cipher
> +       * are identical to avoid a regression with client without NCP */
>          return tls_session_generate_data_channel_keys(session);
>      }

This comment has strayed a bit, I think I'll move it over to the previous
patch (6/8) which introduces it - not so useful to add a comment and then
change it right in the next patch.

Also, somewhat unrelated change...

> diff --git a/src/openvpn/ssl.h b/src/openvpn/ssl.h
> index 2f6f7657..58a9b0d4 100644
> --- a/src/openvpn/ssl.h
> +++ b/src/openvpn/ssl.h
> @@ -89,13 +89,6 @@
>  #define TLS_MULTI_HORIZON 2     /* call tls_multi_process frequently for n 
> seconds after
>                                   * every packet sent/received action */
>  
> -/*
> - * The SSL/TLS worker thread will wait at most this many seconds for the
> - * interprocess communication pipe to the main thread to be ready to accept
> - * writes.
> - */
> -#define TLS_MULTI_THREAD_SEND_TIMEOUT 5
> -
>  /* Interval that tls_multi_process should call tls_authentication_status */
>  #define TLS_MULTI_AUTH_STATUS_INTERVAL 10

This one is easy enough, so

Acked-By: Gert Doering <g...@greenie.muc.de>

gert
-- 
"If was one thing all people took for granted, was conviction that if you 
 feed honest figures into a computer, honest figures come out. Never doubted 
 it myself till I met a computer with a sense of humor."
                             Robert A. Heinlein, The Moon is a Harsh Mistress

Gert Doering - Munich, Germany                             g...@greenie.muc.de

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to