don't smash vp->op to T_OP_EQ

the rest of the code in the server should take care of either
not setting vp->op, or setting it to T_OP_EQ after using the
operator.

diff --git a/src/modules/rlm_detail/rlm_detail.c b/src/modules/rlm_detail/rlm_detail.c
index 3ded8569e80445ddeeeef6dd4f24183c58ed1b31..b05756f1f835f21e20f048ce27e7b92363803ed9 100644 (file)
--- a/src/modules/rlm_detail/rlm_detail.c
+++ b/src/modules/rlm_detail/rlm_detail.c
@@ -322,8 +322,6 @@ static int detail_write(FILE *out, rlm_detail_t const *inst, request_t *request,
                for (vp = fr_pair_list_head(list);
                     vp;
                     vp = fr_pair_list_next(list, vp)) {
-                       fr_token_t op;
-
                        if (inst->ht && fr_hash_table_find(inst->ht, vp->da)) continue;
 
                        /*
@@ -331,13 +329,7 @@ static int detail_write(FILE *out, rlm_detail_t const *inst, request_t *request,
                         */
                        if (compat && (vp->da == attr_user_password)) continue;
 
-                       /*
-                        *      Print all of the attributes, operator should always be '='.
-                        */
-                       op = vp->op;
-                       vp->op = T_OP_EQ;
                        fr_pair_fprint(out, vp);
-                       vp->op = op;
                }
        }