-
-
Notifications
You must be signed in to change notification settings - Fork 10.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ERROR: Exception on thread Thread[video,5,main] #4922
Comments
The vendor rom calls an internal "Sky-something", which fails. Could you please post your framework.jar?
|
framework.jar.zip Thanks |
Hi, I've post the framework.jar. Just wonder if there's any chance that my devices could be available to connect with this. |
It's weird, your framework.jar is almost empty, it doesn't contains the expected classes. |
The OS provider is called Coocaa, maybe they did some tricks reconstructed the framework |
Same issue. Also happen at Skyworth TV, Looks like it's common issue on Skyworth TV |
My Skyworth Coocaa TV is the same issue.I tried to downgrade to scrcpy v1.19 and resolved the issue.But its maximum resolution can only be set to 1920 x 1080, and its HDMI channel is black screen (appears to have DRM protection?) |
With the latest version, does it work if you disable audio ( |
I just tried, but this error still persists.
|
same here and my for 1.19 is OK
for 1.20 is error
for 2.5
|
Between 1.19 and 1.20, this is probably due to this commit 8df42ce. Let's try with this quick&dirty change: diff --git a/server/src/main/java/com/genymobile/scrcpy/Workarounds.java b/server/src/main/java/com/genymobile/scrcpy/Workarounds.java
index 448e7099f..db0d72789 100644
--- a/server/src/main/java/com/genymobile/scrcpy/Workarounds.java
+++ b/server/src/main/java/com/genymobile/scrcpy/Workarounds.java
@@ -74,6 +74,9 @@ public final class Workarounds {
mustFillAppContext = true;
}
+ mustFillAppInfo = true;
+ mustFillAppContext = true;
+
if (audio && Build.VERSION.SDK_INT == Build.VERSION_CODES.R) {
// Before Android 11, audio is not supported.
// Since Android 12, we can properly set a context on the AudioRecord. Replace this file in your v2.5 release:
|
This is an effective modification, thank you very much! |
Oh great! Could you then test this new one and confirm that it works:
Here is the diff: diff --git a/server/src/main/java/com/genymobile/scrcpy/Workarounds.java b/server/src/main/java/com/genymobile/scrcpy/Workarounds.java
index 448e7099f..3f86f892b 100644
--- a/server/src/main/java/com/genymobile/scrcpy/Workarounds.java
+++ b/server/src/main/java/com/genymobile/scrcpy/Workarounds.java
@@ -63,7 +63,7 @@ public final class Workarounds {
// - <https://github.com/Genymobile/scrcpy/issues/940>
// - <https://github.com/Genymobile/scrcpy/issues/994>
mustFillAppInfo = true;
- } else if (Build.BRAND.equalsIgnoreCase("honor")) {
+ } else if (Build.BRAND.equalsIgnoreCase("honor") || Build.BRAND.equalsIgnoreCase("skyworth")) {
// More workarounds must be applied for Honor devices:
// - <https://github.com/Genymobile/scrcpy/issues/4015>
// |
Oh no, the NullPointerException error has occurred again.
|
Oh in your case, the manufacturer is still diff --git a/server/src/main/java/com/genymobile/scrcpy/Workarounds.java b/server/src/main/java/com/genymobile/scrcpy/Workarounds.java
index 999f6a714..c9a26d787 100644
--- a/server/src/main/java/com/genymobile/scrcpy/Workarounds.java
+++ b/server/src/main/java/com/genymobile/scrcpy/Workarounds.java
@@ -63,7 +63,7 @@ public final class Workarounds {
// - <https://github.com/Genymobile/scrcpy/issues/940>
// - <https://github.com/Genymobile/scrcpy/issues/994>
mustFillAppInfo = true;
- } else if (Build.BRAND.equalsIgnoreCase("honor") || Build.BRAND.equalsIgnoreCase("skyworth")) {
+ } else if (Build.BRAND.equalsIgnoreCase("honor") || Build.MANUFACTURER.equalsIgnoreCase("skyworth")) {
// More workarounds must be applied for Honor devices:
// - <https://github.com/Genymobile/scrcpy/issues/4015>
// and Skyworth devices:
|
Congratulate, it works successful again. |
👍 Merged into |
The vendor-modified ROM of Skyworth devices needs a valid app info/context. Fixes Genymobile#4922 <Genymobile#4922>
The vendor-modified ROM of Skyworth devices needs a valid app info/context. Fixes Genymobile#4922 <Genymobile#4922>
Environment
Describe the bug
On errors, please provide the output of the console (and
adb logcat
if relevant).Please do not post screenshots of your terminal, just post the content as text instead.
The text was updated successfully, but these errors were encountered: