When we have a stop record, don't compare it to unused entries.

	This means that if we get two duplicate stops, the second one will
	cause the server to complain.  Previously, the server *may* have
	complained, but not necessarily...

diff --git a/src/modules/rlm_radutmp/rlm_radutmp.c b/src/modules/rlm_radutmp/rlm_radutmp.c
index e4be7c7..a81ec10 100644 (file)
--- a/src/modules/rlm_radutmp/rlm_radutmp.c
+++ b/src/modules/rlm_radutmp/rlm_radutmp.c
@@ -458,6 +458,14 @@ static int radutmp_accounting(void *instance, REQUEST *request)
                    u.nas_port    != ut.nas_port)
                        continue;
                
+               /*
+                *      Don't compare stop records to unused entries.
+                */
+               if (status == PW_STATUS_STOP &&
+                   u.type == P_IDLE) {
+                       continue;
+               }
+
                if (status == PW_STATUS_STOP &&
                    strncmp(ut.session_id, u.session_id,
                            sizeof(u.session_id)) != 0) {