Skip to content

Catch IllegalArgumentException when reading trace input stream for last fatal ANR#3483

Merged
0xnm merged 1 commit into
developfrom
nogorodnikov/catch-illegal-argument-exception-when-reading-trace-input-stream
May 28, 2026
Merged

Catch IllegalArgumentException when reading trace input stream for last fatal ANR#3483
0xnm merged 1 commit into
developfrom
nogorodnikov/catch-illegal-argument-exception-when-reading-trace-input-stream

Conversation

@0xnm

@0xnm 0xnm commented May 28, 2026

Copy link
Copy Markdown
Member

What does this PR do?

Apparently it can be like this:

java.lang.IllegalArgumentException: java.lang.IllegalArgumentException: Invalid package name
at android.os.Parcel.createExceptionOrNull Parcel.java:3083
at android.os.Parcel.createException Parcel.java:3063
at android.os.Parcel.readException Parcel.java:3046
at android.os.Parcel.readException Parcel.java:2988
at android.app.IAppTraceRetriever$Stub$Proxy.getTraceFileDescriptor IAppTraceRetriever.java:151
at android.app.ApplicationExitInfo.getTraceInputStream ApplicationExitInfo.java:813

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@0xnm
0xnm requested review from a team as code owners May 28, 2026 11:58
@codecov-commenter

codecov-commenter commented May 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.16%. Comparing base (6107f38) to head (a483f3e).

Files with missing lines Patch % Lines
...g/android/rum/internal/DatadogLateCrashReporter.kt 0.00% 7 Missing ⚠️
Additional details and impacted files 
@@             Coverage Diff             @@
##           develop    #3483      +/-   ##
===========================================
- Coverage    72.29%   72.16%   -0.13%     
===========================================
  Files          964      964              
  Lines        35576    35583       +7     
  Branches      5926     5926              
===========================================
- Hits         25719    25677      -42     
- Misses        8259     8291      +32     
- Partials      1598     1615      +17
Files with missing lines Coverage Δ
...g/android/rum/internal/DatadogLateCrashReporter.kt 86.79% <0.00%> (-3.94%) ⬇️

... and 35 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@0xnm
0xnm merged commit d8f145f into develop May 28, 2026
28 checks passed
@0xnm
0xnm deleted the nogorodnikov/catch-illegal-argument-exception-when-reading-trace-input-stream branch May 28, 2026 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ambushwork ambushwork ambushwork approved these changes
@kikoveiga kikoveiga kikoveiga approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@0xnm @codecov-commenter @ambushwork @kikoveiga