Skip to content

Conversation

@siddhesh-06
Copy link
Contributor

Description

  1. Implemented null-safety for uri.getPath()
  2. Added validation of the expected provider prefix before performing any substring operations to avoid unsafe assumptions about URI structure.
  3. Added fallback behaviour for unexpected or short paths & improved logging
  4. Prevented crashes when the constructed file does not exist

Issue tracker

Fixes #4477

Automatic tests

  • Added test cases

Manual tests

  • Done

Build tasks success

Successfully running following tasks on local:

  • ./gradlew assembledebug
  • ./gradlew spotlessCheck

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.

[fromContentUri] Null/short path causes substring crash

1 participant