Packet-* -> Net.* in rlm_detail

diff --git a/src/listen/detail/proto_detail.c b/src/listen/detail/proto_detail.c
index 23f688df217f5cbe9bef3205beb13d0694ed56d2..15d7f1370356e5424e0e7c9b410020b6feb2a97d 100644 (file)
--- a/src/listen/detail/proto_detail.c
+++ b/src/listen/detail/proto_detail.c
@@ -81,23 +81,19 @@ fr_dict_autoload_t proto_detail_dict[] = {
 };
 
 static fr_dict_attr_t const *attr_packet_dst_ip_address;
-static fr_dict_attr_t const *attr_packet_dst_ipv6_address;
 static fr_dict_attr_t const *attr_packet_dst_port;
 static fr_dict_attr_t const *attr_packet_original_timestamp;
 static fr_dict_attr_t const *attr_packet_src_ip_address;
-static fr_dict_attr_t const *attr_packet_src_ipv6_address;
 static fr_dict_attr_t const *attr_packet_src_port;
 static fr_dict_attr_t const *attr_protocol;
 
 extern fr_dict_attr_autoload_t proto_detail_dict_attr[];
 fr_dict_attr_autoload_t proto_detail_dict_attr[] = {
-       { .out = &attr_packet_dst_ip_address, .name = "Packet-Dst-IP-Address", .type = FR_TYPE_IPV4_ADDR, .dict = &dict_freeradius },
-       { .out = &attr_packet_dst_ipv6_address, .name = "Packet-Dst-IPv6-Address", .type = FR_TYPE_IPV6_ADDR, .dict = &dict_freeradius },
-       { .out = &attr_packet_dst_port, .name = "Packet-Dst-Port", .type = FR_TYPE_UINT16, .dict = &dict_freeradius },
+       { .out = &attr_packet_dst_ip_address, .name = "Net.Dst.IP", .type = FR_TYPE_COMBO_IP_ADDR, .dict = &dict_freeradius },
+       { .out = &attr_packet_dst_port, .name = "Net.Dst.Port", .type = FR_TYPE_UINT16, .dict = &dict_freeradius },
        { .out = &attr_packet_original_timestamp, .name = "Packet-Original-Timestamp", .type = FR_TYPE_DATE, .dict = &dict_freeradius },
-       { .out = &attr_packet_src_ip_address, .name = "Packet-Src-IP-Address", .type = FR_TYPE_IPV4_ADDR, .dict = &dict_freeradius },
-       { .out = &attr_packet_src_ipv6_address, .name = "Packet-Src-IPv6-Address", .type = FR_TYPE_IPV6_ADDR, .dict = &dict_freeradius },
-       { .out = &attr_packet_src_port, .name = "Packet-Src-Port", .type = FR_TYPE_UINT16, .dict = &dict_freeradius },
+       { .out = &attr_packet_src_ip_address, .name = "Net.Src.IP", .type = FR_TYPE_COMBO_IP_ADDR, .dict = &dict_freeradius },
+       { .out = &attr_packet_src_port, .name = "Net.Src.Port", .type = FR_TYPE_UINT16, .dict = &dict_freeradius },
        { .out = &attr_protocol, .name = "Protocol", .type = FR_TYPE_UINT32, .dict = &dict_freeradius },
 
        { NULL }
@@ -315,11 +311,9 @@ static int mod_decode(void const *instance, request_t *request, uint8_t *const d
                 *      Set the original src/dst ip/port
                 */
                if (vp) {
-                       if ((vp->da == attr_packet_src_ip_address) ||
-                           (vp->da == attr_packet_src_ipv6_address)) {
+                       if (vp->da == attr_packet_src_ip_address) {
                                request->packet->socket.inet.src_ipaddr = vp->vp_ip;
-                       } else if ((vp->da == attr_packet_dst_ip_address) ||
-                                  (vp->da == attr_packet_dst_ipv6_address)) {
+                       } else if (vp->da == attr_packet_dst_ip_address) {
                                request->packet->socket.inet.dst_ipaddr = vp->vp_ip;
                        } else if (vp->da == attr_packet_src_port) {
                                request->packet->socket.inet.src_port = vp->vp_uint16;

diff --git a/src/modules/rlm_detail/rlm_detail.c b/src/modules/rlm_detail/rlm_detail.c
index 0490b485527522c60c41a3a520984f3d8a6398eb..c9de19aeecf6e74b0c3bc8d301064f534b132233 100644 (file)
--- a/src/modules/rlm_detail/rlm_detail.c
+++ b/src/modules/rlm_detail/rlm_detail.c
@@ -95,10 +95,8 @@ fr_dict_autoload_t rlm_detail_dict[] = {
        { NULL }
 };
 
-static fr_dict_attr_t const *attr_packet_src_ipv4_address;
-static fr_dict_attr_t const *attr_packet_dst_ipv4_address;
-static fr_dict_attr_t const *attr_packet_src_ipv6_address;
-static fr_dict_attr_t const *attr_packet_dst_ipv6_address;
+static fr_dict_attr_t const *attr_packet_src_address;
+static fr_dict_attr_t const *attr_packet_dst_address;
 static fr_dict_attr_t const *attr_packet_src_port;
 static fr_dict_attr_t const *attr_packet_dst_port;
 static fr_dict_attr_t const *attr_protocol;
@@ -107,12 +105,10 @@ static fr_dict_attr_t const *attr_user_password;
 
 extern fr_dict_attr_autoload_t rlm_detail_dict_attr[];
 fr_dict_attr_autoload_t rlm_detail_dict_attr[] = {
-       { .out = &attr_packet_dst_ipv4_address, .name = "Packet-Dst-IP-Address", .type = FR_TYPE_IPV4_ADDR, .dict = &dict_freeradius },
-       { .out = &attr_packet_dst_ipv6_address, .name = "Packet-Dst-IPv6-Address", .type = FR_TYPE_IPV6_ADDR, .dict = &dict_freeradius },
-       { .out = &attr_packet_dst_port, .name = "Packet-Dst-Port", .type = FR_TYPE_UINT16, .dict = &dict_freeradius },
-       { .out = &attr_packet_src_ipv4_address, .name = "Packet-Src-IP-Address", .type = FR_TYPE_IPV4_ADDR, .dict = &dict_freeradius },
-       { .out = &attr_packet_src_ipv6_address, .name = "Packet-Src-IPv6-Address", .type = FR_TYPE_IPV6_ADDR, .dict = &dict_freeradius },
-       { .out = &attr_packet_src_port, .name = "Packet-Src-Port", .type = FR_TYPE_UINT16, .dict = &dict_freeradius },
+       { .out = &attr_packet_dst_address, .name = "Net.Dst.IP", .type = FR_TYPE_COMBO_IP_ADDR, .dict = &dict_freeradius },
+       { .out = &attr_packet_dst_port, .name = "Net.Dst.Port", .type = FR_TYPE_UINT16, .dict = &dict_freeradius },
+       { .out = &attr_packet_src_address, .name = "Net.Src.IP", .type = FR_TYPE_COMBO_IP_ADDR, .dict = &dict_freeradius },
+       { .out = &attr_packet_src_port, .name = "Net.Src.Port", .type = FR_TYPE_UINT16, .dict = &dict_freeradius },
        { .out = &attr_protocol, .name = "Protocol", .type = FR_TYPE_UINT32, .dict = &dict_freeradius },
 
        { .out = &attr_user_password, .name = "User-Password", .type = FR_TYPE_STRING, .dict = &dict_radius },
@@ -308,43 +304,19 @@ static int detail_write(FILE *out, rlm_detail_t const *inst, request_t *request,
        }
 
        if (inst->log_srcdst) {
-               fr_pair_t src_vp, dst_vp;
+               fr_pair_t *src_vp, *dst_vp;
 
-               fr_pair_init_null(&src_vp);
-               fr_pair_init_null(&dst_vp);
+               src_vp = fr_pair_find_by_da_nested(&request->control_pairs, NULL, attr_packet_src_address);
+               dst_vp = fr_pair_find_by_da_nested(&request->control_pairs, NULL, attr_packet_dst_address);
 
-               switch (packet->socket.inet.src_ipaddr.af) {
-               case AF_INET:
-                       fr_pair_reinit_from_da(NULL, &src_vp, attr_packet_src_ipv4_address);
-                       fr_value_box(&src_vp.data, &packet->socket.inet.src_ipaddr, true);
+               detail_fr_pair_fprint(request, out, src_vp);
+               detail_fr_pair_fprint(request, out, dst_vp);
 
-                       fr_pair_reinit_from_da(NULL, &dst_vp, attr_packet_dst_ipv4_address);
-                       fr_value_box(&dst_vp.data, &packet->socket.inet.dst_ipaddr, true);
-                       break;
+               src_vp = fr_pair_find_by_da_nested(&request->control_pairs, NULL, attr_packet_src_port);
+               dst_vp = fr_pair_find_by_da_nested(&request->control_pairs, NULL, attr_packet_dst_port);
 
-               case AF_INET6:
-                       fr_pair_reinit_from_da(NULL, &src_vp, attr_packet_src_ipv6_address);
-                       fr_value_box(&src_vp.data, &packet->socket.inet.src_ipaddr, true);
-
-                       fr_pair_reinit_from_da(NULL, &dst_vp, attr_packet_dst_ipv6_address);
-                       fr_value_box(&dst_vp.data, &packet->socket.inet.dst_ipaddr, true);
-                       break;
-
-               default:
-                       break;
-               }
-
-               detail_fr_pair_fprint(request, out, &src_vp);
-               detail_fr_pair_fprint(request, out, &dst_vp);
-
-               fr_pair_reinit_from_da(NULL, &src_vp, attr_packet_src_port);
-               fr_value_box(&src_vp.data, packet->socket.inet.src_port, true);
-
-               fr_pair_reinit_from_da(NULL, &dst_vp, attr_packet_dst_port);
-               fr_value_box(&dst_vp.data, packet->socket.inet.dst_port, true);
-
-               detail_fr_pair_fprint(request, out, &src_vp);
-               detail_fr_pair_fprint(request, out, &dst_vp);
+               detail_fr_pair_fprint(request, out, src_vp);
+               detail_fr_pair_fprint(request, out, dst_vp);
        }
 
        /* Write each attribute/value to the log file */