Fix: Fix crash when opening archive files with content path

By fixing Path.getParent() which shouldn't depend on Path.getRoot().

Fixes: #1225

app/src/main/java/me/zhanghai/android/files/provider/common/AbstractPath.kt

@@ -17,13 +17,6 @@ abstract class AbstractPath<T : AbstractPath<T>> : CovariantPath<T> {
  return if (nameCount != 0) getName(nameCount - 1) else null
  }
 
- override fun getParent(): T? =
- when (val nameCount = nameCount) {
- 0 -> null
- 1 -> root
- else -> root!!.resolve(subpath(0, nameCount - 1))
- }
-
  override fun startsWith(other: String): Boolean = startsWith(fileSystem.getPath(other))
 
  override fun endsWith(other: String): Boolean = endsWith(fileSystem.getPath(other))

app/src/main/java/me/zhanghai/android/files/provider/common/ByteStringListPath.kt

@@ -76,6 +76,9 @@ abstract class ByteStringListPath<T : ByteStringListPath<T>> : AbstractPath<T>,
  val fileNameByteString: ByteString?
  get() = segments.lastOrNull()
 
+ override fun getParent(): T? =
+ if (segments.isNotEmpty()) createPath(isAbsolute, segments.dropLast(1)) else null
+
  override fun getNameCount(): Int = segments.size
 
  override fun getName(index: Int): T = createPath(false, listOf(getNameByteString(index)))

app/src/main/java/me/zhanghai/android/files/provider/content/ContentPath.kt

@@ -53,13 +53,14 @@ internal class ContentPath : ByteStringListPath<ContentPath> {
  override fun createPath(path: ByteString): ContentPath =
  ContentPath(fileSystem, path.toString().toUri())
 
- override fun createPath(absolute: Boolean, segments: List<ByteString>): ContentPath =
+ override fun createPath(absolute: Boolean, segments: List<ByteString>): ContentPath {
  if (absolute) {
- require(segments.size == 1) { segments.toString() }
- createPath(segments.single())
- } else {
- ContentPath(fileSystem, segments)
+ require(segments.size == 2) {
+ "Cannot create absolute ContentPath with segments $segments"
+ }
  }
+ return ContentPath(fileSystem, segments)
+ }
 
  override val uriScheme: String
  get() {
@@ -90,6 +91,8 @@ internal class ContentPath : ByteStringListPath<ContentPath> {
 
  override fun getRoot(): ContentPath? = null
 
+ override fun getParent(): ContentPath? = null
+
  override fun normalize(): ContentPath = this
 
  override fun toUri(): URI = URI.create(uri!!.toString())