[release/v1.33.x] Avoid NullPointerException when jms destination is …

…not available (#11577) Co-authored-by: Lauri Tulmin <ltulmin@splunk.com>
open-telemetry · Jun 13, 2024 · 23883df · 23883df
1 parent b6c5376
commit 23883df
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/.../src/main/java/io/opentelemetry/javaagent/instrumentation/jms/MessageWithDestination.java b/.../src/main/java/io/opentelemetry/javaagent/instrumentation/jms/MessageWithDestination.java
@@ -31,11 +31,13 @@ public static MessageWithDestination create(
       jmsDestination = fallbackDestination;
     }
 
-    if (jmsDestination.isQueue()) {
-      return createMessageWithQueue(message, jmsDestination);
-    }
-    if (jmsDestination.isTopic()) {
-      return createMessageWithTopic(message, jmsDestination);
+    if (jmsDestination != null) {
+      if (jmsDestination.isQueue()) {
+        return createMessageWithQueue(message, jmsDestination);
+      }
+      if (jmsDestination.isTopic()) {
+        return createMessageWithTopic(message, jmsDestination);
+      }
     }
     return new AutoValue_MessageWithDestination(
         message, "unknown", /* isTemporaryDestination= */ false);