]> git.ipfire.org Git - thirdparty/krb5.git/commitdiff
Fix conformance issue in GSSAPI tests 1210/head
authorGreg Hudson <ghudson@mit.edu>
Wed, 18 Aug 2021 16:07:04 +0000 (12:07 -0400)
committerGreg Hudson <ghudson@mit.edu>
Mon, 23 Aug 2021 16:14:11 +0000 (12:14 -0400)
Although some C compilers allow a function returning void to be called
in a return statement from another function returning void, it isn't
conformant and generates an error from (at least) the HP-UX native
compiler.  Reported by Michael Osipov.

ticket: 9023 (new)
tags: pullup
target_version: 1.19-next
target_version: 1.18-next

src/tests/gssapi/common.c

index b49c13d5d40eb3e87f3b5c2ec262255b2a584e37..34f34f7a5706705dfea61cb96d675505879af00b 100644 (file)
@@ -118,10 +118,9 @@ establish_contexts(gss_OID imech, gss_cred_id_t icred, gss_cred_id_t acred,
                    gss_ctx_id_t *actx, gss_name_t *src_name, gss_OID *amech,
                    gss_cred_id_t *deleg_cred)
 {
-    return establish_contexts_ex(imech, icred, acred, tname, flags, ictx, actx,
-                                 GSS_C_NO_CHANNEL_BINDINGS,
-                                 GSS_C_NO_CHANNEL_BINDINGS, NULL, src_name,
-                                 amech, deleg_cred);
+    establish_contexts_ex(imech, icred, acred, tname, flags, ictx, actx,
+                          GSS_C_NO_CHANNEL_BINDINGS, GSS_C_NO_CHANNEL_BINDINGS,
+                          NULL, src_name, amech, deleg_cred);
 }
 
 void