mcnewton/freeradius-server — comment on commit ad07620e9fd97c38b78527c2dd38794b1b1da454
File: src/modules/rlm_detail/rlm_detail.c line 554
By mcnewton:

This isn't quite so simple, unfortunately. do_detail requires a packet, not just a pair_lists_t (it can use packet->code, or packet->src_ipaddr, etc.) radius_list() returns a pair list, not the parent packet.

It looks like there still needs to be a switch (pair_list), a bit like radius_list, but returning a packet rather than a pair_lists_t (which will obviously, for some values of pair_list, be NULL as there isn't a packet).

Unless there is already a function that does this? I can't find one. I'll see if there is a tidy way to do it.