Be a bit more careful about thread transitions

FreeRADIUS · Sep 16, 2015 · c83c3ae · c83c3ae
1 parent f3cec30
commit c83c3ae
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/src/main/process.c b/src/main/process.c
@@ -786,16 +786,20 @@ static void request_cleanup_delay_init(REQUEST *request)
 		if (rad_debug_lvl) printf("(%u) ********\tNEXT-STATE %s -> %s\n", request->number, __FUNCTION__, "request_cleanup_delay");
 #endif
 		request->process = request_cleanup_delay;
-		request->child_state = REQUEST_CLEANUP_DELAY;
+
+		if (!we_are_master()) {
+			NO_CHILD_THREAD;
+			request->child_state = REQUEST_CLEANUP_DELAY;
+		}
 
 		/*
 		 *	Update this if we can, otherwise let the timers pick it up.
 		 */
-		if (we_are_master()) {
-			STATE_MACHINE_TIMER(FR_ACTION_TIMER);
-		} else {
-			NO_CHILD_THREAD;
-		}
+		request->child_state = REQUEST_CLEANUP_DELAY;
+#ifdef HAVE_PTHREAD_H
+		rad_assert(request->child_pid == NO_SUCH_CHILD_PID);
+#endif
+		STATE_MACHINE_TIMER(FR_ACTION_TIMER);
 		return;
 	}