Make DEBUG macro invocations variadic

Use a script to update DEBUG macro invocations to use it as a variadic
macro, supplying format string and its arguments directly, instead of
wrapping them in parens.

This script was used to update the code:

grep -rwl --include '*.[hc]' DEBUG . |
    while read f; do
        mv "$f"{,.orig}
        perl -e \
            'use strict;
             use File::Slurp;
             my $text=read_file(\*STDIN);
             $text=~s#(\bDEBUG\s*\([^(]+)\((.*?)\)\s*\)\s*;#$1$2);#gs;
             print $text;' < "$f.orig" > "$f"
        rm "$f.orig"
    done

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Reviewed-by: Stephen Gallagher <sgallagh@redhat.com>
Reviewed-by: Simo Sorce <simo@redhat.com>

3 files changed, 144 insertions, 144 deletions

diff --git a/src/responder/pac/pacsrv.c b/src/responder/pac/pacsrv.c
index 7fdc476a2..2fb94293c 100644
--- a/src/responder/pac/pacsrv.c
+++ b/src/responder/pac/pacsrv.c
@@ -85,7 +85,7 @@ static void pac_dp_reconnect_init(struct sbus_connection *conn,
 
     /* Did we reconnect successfully? */
     if (status == SBUS_RECONNECT_SUCCESS) {
-        DEBUG(SSSDBG_OP_FAILURE, ("Reconnected to the Data Provider.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "Reconnected to the Data Provider.\n");
 
         /* Identify ourselves to the data provider */
         ret = dp_common_send_id(be_conn->conn,
@@ -99,8 +99,8 @@ static void pac_dp_reconnect_init(struct sbus_connection *conn,
     }
 
     /* Failed to reconnect */
-    DEBUG(SSSDBG_FATAL_FAILURE, ("Could not reconnect to %s provider.\n",
-              be_conn->domain->name));
+    DEBUG(SSSDBG_FATAL_FAILURE, "Could not reconnect to %s provider.\n",
+              be_conn->domain->name);
 
     /* FIXME: kill the frontend and let the monitor restart it ? */
     /* nss_shutdown(rctx); */
@@ -131,13 +131,13 @@ int pac_process_init(TALLOC_CTX *mem_ctx,
                            "PAC", &pac_dp_interface,
                            &rctx);
     if (ret != EOK) {
-        DEBUG(SSSDBG_FATAL_FAILURE, ("sss_process_init() failed\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "sss_process_init() failed\n");
         return ret;
     }
 
     pac_ctx = talloc_zero(rctx, struct pac_ctx);
     if (!pac_ctx) {
-        DEBUG(SSSDBG_FATAL_FAILURE, ("fatal error initializing pac_ctx\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "fatal error initializing pac_ctx\n");
         ret = ENOMEM;
         goto fail;
     }
@@ -150,7 +150,7 @@ int pac_process_init(TALLOC_CTX *mem_ctx,
                             CONFDB_PAC_CONF_ENTRY, CONFDB_SERVICE_ALLOWED_UIDS,
                             DEFAULT_ALLOWED_UIDS, &uid_str);
     if (ret != EOK) {
-        DEBUG(SSSDBG_FATAL_FAILURE, ("Failed to get allowed UIDs.\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "Failed to get allowed UIDs.\n");
         goto fail;
     }
 
@@ -159,7 +159,7 @@ int pac_process_init(TALLOC_CTX *mem_ctx,
                                   &pac_ctx->rctx->allowed_uids);
     talloc_free(uid_str);
     if (ret != EOK) {
-        DEBUG(SSSDBG_FATAL_FAILURE, ("Failed to set allowed UIDs.\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "Failed to set allowed UIDs.\n");
         goto fail;
     }
 
@@ -169,7 +169,7 @@ int pac_process_init(TALLOC_CTX *mem_ctx,
                          CONFDB_SERVICE_RECON_RETRIES,
                          3, &max_retries);
     if (ret != EOK) {
-        DEBUG(SSSDBG_FATAL_FAILURE, ("Failed to set up automatic reconnection\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "Failed to set up automatic reconnection\n");
         goto fail;
     }
 
@@ -181,7 +181,7 @@ int pac_process_init(TALLOC_CTX *mem_ctx,
     err = sss_idmap_init(sss_idmap_talloc, pac_ctx, sss_idmap_talloc_free,
                          &pac_ctx->idmap_ctx);
     if (err != IDMAP_SUCCESS) {
-        DEBUG(SSSDBG_FATAL_FAILURE, ("sss_idmap_init failed.\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "sss_idmap_init failed.\n");
         ret = EFAULT;
         goto fail;
     }
@@ -194,18 +194,18 @@ int pac_process_init(TALLOC_CTX *mem_ctx,
                          &fd_limit);
     if (ret != EOK) {
         DEBUG(SSSDBG_FATAL_FAILURE,
-              ("Failed to set up file descriptor limit\n"));
+              "Failed to set up file descriptor limit\n");
         goto fail;
     }
     responder_set_fd_limit(fd_limit);
 
     ret = schedule_get_domains_task(rctx, rctx->ev, rctx);
     if (ret != EOK) {
-        DEBUG(SSSDBG_FATAL_FAILURE, ("schedule_get_domains_tasks failed.\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "schedule_get_domains_tasks failed.\n");
         goto fail;
     }
 
-    DEBUG(SSSDBG_TRACE_FUNC, ("PAC Initialization complete\n"));
+    DEBUG(SSSDBG_TRACE_FUNC, "PAC Initialization complete\n");
 
     return EOK;
 
@@ -254,7 +254,7 @@ int main(int argc, const char *argv[])
     ret = die_if_parent_died();
     if (ret != EOK) {
         /* This is not fatal, don't return */
-        DEBUG(SSSDBG_OP_FAILURE, ("Could not set up to exit when parent process does\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "Could not set up to exit when parent process does\n");
     }
 
     ret = pac_process_init(main_ctx,
diff --git a/src/responder/pac/pacsrv_cmd.c b/src/responder/pac/pacsrv_cmd.c
index 40ce46bbd..229c30783 100644
--- a/src/responder/pac/pacsrv_cmd.c
+++ b/src/responder/pac/pacsrv_cmd.c
@@ -36,8 +36,8 @@ static errno_t pac_cmd_done(struct cli_ctx *cctx, int cmd_ret)
     ret = sss_packet_new(cctx->creq, 0, sss_packet_get_cmd(cctx->creq->in),
                          &cctx->creq->out);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sss_packet_new failed [%d][%s].\n",
-                                  ret, strerror(ret)));
+        DEBUG(SSSDBG_OP_FAILURE, "sss_packet_new failed [%d][%s].\n",
+                                  ret, strerror(ret));
         return ret;
     }
 
@@ -110,7 +110,7 @@ static errno_t pac_add_pac_user(struct cli_ctx *cctx)
 
     pr_ctx = talloc_zero(cctx, struct pac_req_ctx);
     if (pr_ctx == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_new failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_new failed.\n");
         return ENOMEM;
     }
 
@@ -118,27 +118,27 @@ static errno_t pac_add_pac_user(struct cli_ctx *cctx)
 
     pr_ctx->pac_ctx = talloc_get_type(cctx->rctx->pvt_ctx,  struct pac_ctx);
     if (pr_ctx->pac_ctx == NULL) {
-        DEBUG(SSSDBG_FATAL_FAILURE, ("Cannot find pac responder context.\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "Cannot find pac responder context.\n");
         return EINVAL;
     }
 
     ret = get_data_from_pac(pr_ctx, body, blen,
                             &pr_ctx->logon_info);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("get_data_from_pac failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "get_data_from_pac failed.\n");
         goto done;
     }
 
     pr_ctx->domain_name = pr_ctx->logon_info->info3.base.logon_domain.string;
     if (pr_ctx->domain_name == NULL) {
-        DEBUG(SSSDBG_FATAL_FAILURE, ("No domain name in PAC"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "No domain name in PAC");
         ret = EINVAL;
         goto done;
     }
 
     if (pr_ctx->logon_info->info3.base.account_name.string == NULL) {
         ret = EINVAL;
-        DEBUG(SSSDBG_FATAL_FAILURE, ("Missing account name in PAC.\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "Missing account name in PAC.\n");
         goto done;
     }
 
@@ -148,7 +148,7 @@ static errno_t pac_add_pac_user(struct cli_ctx *cctx)
                             pr_ctx->logon_info->info3.base.account_name.string);
     if (pr_ctx->user_name == NULL) {
         ret = ENOMEM;
-        DEBUG(SSSDBG_FATAL_FAILURE, ("sss_tc_utf8_str_tolower failed.\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "sss_tc_utf8_str_tolower failed.\n");
         goto done;
     }
 
@@ -156,7 +156,7 @@ static errno_t pac_add_pac_user(struct cli_ctx *cctx)
                                    pr_ctx->logon_info->info3.base.domain_sid,
                                    &pr_ctx->user_dom_sid_str);
     if (err != IDMAP_SUCCESS) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sss_idmap_smb_sid_to_sid failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sss_idmap_smb_sid_to_sid failed.\n");
         ret = EFAULT;
         goto done;
     }
@@ -176,7 +176,7 @@ static errno_t pac_add_pac_user(struct cli_ctx *cctx)
         }
         goto done;
     } else if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("responder_get_domain_by_id failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "responder_get_domain_by_id failed.\n");
         goto done;
     }
 
@@ -205,8 +205,8 @@ static void pac_get_domains_done(struct tevent_req *req)
     ret = responder_get_domain_by_id(cctx->rctx, pr_ctx->user_dom_sid_str,
                                      &pr_ctx->dom);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("Corresponding domain [%s] has not been "
-                                  "found\n", pr_ctx->user_dom_sid_str));
+        DEBUG(SSSDBG_OP_FAILURE, "Corresponding domain [%s] has not been "
+                                  "found\n", pr_ctx->user_dom_sid_str);
         ret = ENOENT;
         goto done;
     }
@@ -230,14 +230,14 @@ static errno_t pac_resolve_sids_next(struct pac_req_ctx *pr_ctx)
                             &pr_ctx->primary_group_sid_str,
                             &pr_ctx->sid_table);
     if (ret != 0) {
-        DEBUG(SSSDBG_OP_FAILURE, ("get_sids_from_pac failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "get_sids_from_pac failed.\n");
         return ret;
     }
 
     req = pac_lookup_sids_send(pr_ctx, pr_ctx->cctx->ev, pr_ctx,
                                pr_ctx->pac_ctx, pr_ctx->sid_table);
     if (req == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("pac_lookup_sids_send failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "pac_lookup_sids_send failed.\n");
         return ENOMEM;
     }
 
@@ -281,8 +281,8 @@ static void pac_lookup_sids_done(struct tevent_req *req)
             ret =responder_get_domain_by_id(cctx->rctx,
                                             entries[c].key.str, &dom);
             if (ret != EOK) {
-                DEBUG(SSSDBG_OP_FAILURE, ("No domain found for SID [%s].\n",
-                                          entries[c].key.str));
+                DEBUG(SSSDBG_OP_FAILURE, "No domain found for SID [%s].\n",
+                                          entries[c].key.str);
                 continue;
             }
 
@@ -290,19 +290,19 @@ static void pac_lookup_sids_done(struct tevent_req *req)
             ret = sysdb_search_object_by_sid(pr_ctx, dom, entries[c].key.str,
                                              NULL, &msg);
             if (ret != EOK) {
-                DEBUG(SSSDBG_OP_FAILURE, ("sysdb_search_object_by_sid " \
-                                          "failed.\n"));
+                DEBUG(SSSDBG_OP_FAILURE, "sysdb_search_object_by_sid " \
+                                          "failed.\n");
                 continue;
             }
 
             if (msg->count == 0) {
-                DEBUG(SSSDBG_OP_FAILURE, ("No entry found for SID [%s].\n",
-                                          entries[c].key.str));
+                DEBUG(SSSDBG_OP_FAILURE, "No entry found for SID [%s].\n",
+                                          entries[c].key.str);
                 continue;
             } else if (msg->count > 1) {
-                DEBUG(SSSDBG_CRIT_FAILURE, ("More then one result returned " \
+                DEBUG(SSSDBG_CRIT_FAILURE, "More then one result returned " \
                                             "for SID [%s].\n",
-                                            entries[c].key.str));
+                                            entries[c].key.str);
                 talloc_free(msg);
                 pac_cmd_done(cctx, EINVAL);
                 return;
@@ -316,7 +316,7 @@ static void pac_lookup_sids_done(struct tevent_req *req)
             }
 
             if (id == 0) {
-                DEBUG(SSSDBG_OP_FAILURE, ("No ID found in entry.\n"));
+                DEBUG(SSSDBG_OP_FAILURE, "No ID found in entry.\n");
                 talloc_free(msg);
                 continue;
             }
@@ -326,8 +326,8 @@ static void pac_lookup_sids_done(struct tevent_req *req)
 
             ret = hash_enter(pr_ctx->sid_table, &key, &value);
             if (ret != HASH_SUCCESS) {
-                DEBUG(SSSDBG_OP_FAILURE, ("hash_enter failed [%d][%s].\n",
-                                          ret, hash_error_string(ret)));
+                DEBUG(SSSDBG_OP_FAILURE, "hash_enter failed [%d][%s].\n",
+                                          ret, hash_error_string(ret));
                 continue;
             }
             talloc_free(msg);
@@ -345,7 +345,7 @@ static void pac_add_user_next(struct pac_req_ctx *pr_ctx)
 
     ret = save_pac_user(pr_ctx);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("save_pac_user failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "save_pac_user failed.\n");
         goto done;
     }
 
@@ -353,7 +353,7 @@ static void pac_add_user_next(struct pac_req_ctx *pr_ctx)
                                 &pr_ctx->del_grp_list,
                                 &pr_ctx->add_sid_count, &pr_ctx->add_sids);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("pac_user_get_grp_info failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "pac_user_get_grp_info failed.\n");
         goto done;
     }
 
@@ -405,26 +405,26 @@ static errno_t pac_user_get_grp_info(TALLOC_CTX *mem_ctx,
     sysdb = pr_ctx->dom->sysdb;
     if (sysdb == NULL) {
         ret = EINVAL;
-        DEBUG(SSSDBG_FATAL_FAILURE, ("Fatal: Sysdb CTX not found for this domain!\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "Fatal: Sysdb CTX not found for this domain!\n");
         goto done;
     }
 
     tmp_ctx = talloc_new(NULL);
     if (tmp_ctx == NULL) {
         ret = ENOMEM;
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_new failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_new failed.\n");
         goto done;
     }
 
     ret = sysdb_initgroups(tmp_ctx, pr_ctx->dom, pr_ctx->user_name, &res);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sysdb_initgroups failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sysdb_initgroups failed.\n");
         goto done;
     }
 
     if (res->count == 0) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sysdb_initgroups did not found [%s].\n",
-                                  pr_ctx->user_name));
+        DEBUG(SSSDBG_OP_FAILURE, "sysdb_initgroups did not found [%s].\n",
+                                  pr_ctx->user_name);
         ret = ENOENT;
         goto done;
     }
@@ -435,7 +435,7 @@ static errno_t pac_user_get_grp_info(TALLOC_CTX *mem_ctx,
         del_grp_list = talloc_zero_array(tmp_ctx, struct grp_info,
                                          del_grp_count);
         if (del_grp_list == NULL) {
-            DEBUG(SSSDBG_OP_FAILURE, ("talloc_array failed.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "talloc_array failed.\n");
             ret = ENOMEM;
             goto done;
         }
@@ -447,7 +447,7 @@ static errno_t pac_user_get_grp_info(TALLOC_CTX *mem_ctx,
             cur_sid = ldb_msg_find_attr_as_string(res->msgs[c + 1],
                                                   SYSDB_SID_STR, NULL);
             if (cur_sid == NULL) {
-                DEBUG(SSSDBG_OP_FAILURE, ("Missing SID in group entry.\n"));
+                DEBUG(SSSDBG_OP_FAILURE, "Missing SID in group entry.\n");
                 ret = EINVAL;
                 goto done;
             }
@@ -455,20 +455,20 @@ static errno_t pac_user_get_grp_info(TALLOC_CTX *mem_ctx,
             key.str = discard_const(cur_sid);
             ret = hash_lookup(pr_ctx->sid_table, &key, &value);
             if (ret == HASH_SUCCESS) {
-                DEBUG(SSSDBG_TRACE_ALL, ("User [%s] already member of group " \
+                DEBUG(SSSDBG_TRACE_ALL, "User [%s] already member of group " \
                                          "with SID [%s].\n",
-                                         pr_ctx->user_name, cur_sid));
+                                         pr_ctx->user_name, cur_sid);
 
                 ret = hash_delete(pr_ctx->sid_table, &key);
                 if (ret != HASH_SUCCESS) {
-                    DEBUG(SSSDBG_OP_FAILURE, ("Failed to remove hash entry.\n"));
+                    DEBUG(SSSDBG_OP_FAILURE, "Failed to remove hash entry.\n");
                     ret = EIO;
                     goto done;
                 }
             } else if (ret == HASH_ERROR_KEY_NOT_FOUND) {
-                DEBUG(SSSDBG_TRACE_INTERNAL, ("Group with SID [%s] is not in " \
+                DEBUG(SSSDBG_TRACE_INTERNAL, "Group with SID [%s] is not in " \
                                               "the PAC anymore, membership " \
-                                              "must be removed.\n", cur_sid));
+                                              "must be removed.\n", cur_sid);
 
                 tmp_str = ldb_msg_find_attr_as_string(res->msgs[c + 1],
                                                       SYSDB_ORIG_DN, NULL);
@@ -476,7 +476,7 @@ static errno_t pac_user_get_grp_info(TALLOC_CTX *mem_ctx,
                     del_grp_list[del_idx].orig_dn = talloc_strdup(del_grp_list,
                                                                   tmp_str);
                     if (del_grp_list[del_idx].orig_dn == NULL) {
-                        DEBUG(SSSDBG_OP_FAILURE, ("talloc_strdup failed.\n"));
+                        DEBUG(SSSDBG_OP_FAILURE, "talloc_strdup failed.\n");
                         ret = ENOMEM;
                         goto done;
                     }
@@ -485,7 +485,7 @@ static errno_t pac_user_get_grp_info(TALLOC_CTX *mem_ctx,
                 del_grp_list[del_idx].dn = ldb_dn_copy(del_grp_list,
                                                        res->msgs[c + 1]->dn);
                 if (del_grp_list[del_idx].dn == NULL) {
-                    DEBUG(SSSDBG_OP_FAILURE, ("ldb_dn_copy failed.\n"));
+                    DEBUG(SSSDBG_OP_FAILURE, "ldb_dn_copy failed.\n");
                     ret = ENOMEM;
                     goto done;
                 }
@@ -500,7 +500,7 @@ static errno_t pac_user_get_grp_info(TALLOC_CTX *mem_ctx,
     if (add_sid_count > 0) {
         add_sids = talloc_array(tmp_ctx, char *, add_sid_count);
         if (add_sids == NULL) {
-            DEBUG(SSSDBG_OP_FAILURE, ("talloc_array failed.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "talloc_array failed.\n");
             ret = ENOMEM;
             goto done;
         }
@@ -511,12 +511,12 @@ static errno_t pac_user_get_grp_info(TALLOC_CTX *mem_ctx,
             if (strcmp(entry->key.str, pr_ctx->user_sid_str) != 0) {
                 add_sids[c] = talloc_strdup(add_sids, entry->key.str);
                 if (add_sids[c] == NULL) {
-                    DEBUG(SSSDBG_OP_FAILURE, ("talloc_strdup failed.\n"));
+                    DEBUG(SSSDBG_OP_FAILURE, "talloc_strdup failed.\n");
                     ret = ENOMEM;
                     goto done;
                 }
-                DEBUG(SSSDBG_TRACE_ALL, ("SID [%s] added to add_sids " \
-                                         "list.\n", entry->key.str));
+                DEBUG(SSSDBG_TRACE_ALL, "SID [%s] added to add_sids " \
+                                         "list.\n", entry->key.str);
                 c++;
             }
         }
@@ -553,14 +553,14 @@ static errno_t save_pac_user(struct pac_req_ctx *pr_ctx)
     sysdb = pr_ctx->dom->sysdb;
     if (sysdb == NULL) {
         ret = EINVAL;
-        DEBUG(SSSDBG_FATAL_FAILURE, ("Fatal: Sysdb CTX not found for this domain!\n"));
+        DEBUG(SSSDBG_FATAL_FAILURE, "Fatal: Sysdb CTX not found for this domain!\n");
         goto done;
     }
 
     tmp_ctx = talloc_new(NULL);
     if (tmp_ctx == NULL) {
         ret = ENOMEM;
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_new failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_new failed.\n");
         goto done;
     }
 
@@ -568,7 +568,7 @@ static errno_t save_pac_user(struct pac_req_ctx *pr_ctx)
                            pr_ctx->primary_group_sid_str, pr_ctx->sid_table,
                            pr_ctx->logon_info, &pwd, &user_attrs);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("get_pwd_from_pac failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "get_pwd_from_pac failed.\n");
         goto done;
     }
 
@@ -576,7 +576,7 @@ static errno_t save_pac_user(struct pac_req_ctx *pr_ctx)
                                    &msg);
     if (ret == ENOENT) {
         if (pwd->pw_gid == 0 && !pr_ctx->dom->mpg) {
-            DEBUG(SSSDBG_CRIT_FAILURE, ("Primary group RID from the PAC " \
+            DEBUG(SSSDBG_CRIT_FAILURE, "Primary group RID from the PAC " \
                                         "cannot be translated into a GID for " \
                                         "user [%s]. Typically this happens " \
                                         "when UIDs and GIDs are read from AD " \
@@ -584,7 +584,7 @@ static errno_t save_pac_user(struct pac_req_ctx *pr_ctx)
                                         "have a GID assigned. Make sure the " \
                                         "user is created by the ID provider " \
                                         "before GSSAPI based authentication " \
-                                        "is used in this case.", pwd->pw_name));
+                                        "is used in this case.", pwd->pw_name);
             ret = EINVAL;
             goto done;
         }
@@ -595,12 +595,12 @@ static errno_t save_pac_user(struct pac_req_ctx *pr_ctx)
                                pwd->pw_shell, NULL, user_attrs, NULL,
                                pr_ctx->dom->user_timeout, 0);
         if (ret != EOK) {
-            DEBUG(SSSDBG_OP_FAILURE, ("sysdb_store_user failed [%d][%s].\n",
-                                      ret, strerror(ret)));
+            DEBUG(SSSDBG_OP_FAILURE, "sysdb_store_user failed [%d][%s].\n",
+                                      ret, strerror(ret));
             goto done;
         }
     } else if (ret != EOK && ret != ENOENT) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sysdb_search_user_by_id failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sysdb_search_user_by_id failed.\n");
         goto done;
     }
 
@@ -639,15 +639,15 @@ struct tevent_req *pac_save_memberships_send(struct pac_req_ctx *pr_ctx)
 
     dom_name = sss_get_domain_name(state, pr_ctx->user_name, dom);
     if (dom_name == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_sprintf failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_sprintf failed.\n");
         ret = ENOMEM;
         goto done;
     }
 
     ret = sysdb_search_user_by_name(state, dom, dom_name, NULL, &msg);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sysdb_search_user_by_name failed " \
-                                  "[%d][%s].\n", ret, strerror(ret)));
+        DEBUG(SSSDBG_OP_FAILURE, "sysdb_search_user_by_name failed " \
+                                  "[%d][%s].\n", ret, strerror(ret));
         goto done;
     }
 
@@ -656,7 +656,7 @@ struct tevent_req *pac_save_memberships_send(struct pac_req_ctx *pr_ctx)
 
     ret = pac_save_memberships_delete(state);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("pac_save_memberships_delete failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "pac_save_memberships_delete failed.\n");
         goto done;
     }
 
@@ -694,19 +694,19 @@ pac_save_memberships_delete(struct pac_save_memberships_state *state)
     }
 
     if (pr_ctx->del_grp_list == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("Missing group list.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "Missing group list.\n");
         return EINVAL;
     }
 
     tmp_ctx = talloc_new(NULL);
     if (tmp_ctx == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_new failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_new failed.\n");
         return ENOMEM;
     }
 
     ret = sysdb_transaction_start(pr_ctx->dom->sysdb);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sysdb_transaction_start failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sysdb_transaction_start failed.\n");
         goto done;
     }
     in_transaction = true;
@@ -718,32 +718,32 @@ pac_save_memberships_delete(struct pac_save_memberships_state *state)
                                      pr_ctx->del_grp_list[c].dn,
                                      LDB_FLAG_MOD_DELETE);
         if (ret != EOK) {
-            DEBUG(SSSDBG_OP_FAILURE, ("sysdb_mod_group_member failed for " \
+            DEBUG(SSSDBG_OP_FAILURE, "sysdb_mod_group_member failed for " \
                                       "user [%s] and group[%s].\n",
                                       ldb_dn_get_linearized(state->user_dn),
                                       ldb_dn_get_linearized(
-                                                  pr_ctx->del_grp_list[c].dn)));
+                                                  pr_ctx->del_grp_list[c].dn));
             continue;
         }
 
         if (pr_ctx->del_grp_list[c].orig_dn != NULL) {
             user_attrs = sysdb_new_attrs(tmp_ctx);
             if (user_attrs == NULL) {
-                DEBUG(SSSDBG_OP_FAILURE, ("sysdb_new_attrs failed.\n"));
+                DEBUG(SSSDBG_OP_FAILURE, "sysdb_new_attrs failed.\n");
                 continue;
             }
 
             ret = sysdb_attrs_add_string(user_attrs, SYSDB_ORIG_MEMBEROF,
                                          pr_ctx->del_grp_list[c].orig_dn);
             if (ret != EOK) {
-                DEBUG(SSSDBG_OP_FAILURE, ("sysdb_attrs_add_string failed.\n"));
+                DEBUG(SSSDBG_OP_FAILURE, "sysdb_attrs_add_string failed.\n");
                 continue;
             }
 
             ret = sysdb_set_entry_attr(pr_ctx->dom->sysdb, state->user_dn, user_attrs,
                                        LDB_FLAG_MOD_DELETE);
             if (ret != EOK) {
-                DEBUG(SSSDBG_OP_FAILURE, ("sysdb_set_entry_attr failed.\n"));
+                DEBUG(SSSDBG_OP_FAILURE, "sysdb_set_entry_attr failed.\n");
                 continue;
             }
             talloc_free(user_attrs);
@@ -752,7 +752,7 @@ pac_save_memberships_delete(struct pac_save_memberships_state *state)
 
     ret = sysdb_transaction_commit(pr_ctx->dom->sysdb);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sysdb_transaction_commit failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sysdb_transaction_commit failed.\n");
         goto done;
     }
     in_transaction = false;
@@ -762,7 +762,7 @@ done:
     if (in_transaction) {
         sret = sysdb_transaction_cancel(pr_ctx->dom->sysdb);
         if (sret != EOK) {
-            DEBUG(SSSDBG_OP_FAILURE, ("sysdb_transaction_cancel failed.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "sysdb_transaction_cancel failed.\n");
         }
     }
 
@@ -788,7 +788,7 @@ static errno_t pac_save_memberships_next(struct tevent_req *req)
     }
 
     if (pr_ctx->add_sids == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("Missing list of SIDs.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "Missing list of SIDs.\n");
         return EINVAL;
     }
 
@@ -796,8 +796,8 @@ static errno_t pac_save_memberships_next(struct tevent_req *req)
         sid = pr_ctx->add_sids[state->sid_iter];
         ret = responder_get_domain_by_id(pr_ctx->pac_ctx->rctx, sid, &grp_dom);
         if (ret != EOK) {
-            DEBUG(SSSDBG_MINOR_FAILURE, ("responder_get_domain_by_id failed, " \
-                                         "will try next group\n"));
+            DEBUG(SSSDBG_MINOR_FAILURE, "responder_get_domain_by_id failed, " \
+                                         "will try next group\n");
             state->sid_iter++;
             continue;
         }
@@ -818,8 +818,8 @@ static errno_t pac_save_memberships_next(struct tevent_req *req)
 
             return EAGAIN;
         } else  {
-            DEBUG(SSSDBG_OP_FAILURE, ("pac_store_membership failed, "
-                                      "trying next group.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "pac_store_membership failed, "
+                                      "trying next group.\n");
             state->sid_iter++;
             continue;
         }
@@ -858,14 +858,14 @@ static void pac_get_group_done(struct tevent_req *subreq)
     sid = pr_ctx->add_sids[state->sid_iter];
     ret = responder_get_domain_by_id(pr_ctx->pac_ctx->rctx,sid, &grp_dom);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("responder_get_domain_by_id failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "responder_get_domain_by_id failed.\n");
         goto error;
     }
 
     ret = pac_store_membership(state->pr_ctx, state->user_dn, sid, grp_dom);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("pac_store_membership failed, "
-                                  "trying next group.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "pac_store_membership failed, "
+                                  "trying next group.\n");
     }
     state->sid_iter++;
 
@@ -904,48 +904,48 @@ pac_store_membership(struct pac_req_ctx *pr_ctx,
     ret = sysdb_search_object_by_sid(tmp_ctx, grp_dom, grp_sid_str,
                                      group_attrs, &group);
     if (ret != EOK) {
-        DEBUG(SSSDBG_TRACE_INTERNAL, ("sysdb_search_object_by_sid " \
+        DEBUG(SSSDBG_TRACE_INTERNAL, "sysdb_search_object_by_sid " \
                                       "for SID [%s] failed [%d][%s].\n",
-                                      grp_sid_str, ret, strerror(ret)));
+                                      grp_sid_str, ret, strerror(ret));
         goto done;
     }
 
     if (group->count != 1) {
-        DEBUG(SSSDBG_OP_FAILURE, ("Unexpected number of groups returned.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "Unexpected number of groups returned.\n");
         ret = EINVAL;
         goto done;
     }
 
     oc = ldb_msg_find_attr_as_string(group->msgs[0], SYSDB_OBJECTCLASS, NULL);
     if (oc == NULL || strcmp(oc, SYSDB_GROUP_CLASS) != 0) {
-        DEBUG(SSSDBG_OP_FAILURE, ("Return object does not have group " \
-                                  "objectclass.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "Return object does not have group " \
+                                  "objectclass.\n");
         ret = EINVAL;
         goto done;
     }
 
-    DEBUG(SSSDBG_TRACE_ALL, ("Adding user [%s] to group [%s][%s].\n",
+    DEBUG(SSSDBG_TRACE_ALL, "Adding user [%s] to group [%s][%s].\n",
                              ldb_dn_get_linearized(user_dn), grp_sid_str,
-                             ldb_dn_get_linearized(group->msgs[0]->dn)));
+                             ldb_dn_get_linearized(group->msgs[0]->dn));
     ret = sysdb_mod_group_member(grp_dom, user_dn, group->msgs[0]->dn,
                                  LDB_FLAG_MOD_ADD);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sysdb_mod_group_member failed user [%s] " \
+        DEBUG(SSSDBG_OP_FAILURE, "sysdb_mod_group_member failed user [%s] " \
                                   "group [%s].\n",
                                   ldb_dn_get_linearized(user_dn),
-                                  ldb_dn_get_linearized(group->msgs[0]->dn)));
+                                  ldb_dn_get_linearized(group->msgs[0]->dn));
         goto done;
     }
 
     orig_group_dn = ldb_msg_find_attr_as_string(group->msgs[0], SYSDB_ORIG_DN,
                                                 NULL);
     if (orig_group_dn != NULL) {
-        DEBUG(SSSDBG_TRACE_ALL, ("Adding original group DN [%s] to user [%s].\n",
+        DEBUG(SSSDBG_TRACE_ALL, "Adding original group DN [%s] to user [%s].\n",
                                  orig_group_dn,
-                                 ldb_dn_get_linearized(user_dn)));
+                                 ldb_dn_get_linearized(user_dn));
         user_attrs = sysdb_new_attrs(tmp_ctx);
         if (user_attrs == NULL) {
-            DEBUG(SSSDBG_OP_FAILURE, ("sysdb_new_attrs failed.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "sysdb_new_attrs failed.\n");
             ret = ENOMEM;
             goto done;
         }
@@ -953,20 +953,20 @@ pac_store_membership(struct pac_req_ctx *pr_ctx,
         ret = sysdb_attrs_add_string(user_attrs, SYSDB_ORIG_MEMBEROF,
                                      orig_group_dn);
         if (ret != EOK) {
-            DEBUG(SSSDBG_OP_FAILURE, ("sysdb_attrs_add_string failed.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "sysdb_attrs_add_string failed.\n");
             goto done;
         }
 
         ret = sysdb_set_entry_attr(pr_ctx->dom->sysdb, user_dn, user_attrs,
                                    LDB_FLAG_MOD_ADD);
         if (ret != EOK) {
-            DEBUG(SSSDBG_OP_FAILURE, ("sysdb_set_entry_attr failed.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "sysdb_set_entry_attr failed.\n");
             goto done;
         }
     } else {
-        DEBUG(SSSDBG_MINOR_FAILURE, ("Original DN not available for group " \
+        DEBUG(SSSDBG_MINOR_FAILURE, "Original DN not available for group " \
                                      "[%s][%s].\n", grp_sid_str,
-                                     ldb_dn_get_linearized(group->msgs[0]->dn)));
+                                     ldb_dn_get_linearized(group->msgs[0]->dn));
     }
 
 done:
diff --git a/src/responder/pac/pacsrv_utils.c b/src/responder/pac/pacsrv_utils.c
index 0872e86dc..035fe84c0 100644
--- a/src/responder/pac/pacsrv_utils.c
+++ b/src/responder/pac/pacsrv_utils.c
@@ -87,7 +87,7 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
     char *primary_group_sid_str = NULL;
 
     if (pac_ctx == NULL || logon_info == NULL || _sid_table == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("Missing parameter.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "Missing parameter.\n");
         return EINVAL;
     }
 
@@ -97,7 +97,7 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
                           info3->sidcount + info3->base.groups.count + 2,
                           &sid_table);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sss_hash_create failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sss_hash_create failed.\n");
         goto done;
     }
 
@@ -107,7 +107,7 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
     err = sss_idmap_smb_sid_to_sid(pac_ctx->idmap_ctx, info3->base.domain_sid,
                                    &user_dom_sid_str);
     if (err != IDMAP_SUCCESS) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sss_idmap_smb_sid_to_sid failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sss_idmap_smb_sid_to_sid failed.\n");
         ret = EFAULT;
         goto done;
     }
@@ -115,7 +115,7 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
     ret = responder_get_domain_by_id(pac_ctx->rctx, user_dom_sid_str,
                                      &user_dom);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("responder_get_domain_by_id failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "responder_get_domain_by_id failed.\n");
         ret = EINVAL;
         goto done;
     }
@@ -123,7 +123,7 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
     user_dom_sid_str_len = strlen(user_dom_sid_str);
     sid_str = talloc_zero_size(mem_ctx, user_dom_sid_str_len + 12);
     if (sid_str == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_zero_size failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_zero_size failed.\n");
         ret = ENOMEM;
         goto done;
     }
@@ -135,14 +135,14 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
     ret = snprintf(rid_start, 12, "-%lu",
                                   (unsigned long) info3->base.rid);
     if (ret < 0 || ret > 12) {
-        DEBUG(SSSDBG_OP_FAILURE, ("snprintf failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "snprintf failed.\n");
         ret = EIO;
         goto done;
     }
 
     user_sid_str = talloc_strdup(mem_ctx, sid_str);
     if (user_sid_str == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_strdup failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_strdup failed.\n");
         ret = ENOMEM;
         goto done;
     }
@@ -158,8 +158,8 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
 
     ret = hash_enter(sid_table, &key, &value);
     if (ret != HASH_SUCCESS) {
-        DEBUG(SSSDBG_OP_FAILURE, ("hash_enter failed [%d][%s].\n",
-                                  ret, hash_error_string(ret)));
+        DEBUG(SSSDBG_OP_FAILURE, "hash_enter failed [%d][%s].\n",
+                                  ret, hash_error_string(ret));
         ret = EIO;
         goto done;
     }
@@ -169,14 +169,14 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
     ret = snprintf(rid_start, 12, "-%lu",
                                   (unsigned long) info3->base.primary_gid);
     if (ret < 0 || ret > 12) {
-        DEBUG(SSSDBG_OP_FAILURE, ("snprintf failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "snprintf failed.\n");
         ret = EIO;
         goto done;
     }
 
     primary_group_sid_str = talloc_strdup(mem_ctx, sid_str);
     if (primary_group_sid_str == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_strdup failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_strdup failed.\n");
         ret = ENOMEM;
         goto done;
     }
@@ -192,8 +192,8 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
 
     ret = hash_enter(sid_table, &key, &value);
     if (ret != HASH_SUCCESS) {
-        DEBUG(SSSDBG_OP_FAILURE, ("hash_enter failed [%d][%s].\n",
-                                  ret, hash_error_string(ret)));
+        DEBUG(SSSDBG_OP_FAILURE, "hash_enter failed [%d][%s].\n",
+                                  ret, hash_error_string(ret));
         ret = EIO;
         goto done;
     }
@@ -204,7 +204,7 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
         ret = snprintf(rid_start, 12, "-%lu",
                                 (unsigned long) info3->base.groups.rids[s].rid);
         if (ret < 0 || ret > 12) {
-            DEBUG(SSSDBG_OP_FAILURE, ("snprintf failed.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "snprintf failed.\n");
             ret = EIO;
             goto done;
         }
@@ -222,8 +222,8 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
 
         ret = hash_enter(sid_table, &key, &value);
         if (ret != HASH_SUCCESS) {
-            DEBUG(SSSDBG_OP_FAILURE, ("hash_enter failed [%d][%s].\n",
-                                      ret, hash_error_string(ret)));
+            DEBUG(SSSDBG_OP_FAILURE, "hash_enter failed [%d][%s].\n",
+                                      ret, hash_error_string(ret));
             ret = EIO;
             goto done;
         }
@@ -234,7 +234,7 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
         err = sss_idmap_smb_sid_to_sid(pac_ctx->idmap_ctx, info3->sids[s].sid,
                                        &msid_str);
         if (err != IDMAP_SUCCESS) {
-            DEBUG(SSSDBG_OP_FAILURE, ("sss_idmap_smb_sid_to_sid failed.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "sss_idmap_smb_sid_to_sid failed.\n");
             ret = EFAULT;
             goto done;
         }
@@ -256,8 +256,8 @@ errno_t get_sids_from_pac(TALLOC_CTX *mem_ctx,
         ret = hash_enter(sid_table, &key, &value);
         sss_idmap_free_sid(pac_ctx->idmap_ctx, msid_str);
         if (ret != HASH_SUCCESS) {
-            DEBUG(SSSDBG_OP_FAILURE, ("hash_enter failed [%d][%s].\n",
-                                      ret, hash_error_string(ret)));
+            DEBUG(SSSDBG_OP_FAILURE, "hash_enter failed [%d][%s].\n",
+                                      ret, hash_error_string(ret));
             ret = EIO;
             goto done;
         }
@@ -301,20 +301,20 @@ errno_t get_data_from_pac(TALLOC_CTX *mem_ctx,
 
     ndr_pull = ndr_pull_init_blob(&blob, mem_ctx);
     if (ndr_pull == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("ndr_pull_init_blob failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "ndr_pull_init_blob failed.\n");
         return ENOMEM;
     }
     ndr_pull->flags |= LIBNDR_FLAG_REF_ALLOC; /* FIXME: is this really needed ? */
 
     pac_data = talloc_zero(mem_ctx, struct PAC_DATA);
     if (pac_data == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_zero failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_zero failed.\n");
         return ENOMEM;
     }
 
     ndr_err = ndr_pull_PAC_DATA(ndr_pull, NDR_SCALARS|NDR_BUFFERS, pac_data);
     if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
-        DEBUG(SSSDBG_OP_FAILURE, ("ndr_pull_PAC_DATA failed [%d]\n", ndr_err));
+        DEBUG(SSSDBG_OP_FAILURE, "ndr_pull_PAC_DATA failed [%d]\n", ndr_err);
         return EBADMSG;
     }
 
@@ -356,19 +356,19 @@ errno_t get_pwd_from_pac(TALLOC_CTX *mem_ctx,
 
     pwd = talloc_zero(mem_ctx, struct passwd);
     if (pwd == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_zero failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_zero failed.\n");
         return ENOMEM;
     }
 
     base_info = &logon_info->info3.base;
 
     if (base_info->account_name.size == 0) {
-        DEBUG(SSSDBG_OP_FAILURE, ("Missing account name in PAC.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "Missing account name in PAC.\n");
         ret = EINVAL;
         goto done;
     }
     if (base_info->rid == 0) {
-        DEBUG(SSSDBG_OP_FAILURE, ("Missing user RID in PAC.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "Missing user RID in PAC.\n");
         ret = EINVAL;
         goto done;
     }
@@ -377,7 +377,7 @@ errno_t get_pwd_from_pac(TALLOC_CTX *mem_ctx,
      * case names only, effectively making the domain case-insenvitive. */
     lname = sss_tc_utf8_str_tolower(pwd, base_info->account_name.string);
     if (lname == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sss_tc_utf8_str_tolower failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sss_tc_utf8_str_tolower failed.\n");
         ret = ENOMEM;
         goto done;
     }
@@ -385,7 +385,7 @@ errno_t get_pwd_from_pac(TALLOC_CTX *mem_ctx,
     /* Subdomain use fully qualified names */
     pwd->pw_name = sss_get_domain_name(pwd, lname, dom);
     if (!pwd->pw_name) {
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_sprintf failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_sprintf failed.\n");
         ret = ENOMEM;
         goto done;
     }
@@ -394,12 +394,12 @@ errno_t get_pwd_from_pac(TALLOC_CTX *mem_ctx,
     key.str = user_sid_str;
     ret = hash_lookup(sid_table, &key, &value);
     if (ret != HASH_SUCCESS) {
-        DEBUG(SSSDBG_OP_FAILURE, ("hash_lookup failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "hash_lookup failed.\n");
         ret = EIO;
         goto done;
     }
     if (value.type != HASH_VALUE_ULONG) {
-        DEBUG(SSSDBG_OP_FAILURE, ("Wrong value type.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "Wrong value type.\n");
         ret = EIO;
         goto done;
     }
@@ -412,12 +412,12 @@ errno_t get_pwd_from_pac(TALLOC_CTX *mem_ctx,
         key.str = primary_group_sid_str;
         ret = hash_lookup(sid_table, &key, &value);
         if (ret != HASH_SUCCESS) {
-            DEBUG(SSSDBG_OP_FAILURE, ("hash_lookup failed.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "hash_lookup failed.\n");
             ret = EIO;
             goto done;
         }
         if (value.type != HASH_VALUE_ULONG) {
-            DEBUG(SSSDBG_OP_FAILURE, ("Wrong value type.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "Wrong value type.\n");
             ret = EIO;
             goto done;
         }
@@ -427,13 +427,13 @@ errno_t get_pwd_from_pac(TALLOC_CTX *mem_ctx,
     if (base_info->full_name.size != 0) {
         pwd->pw_gecos = talloc_strdup(pwd, base_info->full_name.string);
         if (pwd->pw_gecos == NULL) {
-            DEBUG(SSSDBG_OP_FAILURE, ("talloc_strdup failed.\n"));
+            DEBUG(SSSDBG_OP_FAILURE, "talloc_strdup failed.\n");
             ret = ENOMEM;
             goto done;
         }
     } else {
-        DEBUG(SSSDBG_OP_FAILURE, ("Missing full name in PAC, "
-                                  "gecos field will by empty.\n "));
+        DEBUG(SSSDBG_OP_FAILURE, "Missing full name in PAC, "
+                                  "gecos field will by empty.\n ");
     }
 
     /* Check if there is a special homedir template for sub-domains. If not a
@@ -452,14 +452,14 @@ errno_t get_pwd_from_pac(TALLOC_CTX *mem_ctx,
 
     attrs = sysdb_new_attrs(mem_ctx);
     if (attrs == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sysdb_new_attrs failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sysdb_new_attrs failed.\n");
         ret = ENOMEM;
         goto done;
     }
 
     uc_realm = get_uppercase_realm(mem_ctx, dom->name);
     if (uc_realm == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("get_uppercase_realm failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "get_uppercase_realm failed.\n");
         ret = ENOMEM;
         goto done;
     }
@@ -467,7 +467,7 @@ errno_t get_pwd_from_pac(TALLOC_CTX *mem_ctx,
     upn = talloc_asprintf(mem_ctx, "%s@%s", lname, uc_realm);
     talloc_free(uc_realm);
     if (upn == NULL) {
-        DEBUG(SSSDBG_OP_FAILURE, ("talloc_asprintf failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "talloc_asprintf failed.\n");
         ret = ENOMEM;
         goto done;
     }
@@ -475,19 +475,19 @@ errno_t get_pwd_from_pac(TALLOC_CTX *mem_ctx,
     ret = sysdb_attrs_add_string(attrs, SYSDB_UPN, upn);
     talloc_free(upn);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sysdb_attrs_add_string failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sysdb_attrs_add_string failed.\n");
         goto done;
     }
 
     ret = sysdb_attrs_add_lc_name_alias(attrs, pwd->pw_name);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sysdb_attrs_add_lc_name_alias failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sysdb_attrs_add_lc_name_alias failed.\n");
         goto done;
     }
 
     ret = sysdb_attrs_add_string(attrs, SYSDB_SID_STR, user_sid_str);
     if (ret != EOK) {
-        DEBUG(SSSDBG_OP_FAILURE, ("sysdb_attrs_add_string failed.\n"));
+        DEBUG(SSSDBG_OP_FAILURE, "sysdb_attrs_add_string failed.\n");
         goto done;
     }