From 0885d16ac22a0ecd2267bf3c8818a3a8a5a9dadb Mon Sep 17 00:00:00 2001
From: Valery Kharseko <vharseko@3a-systems.ru>
Date: Fri, 31 Jul 2026 07:48:54 +0000
Subject: [PATCH] Fix CodeQL warning-severity alerts: missed wakeups, resource leaks, escaping threads (#790)
---
opendj-server-legacy/src/main/java/org/opends/server/loggers/MultifileTextWriter.java | 16 +++++++++++++++-
1 files changed, 15 insertions(+), 1 deletions(-)
diff --git a/opendj-server-legacy/src/main/java/org/opends/server/loggers/MultifileTextWriter.java b/opendj-server-legacy/src/main/java/org/opends/server/loggers/MultifileTextWriter.java
index 7fb0743..30372b1 100644
--- a/opendj-server-legacy/src/main/java/org/opends/server/loggers/MultifileTextWriter.java
+++ b/opendj-server-legacy/src/main/java/org/opends/server/loggers/MultifileTextWriter.java
@@ -13,6 +13,7 @@
*
* Copyright 2006-2008 Sun Microsystems, Inc.
* Portions Copyright 2014-2016 ForgeRock AS.
+ * Portions Copyright 2026 3A Systems, LLC.
*/
package org.opends.server.loggers;
@@ -134,12 +135,25 @@
this.stopRequested = false;
rotaterThread = new RotaterThread(this);
- rotaterThread.start();
DirectoryServer.registerShutdownListener(this);
}
/**
+ * Starts the background thread which rotates and retains the log files.
+ * <p>
+ * The thread is not started by the constructor so that {@code this} is not published to it before
+ * construction has completed.
+ *
+ * @return this writer
+ */
+ MultifileTextWriter start()
+ {
+ rotaterThread.start();
+ return this;
+ }
+
+ /**
* Construct a PrintWriter for a file.
* @param file - the file to open for writing
* @param filePermissions - the file permissions to set on the file.
--
Gitblit v1.10.0