From 504a43fc479d884085df9895900608dc5b0aca6f Mon Sep 17 00:00:00 2001
From: Valery Kharseko <vharseko@3a-systems.ru>
Date: Fri, 31 Jul 2026 07:52:35 +0000
Subject: [PATCH] Fix CodeQL warning-severity alerts: dead checks, boxed locals and three null dereferences (#793)
---
opendj-server-legacy/src/main/java/org/opends/server/loggers/JDKLogging.java | 9 ++++++---
1 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/opendj-server-legacy/src/main/java/org/opends/server/loggers/JDKLogging.java b/opendj-server-legacy/src/main/java/org/opends/server/loggers/JDKLogging.java
index 5fbb17a..51f0986 100644
--- a/opendj-server-legacy/src/main/java/org/opends/server/loggers/JDKLogging.java
+++ b/opendj-server-legacy/src/main/java/org/opends/server/loggers/JDKLogging.java
@@ -12,6 +12,7 @@
* information: "Portions Copyright [year] [name of copyright owner]".
*
* Copyright 2014-2016 ForgeRock AS.
+ * Portions Copyright 2026 3A Systems, LLC.
*/
package org.opends.server.loggers;
@@ -112,14 +113,16 @@
@Override
public void publish(LogRecord record)
{
- if (getFormatter() == null)
+ // These calls resolve to Handler's own formatter, not to the enclosing
+ // JDKLogging.getFormatter(); qualify them so that is unambiguous.
+ if (super.getFormatter() == null)
{
- setFormatter(new SimpleFormatter());
+ super.setFormatter(new SimpleFormatter());
}
try
{
- String message = getFormatter().format(record);
+ String message = super.getFormatter().format(record);
if (record.getLevel().intValue() >= Level.WARNING.intValue())
{
err.write(message.getBytes());
--
Gitblit v1.10.0