From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Mon, 5 Apr 2021 21:51:48 +0000 (+0100)
Subject: Actually need a worker pointer...
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=01f1714f4c67a67116c9b86f10a6166a8f5be67e;p=thirdparty%2Ffreeradius-server.git

Actually need a worker pointer...
---

diff --git a/src/lib/io/worker.c b/src/lib/io/worker.c
index d927307bcd..9356b5bf17 100644
--- a/src/lib/io/worker.c
+++ b/src/lib/io/worker.c
@@ -996,8 +996,10 @@ static void _worker_request_done_internal(request_t *request, UNUSED rlm_rcode_t
  * As the request has no parent, then there's nothing to free it
  * so we have to.
  */
-static void _worker_request_done_detached(request_t *request, UNUSED rlm_rcode_t rcode, UNUSED void *uctx)
+static void _worker_request_done_detached(request_t *request, UNUSED rlm_rcode_t rcode, void *uctx)
 {
+	fr_worker_t	*worker = talloc_get_type_abort(uctx, fr_worker_t);
+
 	/*
 	 *	No time tracking for detached requests
 	 *	so we don't need to call
@@ -1011,7 +1013,7 @@ static void _worker_request_done_detached(request_t *request, UNUSED rlm_rcode_t
 	 *	order heap, but we need to do that for
 	 *	detached requests.
 	 */
-	if (fr_heap_entry_inserted(request->time_order_id)) (void) fr_heap_extract(worker->time_order, request);
+	(void)fr_heap_extract(worker->time_order, request);
 
 	/*
 	 *	Detached requests have to be freed by us