cron2 has uploaded a new patch set (#2) to the change originally created by flichtenheld. ( http://gerrit.openvpn.net/c/openvpn/+/1355?usp=email )
The following approvals got outdated and were removed: Code-Review+2 by plaisthos Change subject: ssl: Change tls_send_payload size argument to size_t ...................................................................... ssl: Change tls_send_payload size argument to size_t There is only one caller of this function and it wants it to be size_t. So move the size_t to int conversion one step down in the call chain. Do not switch key_state_write_plaintext_const, yet, since that is a backend function and so needs way more work. Change-Id: Ic90c5a0e48bda4a02d5e11c4c161f388cc8805af Signed-off-by: Frank Lichtenheld <[email protected]> Acked-by: Arne Schwabe <[email protected]> Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1355 Message-Id: <[email protected]> URL: https://www.mail-archive.com/[email protected]/msg34320.html Signed-off-by: Gert Doering <[email protected]> --- M src/openvpn/forward.c M src/openvpn/ssl.c M src/openvpn/ssl.h 3 files changed, 10 insertions(+), 9 deletions(-) git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/55/1355/2 diff --git a/src/openvpn/forward.c b/src/openvpn/forward.c index c355f66..aa1f858 100644 --- a/src/openvpn/forward.c +++ b/src/openvpn/forward.c @@ -365,11 +365,6 @@ } } -#if defined(__GNUC__) || defined(__clang__) -#pragma GCC diagnostic push -#pragma GCC diagnostic ignored "-Wconversion" -#endif - bool send_control_channel_string_dowork(struct tls_session *session, const char *str, msglvl_t msglevel) @@ -834,6 +829,11 @@ #endif /* ENABLE_MANAGEMENT */ } +#if defined(__GNUC__) || defined(__clang__) +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wconversion" +#endif + static void check_coarse_timers(struct context *c) { diff --git a/src/openvpn/ssl.c b/src/openvpn/ssl.c index 908854a..398c9ae 100644 --- a/src/openvpn/ssl.c +++ b/src/openvpn/ssl.c @@ -4032,7 +4032,7 @@ */ bool -tls_send_payload(struct key_state *ks, const uint8_t *data, int size) +tls_send_payload(struct key_state *ks, const uint8_t *data, size_t size) { bool ret = false; @@ -4042,7 +4042,8 @@ if (ks->state >= S_ACTIVE) { - if (key_state_write_plaintext_const(&ks->ks_ssl, data, size) == 1) + ASSERT(size <= INT_MAX); + if (key_state_write_plaintext_const(&ks->ks_ssl, data, (int)size) == 1) { ret = true; } @@ -4053,7 +4054,7 @@ { ks->paybuf = buffer_list_new(); } - buffer_list_push_data(ks->paybuf, data, (size_t)size); + buffer_list_push_data(ks->paybuf, data, size); ret = true; } diff --git a/src/openvpn/ssl.h b/src/openvpn/ssl.h index ffcc7c4..db8a798 100644 --- a/src/openvpn/ssl.h +++ b/src/openvpn/ssl.h @@ -426,7 +426,7 @@ /* * Send a payload over the TLS control channel */ -bool tls_send_payload(struct key_state *ks, const uint8_t *data, int size); +bool tls_send_payload(struct key_state *ks, const uint8_t *data, size_t size); /* * Receive a payload through the TLS control channel -- To view, visit http://gerrit.openvpn.net/c/openvpn/+/1355?usp=email To unsubscribe, or for help writing mail filters, visit http://gerrit.openvpn.net/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: openvpn Gerrit-Branch: master Gerrit-Change-Id: Ic90c5a0e48bda4a02d5e11c4c161f388cc8805af Gerrit-Change-Number: 1355 Gerrit-PatchSet: 2 Gerrit-Owner: flichtenheld <[email protected]> Gerrit-Reviewer: plaisthos <[email protected]> Gerrit-CC: openvpn-devel <[email protected]>
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
