I've modified rlm_detail to check its writes and return RLM_MODULE_FAIL on short writes. I believe this should cause FreeRADIUS to avoid sending an Accounting-Response to the NAS, so the NAS will retry on disk-full/error situations. A while ago, I noticed rlm_detail writes were unchecked[1], causing FreeRADIUS to exit on an assertion in the detail reader due to the partial detail entries. Alan kindly fixed the detail reader's behavior in this situation[2], and I also wanted to have the detail writer check its writes. Truncating the detail file on error to remove the partial detail entry isn't strictly necessary, but it seems like good form and is straightforward. I haven't yet gotten rid of the vp_print() calls (its write is unchecked, too) and this patch is untested (though it compiles), but I wanted to get feedback and see if it's worthwhile for me to finish and test it. john [1] https://lists.freeradius.org/pipermail/freeradius-devel/2010-May/013826.html [2] http://github.com/alandekok/freeradius-server/commit/73fe4311083eb684b3d7f85... -- John Morrissey _o /\ ---- __o jwm@horde.net _-< \_ / \ ---- < \, www.horde.net/ __(_)/_(_)________/ \_______(_) /_(_)__