Fixed the writer thread. (#42)

* Fixed the writer thread.
* Version increment.
* Added todo and info.
This commit is contained in:
Revertron 2022-12-29 19:59:19 +01:00 committed by GitHub
parent 05de180066
commit ce50b01781
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 2 deletions

View file

@ -11,8 +11,8 @@ android {
applicationId "eu.neilalexander.yggdrasil"
minSdkVersion 21
targetSdkVersion 29
versionCode 12
versionName "0.1-012"
versionCode 13
versionName "0.1-013"
testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
}

View file

@ -282,9 +282,11 @@ open class PacketTunnelProvider: VpnService() {
val writerStream = writerStream
val writerThread = writerThread
if (writerThread == null || writerStream == null) {
Log.i(TAG, "Write thread or stream is null")
break@writes
}
if (Thread.currentThread().isInterrupted || !writerStream.fd.valid()) {
Log.i(TAG, "Write thread interrupted or file descriptor is invalid")
break@writes
}
try {
@ -293,6 +295,12 @@ open class PacketTunnelProvider: VpnService() {
writerStream.write(buf, 0, len.toInt())
}
} catch (e: Exception) {
Log.i(TAG, "Error in write: $e")
if (e.toString().contains("ENOBUFS")) {
//TODO Check this by some error code
//More info about this: https://github.com/AdguardTeam/AdguardForAndroid/issues/724
continue
}
break@writes
}
}
@ -308,15 +316,18 @@ open class PacketTunnelProvider: VpnService() {
val readerStream = readerStream
val readerThread = readerThread
if (readerThread == null || readerStream == null) {
Log.i(TAG, "Read thread or stream is null")
break@reads
}
if (Thread.currentThread().isInterrupted ||!readerStream.fd.valid()) {
Log.i(TAG, "Read thread interrupted or file descriptor is invalid")
break@reads
}
try {
val n = readerStream.read(b)
yggdrasil.sendBuffer(b, n.toLong())
} catch (e: Exception) {
Log.i(TAG, "Error in sendBuffer: $e")
break@reads
}
}