FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Protect Lookup::resolveMethod() with siglongjmp by zhengyu123 · Pull Request #752 · DataDog/java-profiler · GitHub

Protect Lookup::resolveMethod() with siglongjmp - #752

Draft
zhengyu123 wants to merge 5 commits into
mainfrom
zgu/lookup_resolveMethod
Draft

Protect Lookup::resolveMethod() with siglongjmp#752
zhengyu123 wants to merge 5 commits into
mainfrom
zgu/lookup_resolveMethod

Conversation

Copy link
Copy Markdown
Contributor

What does this PR do?:

Motivation:

Additional Notes:

How to test the change?:

For Datadog employees:

  • If this PR touches code that signs or publishes builds or packages, or handles
    credentials of any kind, I've requested a security review (run the dd:platform-security-review
    skill, or file a request via the PSEC review form).
    bewaire also runs automatically on every PR.
  • This PR doesn't touch any of that.
  • JIRA: [JIRA-XXXX]

Unsure? Have a question? Request a review!

dd-octo-sts Bot commented Aug 24, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Scan-Build Report

User:runner@runnervm76f27
Working Directory:/home/runner/work/java-profiler/java-profiler/ddprof-lib/src/test/make
Command Line:make -j4 all
Clang Version:Ubuntu clang version 18.1.3 (1ubuntu1)
Date:Tue Aug 25 14:36:18 2026

Bug Summary

Bug TypeQuantityDisplay?
All Bugs1
Logic error
Dereference of null pointer1

Reports

Bug Group Bug Type ▾ File Function/Method Line Path Length
Logic errorDereference of null pointerfaultInjection.cppcrashNow222

dd-octo-sts Bot commented Aug 24, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

CI Test Results

Run: #32860344818 | Commit: a40ef50 | Duration: 14m 35s (longest job)

All 32 test jobs passed

Status Overview

JDK glibc-aarch64/debug glibc-amd64/debug musl-aarch64/debug musl-amd64/debug
8 - - -
8-ibm - - -
8-j9 - -
8-librca - -
8-orcl - - -
11 - - -
11-j9 - -
11-librca - -
17 - -
17-graal - -
17-j9 - -
17-librca - -
21 - -
21-graal - -
21-librca - -
25 - -
25-graal - -
25-librca - -

Legend: ✅ passed | ❌ failed | ⚪ skipped | 🚫 cancelled

Summary: Total: 32 | Passed: 32 | Failed: 0


Updated: 2026-08-25 14:54:57 UTC

dd-octo-sts Bot commented Aug 24, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

All 40 integration tests passed

📊 Dashboard · 👷 Pipeline · 📦 80e4cd7a

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL