fix: siginfo now handle other informations

Author: bric3

dd-java-agent/agent-crashtracking/src/main/java/datadog/crashtracking/CrashUploader.java

@@ -544,6 +544,16 @@ private RequestBody makeErrorTrackingRequestBody(@Nonnull CrashLog payload, bool
             writer.name("si_signo_human_readable").value(payload.sigInfo.name);
             writer.name("si_signo").value(payload.sigInfo.number);
           }
+          if (payload.sigInfo.action != null) {
+            writer.name("si_code").value(payload.sigInfo.code);
+            writer.name("si_code_human_readable").value(payload.sigInfo.action);
+          }
+          if (payload.sigInfo.pid != null) {
+            writer.name("si_pid").value(payload.sigInfo.pid);
+          }
+          if (payload.sigInfo.uid != null) {
+            writer.name("si_uid").value(payload.sigInfo.uid);
+          }
           writer.endObject();
         }
 

dd-java-agent/agent-crashtracking/src/main/java/datadog/crashtracking/dto/SigInfo.java

@@ -19,12 +19,27 @@ public class SigInfo {
   @Json(name = "si_addr")
   public final String address;
 
-  public SigInfo(Integer number, String name, Integer code, String action, String address) {
+  @Json(name = "si_pid")
+  public final Integer pid;
+
+  @Json(name = "si_uid")
+  public final Integer uid;
+
+  public SigInfo(
+      Integer number,
+      String name,
+      Integer code,
+      String action,
+      String address,
+      Integer pid,
+      Integer uid) {
     this.number = number;
     this.name = name;
     this.address = address;
     this.code = code;
     this.action = action;
+    this.pid = pid;
+    this.uid = uid;
   }
 
   @Override
@@ -35,11 +50,13 @@ public boolean equals(Object o) {
         && Objects.equals(name, sigInfo.name)
         && Objects.equals(address, sigInfo.address)
         && Objects.equals(code, sigInfo.code)
-        && Objects.equals(action, sigInfo.action);
+        && Objects.equals(action, sigInfo.action)
+        && Objects.equals(pid, sigInfo.pid)
+        && Objects.equals(uid, sigInfo.uid);
   }
 
   @Override
   public int hashCode() {
-    return Objects.hash(number, name, address, code, action);
+    return Objects.hash(number, name, address, code, action, pid, uid);
   }
 }

dd-java-agent/agent-crashtracking/src/main/java/datadog/crashtracking/parsers/HotspotCrashLogParser.java

@@ -63,6 +63,9 @@ public HotspotCrashLogParser() {
   private static final Pattern SIGINFO_PARSER =
       Pattern.compile(
           "siginfo:\\s+si_signo:\\s+(\\d+)\\s+\\((\\w+)\\),\\s+si_code:\\s+(\\d+)\\s+\\(([^)]+)\\),\\s+si_addr:\\s+(0x[0-9a-fA-F]+)");
+  private static final Pattern SIGINFO_SI_USER_PARSER =
+      Pattern.compile(
+          "siginfo:\\s+si_signo:\\s+(\\d+)\\s+\\((\\w+)\\),\\s+si_code:\\s+(\\d+)\\s+\\(([^)]+)\\),\\s+si_pid:\\s+(\\d+),\\s+si_uid:\\s+(\\d+)");
   private static final Pattern DYNAMIC_LIBS_PATH_PARSER =
       Pattern.compile("^(?:0x)?[0-9a-fA-F]+(?:-[0-9a-fA-F]+)?\\s+(?:[^\\s/\\[]+\\s+)*(.*)$");
 
@@ -284,7 +287,20 @@ public CrashLog parse(String uuid, String crashLog) {
               Integer siCode = safelyParseInt(siginfoMatcher.group(3));
               String sigAction = siginfoMatcher.group(4);
               String address = siginfoMatcher.group(5);
-              sigInfo = new SigInfo(number, name, siCode, sigAction, address);
+              sigInfo = new SigInfo(number, name, siCode, sigAction, address, null, null);
+            } else {
+              // siginfo: si_signo: 11 (SIGSEGV), si_code: 0 (SI_USER), si_pid: 554848, si_uid:
+              // 1000
+              final Matcher siUserMatcher = SIGINFO_SI_USER_PARSER.matcher(line);
+              if (siUserMatcher.matches()) {
+                Integer number = safelyParseInt(siUserMatcher.group(1));
+                String name = siUserMatcher.group(2);
+                Integer siCode = safelyParseInt(siUserMatcher.group(3));
+                String sigAction = siUserMatcher.group(4);
+                Integer siPid = safelyParseInt(siUserMatcher.group(5));
+                Integer siUid = safelyParseInt(siUserMatcher.group(6));
+                sigInfo = new SigInfo(number, name, siCode, sigAction, null, siPid, siUid);
+              }
             }
           } else if (line.contains("P R O C E S S")) {
             state = State.SEEK_DYNAMIC_LIBRARIES;

dd-java-agent/agent-crashtracking/src/main/java/datadog/crashtracking/parsers/J9JavacoreParser.java

@@ -330,7 +330,7 @@ private SigInfo buildSigInfo(String eventType, String eventCode) {
         signalNumber = parseEventCode(eventCode);
     }
 
-    return new SigInfo(signalNumber, signalName, null, null, null);
+    return new SigInfo(signalNumber, signalName, null, null, null, null, null);
   }
 
   private int parseEventCode(String eventCode) {

dd-java-agent/agent-crashtracking/src/test/resources/golden/errortracking/sample-crash-for-telemetry-2.txt

@@ -1 +1 @@
-{"timestamp":"2024-09-20T13:19:06Z","ddsource":"crashtracker","error":{"is_crash":true,"type":"UNKNOWN","message":"Process terminated by signal UNKNOWN","source_type":"Crashtracking","stack":{"format":"CrashTrackerV1","frames":[{"function":"__pthread_clockjoin_ex+0x255","path":"libpthread.so.0","relative_address":"0x9cd5"}]}}}
+{"timestamp":"2024-09-20T13:19:06Z","ddsource":"crashtracker","error":{"is_crash":true,"type":"SIGSEGV","message":"Process terminated by signal SIGSEGV","source_type":"Crashtracking","stack":{"format":"CrashTrackerV1","frames":[{"function":"__pthread_clockjoin_ex+0x255","path":"libpthread.so.0","relative_address":"0x9cd5"}]}},"sig_info":{"si_signo_human_readable":"SIGSEGV","si_signo":11,"si_code":0,"si_code_human_readable":"SI_USER","si_pid":554848,"si_uid":1000}}

dd-java-agent/agent-crashtracking/src/test/resources/golden/errortracking/sample-crash-for-telemetry.txt

@@ -1 +1 @@
- {"data_schema_version":"1.0","error":{"is_crash":true,"kind":"UNKNOWN","message":"Process terminated by signal UNKNOWN","source_type":"Crashtracking","stack":{"format":"CrashTrackerV1","frames":[{"function":"__pthread_clockjoin_ex+0x255","path":"libpthread.so.0","relative_address":"0x9cd5"}]}},"incomplete":false,"metadata":{"family":"java","library_name":"dd-trace-java","library_version":"1.60.0-SNAPSHOT~cffe9c6085"},"os_info":{"architecture":"aarch64","bitness":"64","os_type":"Mac OS X","version":{"Semantic":[15,7,1]}},"proc_info":{"pid":576034},"timestamp":"2024-09-20T13:19:06Z","uuid":"a4194cd6-8cb3-45fd-9bd9-3af83e0a3ad3","version_id":0}
+{"data_schema_version":"1.0","error":{"is_crash":true,"kind":"SIGSEGV","message":"Process terminated by signal SIGSEGV","source_type":"Crashtracking","stack":{"format":"CrashTrackerV1","frames":[{"function":"__pthread_clockjoin_ex+0x255","path":"libpthread.so.0","relative_address":"0x9cd5"}]}},"incomplete":false,"metadata":{"family":"java","library_name":"dd-trace-java","library_version":"1.60.0-SNAPSHOT~cffe9c6085"},"os_info":{"architecture":"aarch64","bitness":"64","os_type":"Mac OS X","version":{"Semantic":[15,7,1]}},"proc_info":{"pid":576034},"sig_info":{"si_code":0,"si_code_human_readable":"SI_USER","si_pid":554848,"si_signo":11,"si_signo_human_readable":"SIGSEGV","si_uid":1000},"timestamp":"2024-09-20T13:19:06Z","uuid":"a4194cd6-8cb3-45fd-9bd9-3af83e0a3ad3","version_id":0}