From fde473477b555d99279db452dbb046fbcc183a81 Mon Sep 17 00:00:00 2001 From: Marvin Scholz Date: Sat, 8 May 2021 17:18:20 +0200 Subject: [PATCH] Fix: Do not initialize char* with an integer AUTH_ALTER_NOOP is 0, which is not necessarily equivalent to a NULL pointer, causing the compiler to complain about this assignment. And logically it seems not the right thing either as failed_arg and deny_arg will not hold an AUTH_ALTER_* enum value but a string. --- src/auth.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/auth.c b/src/auth.c index 5a550ac0..ac2a9cca 100644 --- a/src/auth.c +++ b/src/auth.c @@ -817,8 +817,8 @@ auth_t *auth_get_authenticator(xmlNodePtr node) auth->management_url = (char*)xmlGetProp(node, XMLSTR("management-url")); auth->filter_web_policy = AUTH_MATCHTYPE_MATCH; auth->filter_admin_policy = AUTH_MATCHTYPE_MATCH; - auth->failed_arg = AUTH_ALTER_NOOP; - auth->deny_arg = AUTH_ALTER_NOOP; + auth->failed_arg = NULL; + auth->deny_arg = NULL; for (i = 0; i < (sizeof(auth->filter_admin)/sizeof(*(auth->filter_admin))); i++) { auth->filter_admin[i].type = AUTH_MATCHTYPE_UNUSED;