From 5e13fc5671fec5a1415b8d82a6344ceb71942478 Mon Sep 17 00:00:00 2001 From: Phil Pennock Date: Mon, 9 Apr 2018 17:49:57 -0400 Subject: [PATCH] bugfix: heimdal interaction, check length clang noted that taking the address of a struct member will never be 0, so checking against 0 was wrong. It was a `.length` member. I've compiled RC4 with this change and deployed it to my box and I can still authenticate fine. --- src/src/auths/heimdal_gssapi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/src/auths/heimdal_gssapi.c b/src/src/auths/heimdal_gssapi.c index c0611b895..eafdccc5c 100644 --- a/src/src/auths/heimdal_gssapi.c +++ b/src/src/auths/heimdal_gssapi.c @@ -357,7 +357,7 @@ auth_heimdal_gssapi_server(auth_instance *ablock, uschar *initial_data) error_out = FAIL; goto ERROR_OUT; } - if (&gbufdesc_out.length != 0) { + if (gbufdesc_out.length != 0) { error_out = auth_get_data(&from_client, gbufdesc_out.value, gbufdesc_out.length); if (error_out != OK) -- 2.25.1