Merge pull request #455 from dmlloyd/logmgr-334

[LOGMGR-344] Fix signature of `invokeExact` sigpoly call
jboss-logging · Dec 20, 2023 · b2a00b5 · b2a00b5
2 parents df2b5e3 + 02ddd2d
commit b2a00b5
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/src/main/java/org/jboss/logmanager/ExtLogRecord.java b/src/main/java/org/jboss/logmanager/ExtLogRecord.java
@@ -689,7 +689,7 @@ public long getLongThreadID() {
     void superSetLongThreadID(final long id) {
         if (superSetLongThreadID != null) {
             try {
-                superSetLongThreadID.invokeExact(this, id);
+                LogRecord ignored = (LogRecord) superSetLongThreadID.invokeExact(this, id);
             } catch (RuntimeException | Error e) {
                 throw e;
             } catch (Throwable e) {

diff --git a/src/test/java/org/jboss/logmanager/ExtLogRecordTests.java b/src/test/java/org/jboss/logmanager/ExtLogRecordTests.java
@@ -0,0 +1,16 @@
+package org.jboss.logmanager;
+
+import org.junit.jupiter.api.Test;
+
+/**
+ *
+ */
+public final class ExtLogRecordTests {
+
+    @Test
+    public void checkForLongThreadIdRegression() {
+        ExtLogRecord rec = new ExtLogRecord(Level.INFO, "Hello world!", ExtLogRecordTests.class.getName());
+        // expect this to not blow up on 11 or 17
+        rec.setLongThreadID(1234);
+    }
+}