Skip to content

Fix unsafe access to CodeCache - #175

Merged
jbachorik merged 1 commit into
mainfrom
jb/adustsp_crash_1
Feb 4, 2025
Merged

jbachorik merged 1 commit into
mainfrom
jb/adustsp_crash_1

Conversation

@jbachorik

Copy link
Copy Markdown
Collaborator

What does this PR do?:
See async-profiler/async-profiler@6207d5d

@github-actions

github-actions Bot commented Feb 4, 2025

Copy link
Copy Markdown
Contributor

🔧 Report generated by pr-comment-scanbuild

Scan-Build Report

User:runner@fv-az1689-795
Working Directory:/home/runner/work/java-profiler/java-profiler/ddprof-lib/src/test/make
Command Line:make -j4 clean all
Clang Version:Ubuntu clang version 18.1.3 (1ubuntu1)
Date:Tue Feb 4 14:00:46 2025

Bug Summary

Bug TypeQuantityDisplay?
All Bugs6
Logic error
Dereference of null pointer3
Suspicious operation
Bitwise shift1
Unused code
Dead initialization1
Dead nested assignment1

Reports

Bug Group Bug Type ▾ File Function/Method Line Path Length
Suspicious operationBitwise shiftvmStructs.cppfind87216
Unused codeDead initializationstackWalker.cppwalkDwarf1601
Unused codeDead nested assignmentvmStructs.cppcheckNativeBinding9771
Logic errorDereference of null pointersafeAccess.hload3318
Logic errorDereference of null pointersymbols_linux.hElfParser12928
Logic errorDereference of null pointerflightRecorder.cppflush15138

@github-actions

github-actions Bot commented Feb 4, 2025

Copy link
Copy Markdown
Contributor

🔧 Report generated by pr-comment-cppcheck

CppCheck Report

Warnings (7)

Style Violations (405)

@jbachorik
jbachorik merged commit 3bf0964 into main Feb 4, 2025
@jbachorik
jbachorik deleted the jb/adustsp_crash_1 branch February 4, 2025 14:39
@github-actions github-actions Bot added this to the 1.20.0 milestone Feb 4, 2025
@zhengyu123 zhengyu123 added the fixed_version: v_1.20.0 Version this PR's fix shipped in label Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fixed_version: v_1.20.0 Version this PR's fix shipped in

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants