Your patch has been applied to the master and release/2.4 branch.
Tested on FreeBSD 10, Linux and, indeed, AIX 7 :-) - Linux/gcc is nicely
clean, FreeBSD/clang finds a few that are silly and want fixing...
../../../openvpn/src/openvpn/push.c:42:13: warning: unused variable
'push_reply_cmd' [-Wunused-variable]
static char push_reply_cmd[] = "PUSH_REPLY";
../../../openvpn/src/openvpn/ssl_verify.c:1222:1: warning: unused label
'cleanup' [-Wunused-label]
cleanup:
AIX 7 with gcc has quite a few warnings, but those are "AIX is annoying"
style things like these two:
options.c:4947:16: warning: missing braces around initializer
[-Wmissing-braces]
options.c:4947:16: warning: (near initialization for 'remote.u6_addr')
[-Wmissing-braces]
tun.c:1791:13: warning: implicit declaration of function 'if_nametoindex'
[-Wimplicit-function-declaration]
and interesting enough, one that should have been visible on the other
platforms as well - maybe different OpenSSL version?
ssl_openssl.c: In function 'backend_tls_ctx_reload_crl':
ssl_openssl.c:967:13: warning: value computed is not used [-Wunused-value]
.. but overall, AIX is much less noisy than I would have expected
(like, 20 warnings in total, half of them format warnings that Selva's
patch will address). Good :-)
I have applied the patch to release/2.4 as well, as my worries about
"breaking some platform that previously was compiling nicely" shouldn't
be triggered (this is just turning on *warnings*, after all), and also
because the amount of warnings triggered is so small.
commit adbf68c00bf40089489c5e039138f855fc5e2392 (master)
commit b9d186211c87f52711592f1c29399a088912bf39 (release/2.4)
Author: Steffan Karger
Date: Thu Feb 1 16:45:21 2018 +0100
Enable stricter compiler warnings by default
Signed-off-by: Steffan Karger <[email protected]>
Acked-by: Selva Nair <[email protected]>
Message-Id: <[email protected]>
URL:
https://www.mail-archive.com/[email protected]/msg16426.html
Signed-off-by: Gert Doering <[email protected]>
--
kind regards,
Gert Doering
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
