]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
just rely on fr_pair_print_value_quoted()
authorAlan T. DeKok <aland@freeradius.org>
Fri, 16 Jan 2026 00:15:02 +0000 (19:15 -0500)
committerAlan T. DeKok <aland@freeradius.org>
Fri, 16 Jan 2026 00:15:02 +0000 (19:15 -0500)
src/lib/server/log.c

index d011ea8ab02a2e6c8a13dd7054e0eae9823af013..cb251b62f6378419c70766c3798eb2652701c5bf 100644 (file)
@@ -780,7 +780,6 @@ void log_request_pair(fr_log_lvl_t lvl, request_t *request,
                      fr_pair_t const *parent, fr_pair_t const *vp, char const *prefix)
 {
        fr_sbuff_t              *oid_buff;
-       char const              *name;
 
        if (!request->log.dst) return;
 
@@ -802,28 +801,11 @@ void log_request_pair(fr_log_lvl_t lvl, request_t *request,
                RDEBUGX(lvl, "}");
                break;
 
-       case FR_TYPE_QUOTED:
-               RDEBUGX(lvl, "%s%s\"%pV\"", prefix ? prefix : "", fr_sbuff_start(oid_buff),
-                       &vp->data);
-               break;
-
        default:
                fr_assert(fr_type_is_leaf(vp->vp_type));
+               if (fr_pair_print_value_quoted(oid_buff, vp, T_DOUBLE_QUOTED_STRING) <= 0) return;
 
-               /*
-                *      Manually add enum prefix when printing.
-                */
-               if ((name = fr_value_box_enum_name(&vp->data)) != NULL) {
-                       RDEBUGX(lvl, "%s::%s", prefix ? prefix : "", fr_sbuff_start(oid_buff),
-                               name);
-                       break;
-               }
-               FALL_THROUGH;
-
-       case FR_TYPE_INTERNAL:
-               RDEBUGX(lvl, "%s%pV = %pV", prefix ? prefix : "",
-                       fr_box_strvalue_len(fr_sbuff_start(oid_buff), fr_sbuff_used(oid_buff)),
-                       &vp->data);
+               RDEBUGX(lvl, "%s%s", prefix ? prefix : "", fr_sbuff_start(oid_buff));
                break;
        }
 }