On 15/08/18 13:32, Antonio Quartulli wrote: > Hi, > > On 15/08/18 13:15, Antonio Quartulli wrote: >> The "cleanup" label in ssl_verify.c:verify_user_pass_plugin() is used >> only when PLUGIN_DEF_AUTH is defined, therefore make the label >> definition dependent on the same define. >> >> At the moment, gcc throws an error for unused label if PLUGIN_DEF_AUTH >> is not defined. >> >> Signed-off-by: Antonio Quartulli <a...@unstable.cc> >> --- >> src/openvpn/ssl_verify.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/src/openvpn/ssl_verify.c b/src/openvpn/ssl_verify.c >> index 61872251..9dfe23f8 100644 >> --- a/src/openvpn/ssl_verify.c >> +++ b/src/openvpn/ssl_verify.c >> @@ -1220,7 +1220,9 @@ verify_user_pass_plugin(struct tls_session *session, >> const struct user_pass *up, >> msg(D_TLS_ERRORS, "TLS Auth Error (verify_user_pass_plugin): peer >> provided a blank username"); >> } >> >> +#ifdef PLUGIN_DEF_AUT >> cleanup: >> +#endif >> return retval; >> } >> > > > Not sure which gcc version is being used on Gitlab-CI, but my jobs are > failing with: > > ssl_verify.c:1197:13: error: label 'cleanup' used but not defined > goto cleanup; > ^~~~ > Makefile:680: recipe for target 'ssl_verify.o' failed > make[3]: *** [ssl_verify.o] Error 1 > > It seems this is not just a warning anymore.
Sorry, I acted before being fully awake. This is obviously not a warning because it tries to use a label that does not exist... Indeed my patch is wrong and I messed it up. Forget about this. I'll send a proper v2. Cheers, -- Antonio Quartulli
signature.asc
Description: OpenPGP digital signature
------------------------------------------------------------------------------ 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 Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
