Attention is currently required from: plaisthos.
Hello plaisthos,
I'd like you to do a code review.
Please visit
http://gerrit.openvpn.net/c/openvpn/+/1137?usp=email
to review the following change.
Change subject: ssl: Fix -Wconversion warnings in pem_password_callback
......................................................................
ssl: Fix -Wconversion warnings in pem_password_callback
The OpenSSL API is how it is, so adapt with casts.
Change-Id: I053ddbb71cc5b9ae16c5a49be833035d943d7eba
Signed-off-by: Frank Lichtenheld <fr...@li...>
---
M src/openvpn/ssl.c
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/37/1137/1
diff --git a/src/openvpn/ssl.c b/src/openvpn/ssl.c
index 17065aa..d586ba1 100644
--- a/src/openvpn/ssl.c
+++ b/src/openvpn/ssl.c
@@ -264,10 +264,10 @@
/* prompt for password even if --askpass wasn't specified */
pem_password_setup(NULL);
ASSERT(!passbuf.protected);
- strncpynt(buf, passbuf.password, size);
+ strncpynt(buf, passbuf.password, (size_t)size);
purge_user_pass(&passbuf, false);
- return strlen(buf);
+ return (int)strlen(buf);
}
return 0;
}
--
To view, visit http://gerrit.openvpn.net/c/openvpn/+/1137?usp=email
To unsubscribe, or for help writing mail filters, visit http://gerrit.openvpn.net/settings
Gerrit-Project: openvpn
Gerrit-Branch: master
Gerrit-Change-Id: I053ddbb71cc5b9ae16c5a49be833035d943d7eba
Gerrit-Change-Number: 1137
Gerrit-PatchSet: 1
Gerrit-Owner: flichtenheld <fr...@li...>
Gerrit-Reviewer: plaisthos <arn...@rf...>
Gerrit-CC: openvpn-devel <ope...@li...>
Gerrit-Attention: plaisthos <arn...@rf...>
Gerrit-MessageType: newchange
|
