Re: git: 2e2a1c8ecf63 - main - libgssapi: Remove set but unused variable.
- In reply to: Benjamin Kaduk : "Re: git: 2e2a1c8ecf63 - main - libgssapi: Remove set but unused variable."
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Tue, 20 Jun 2023 23:04:08 UTC
On 6/20/23 2:56 PM, Benjamin Kaduk wrote: > On Tue, Jun 20, 2023 at 9:29 AM John Baldwin <jhb@freebsd.org> wrote: > >> The branch main has been updated by jhb: >> >> URL: >> https://cgit.FreeBSD.org/src/commit/?id=2e2a1c8ecf639317f40cdb1c2a0c5b9abe990e58 >> >> commit 2e2a1c8ecf639317f40cdb1c2a0c5b9abe990e58 >> Author: John Baldwin <jhb@FreeBSD.org> >> AuthorDate: 2023-06-20 16:28:59 +0000 >> Commit: John Baldwin <jhb@FreeBSD.org> >> CommitDate: 2023-06-20 16:28:59 +0000 >> >> libgssapi: Remove set but unused variable. >> >> Reported by: GCC >> --- >> lib/libgssapi/gss_delete_sec_context.c | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/lib/libgssapi/gss_delete_sec_context.c >> b/lib/libgssapi/gss_delete_sec_context.c >> index 930f51399cce..b64e32c5dc2c 100644 >> --- a/lib/libgssapi/gss_delete_sec_context.c >> +++ b/lib/libgssapi/gss_delete_sec_context.c >> @@ -41,7 +41,6 @@ gss_delete_sec_context(OM_uint32 *minor_status, >> gss_ctx_id_t *context_handle, >> gss_buffer_t output_token) >> { >> - OM_uint32 major_status; >> struct _gss_context *ctx = (struct _gss_context *) *context_handle; >> >> if (output_token) >> @@ -54,7 +53,7 @@ gss_delete_sec_context(OM_uint32 *minor_status, >> * otherwise fake an empty token. >> */ >> if (ctx->gc_ctx) { >> - major_status = ctx->gc_mech->gm_delete_sec_context( >> + (void) ctx->gc_mech->gm_delete_sec_context( >> > > Looks like gcc was catching a latent bug here -- if we consult the > corresponding heimdal implementation, major_status is initialized to > GSS_S_COMPLETE but this assignment can override it, and the function itself > returns major_status rather than a hardcoded GSS_S_COMPLETE. I'd be more than happy to have this fixed correctly. I've cc'd the original author (Doug) to see what he thinks. -- John Baldwin