bug(okhttp): Fatal SIGSEGV on Android 14+ JIT due to R8 optimization mismatch with OkHttp bytecode manipulation
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 48/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Lĩnh vực
- build-system, mobile
Hướng nghiên cứu
Start by reviewing sentry-android-core/proguard-rules.pro around lines 46-47 and the ASM logic in ResponseWithInterceptorChainMethodVisitor.kt, then reproduce the release build with R8 Full Mode on Android 14+. Trace whether the obsolete package rules leave io.sentry.okhttp and the manipulated RealCall path unprotected; done means the crash no longer occurs while HTTP instrumentation still works.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Integration
sentry-okhttp
Build System
Gradle
AGP Version
8.x
Proguard
Enabled
Other Error Monitoring Solution
No
Version
8.53.0
Steps to Reproduce
- Enable Sentry HTTP Auto-Instrumentation in an Android project using the Sentry Android Gradle Plugin:
sentry { tracingInstrumentation { enabled = true features = [sentryOkHttpFeature] as Set } } - Compile the application in Release mode with R8 Full Mode enabled:
android.enableR8.fullMode=true - Run the compiled APK on any Android device running Android 14, 15, or 16 (equipped with updated Google Play Mainline ART modules).
- Trigger rapid concurrent network requests (e.g., loading a feed) to quickly increase the execution count of OkHttp's internal execution path, forcing the ART runtime to promote the method to a "hot method" and trigger JIT compilation.
Expected Result
The network calls complete smoothly and Sentry captures HTTP spans and breadcrumbs correctly without any native stability issues.
Actual Result
The background JIT compilation thread of ART (Android Runtime) crashes with a fatal native SIGSEGV (11) segmentation fault while compiling okhttp3.internal.connection.RealCall.getResponseWithInterceptorChain(), causing the application process to die immediately.
Obfuscated Native Tombstone Stacktrace:
at __start_thread (<unknown>)
at __pthread_start (<unknown>)
...
at okhttp3.internal.connection.RealCall.G0 (<unknown>) # R8 optimized getResponseWithInterceptorChain
at io.sentry.okhttp.b.f (<unknown>) # Obfuscated SentryOkHttpEventListener
Detailed Technical Analysis
- ASM Bytecode Injection: The
sentry-android-gradle-pluginuses ASM bytecode manipulation (defined inResponseWithInterceptorChainMethodVisitor.kt) to inject a complexwhileloop that allocates registers/variables on the fly to detect and addSentryOkHttpInterceptorinside OkHttp'sokhttp3.internal.connection.RealCall.getResponseWithInterceptorChain(). - Missing / Obsolete Consumer Proguard Rules:
- The
sentry-okhttpmodule is packaged as a pure Java.jarlibrary, meaning it cannot ship native Android.aarconsumer proguard files. - The main
sentry-android-coremodule ships Proguard rules, but they are obsolete: they reference the old package nameio.sentry.android.okhttp.SentryOkHttpInterceptorinstead of the modernio.sentry.okhttp.SentryOkHttpInterceptor(refactored in v8.x). See line 46-47 in sentry-android-core/proguard-rules.pro. - Therefore, there are no Proguard keep rules anywhere in Sentry's libraries protecting the
io.sentry.okhttp.**package from optimization.
- The
- R8 Optimization Conflict: Since Sentry OkHttp classes are completely unprotected, R8 (under Full Mode) aggressively optimizes, obfuscates, and inlines them (e.g., companion objects and helpers) on top of the raw bytecode manually woven by the Sentry ASM visitor. This generates inconsistent local variables allocation and corrupt
StackMapTableframes in the final.dexfile. - JIT Compiler Crash: When ART's background JIT compiler thread (which is shared across Android 14+ via mainline system updates) attempts to compile this corrupt method into native ARM64 instructions, the strict verificator in ART fails to resolve the incorrect stack maps, resulting in a fatal native memory pointer disintegration (
SIGSEGV).
Temporary Workaround
Adding these rules to the application's proguard-rules.pro file completely resolves the issue, since it prevents R8 from inlining/optimizing the woven bridge:
# Prevent R8 from optimizing/inlining the unprotected Sentry OkHttp classes
-keep class io.sentry.okhttp.** { *; }
-dontwarn io.sentry.okhttp.**
# Protect internal RealCall bytecode manipulated by the plugin
-keep class okhttp3.internal.connection.RealCall { *; }
- Ngôn ngữ chính
- Kotlin
- Star
- 1.4k
- Fork
- 478
- Merge trung bình
- 2 ngày 20 giờ
- Pull request đã merge (30 ngày)
- 71
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của getsentry/sentry-java
-
Improvement Java Platform: Android Platform: Java
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
getsentry/sentry-java#6145 · 1 bình luận · 1 người được giao ·
-
Bug Java Platform: Android Platform: Java
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
getsentry/sentry-java#6138 · 1 bình luận ·
-
Feature Java Platform: Java Spans
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
getsentry/sentry-java#5984 · 1 bình luận ·
-
Android Task Traces
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
getsentry/sentry-java#5376 · 1 bình luận ·
-
Android Docs Errors
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 64/100
getsentry/sentry-java#5375 · 1 bình luận ·
Tất cả issue của getsentry/sentry-java
Issue tương tự
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Richie97/track-history#325 ·
-
bug CLI documentation reliability
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
KW-related Type:bug UI / UX
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
RevenueCat/purchases-android#4299 · 1 bình luận ·
-
[Bug] 统计页面无法重置token和汇率 Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
AAswordman/Operit#1265 · 3 bình luận ·