Skip to content

Commit 4a22af7

Browse files
simo5frozencemetery
authored andcommitted
Move check for persistent send
There are a few paths that lead to the done label being reached with a success status (ret = OK) so the check for sending the persistent header should probably be done after the done label to catch those other paths too. Signed-off-by: Simo Sorce <simo@redhat.com>
1 parent 318db0b commit 4a22af7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/mod_auth_gssapi.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1157,11 +1157,11 @@ static int mag_auth(request_rec *req)
11571157

11581158
ret = mag_complete(req_cfg, mc, client, mech_type, vtime, delegated_cred);
11591159

1160+
done:
11601161
if (ret == OK && req_cfg->send_persist)
11611162
apr_table_set(req->err_headers_out, "Persistent-Auth",
11621163
cfg->gss_conn_ctx ? "true" : "false");
11631164

1164-
done:
11651165
if ((auth_type != AUTH_TYPE_BASIC) && (output.length != 0)) {
11661166
int prefixlen = strlen(mag_str_auth_type(auth_type)) + 1;
11671167
replen = apr_base64_encode_len(output.length) + 1;

0 commit comments

Comments
 (0)