fix(video-ads): block remaining video ads

ReVanced · Oct 7, 2022 · 5147404 · 5147404
1 parent dccb2ab
commit 5147404
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 55 deletions.
diff --git a/...o/fingerprints/ShowVideoAdsFingerprint.kt → .../video/fingerprints/LoadAdsFingerprint.kt b/...o/fingerprints/ShowVideoAdsFingerprint.kt → .../video/fingerprints/LoadAdsFingerprint.kt
@@ -2,16 +2,17 @@ package app.revanced.patches.youtube.ad.video.fingerprints
 
 import app.revanced.patcher.annotation.Name
 import app.revanced.patcher.annotation.Version
-import app.revanced.patcher.extensions.or
 
 import app.revanced.patcher.fingerprint.method.impl.MethodFingerprint
 import app.revanced.patches.youtube.ad.video.annotations.VideoAdsCompatibility
-import org.jf.dexlib2.AccessFlags
 
-@Name("show-video-ads-method-fingerprint")
+@Name("load-ads-fingerprint")
 
 @VideoAdsCompatibility
 @Version("0.0.1")
-object ShowVideoAdsFingerprint : MethodFingerprint(
-    "V", AccessFlags.PUBLIC or AccessFlags.FINAL, listOf("Z")
+object LoadAdsFingerprint : MethodFingerprint(
+    strings = listOf("Unexpected playerAd type: "),
+    customFingerprint = { method ->
+        method.parameterTypes.size > 0 && method.parameterTypes.first() == "Ljava/lang/String;"
+    }
 )
diff --git a/.../app/revanced/patches/youtube/ad/video/fingerprints/ShowVideoAdsConstructorFingerprint.kt b/.../app/revanced/patches/youtube/ad/video/fingerprints/ShowVideoAdsConstructorFingerprint.kt
diff --git a/src/main/kotlin/app/revanced/patches/youtube/ad/video/patch/VideoAdsPatch.kt b/src/main/kotlin/app/revanced/patches/youtube/ad/video/patch/VideoAdsPatch.kt
@@ -5,15 +5,15 @@ import app.revanced.patcher.annotation.Name
 import app.revanced.patcher.annotation.Version
 import app.revanced.patcher.data.BytecodeContext
 import app.revanced.patcher.extensions.addInstructions
-import app.revanced.patcher.fingerprint.method.impl.MethodFingerprint.Companion.resolve
+import app.revanced.patcher.extensions.instruction
 import app.revanced.patcher.patch.BytecodePatch
 import app.revanced.patcher.patch.PatchResult
 import app.revanced.patcher.patch.PatchResultSuccess
 import app.revanced.patcher.patch.annotations.DependsOn
 import app.revanced.patcher.patch.annotations.Patch
+import app.revanced.patcher.util.smali.ExternalLabel
 import app.revanced.patches.youtube.ad.video.annotations.VideoAdsCompatibility
-import app.revanced.patches.youtube.ad.video.fingerprints.ShowVideoAdsConstructorFingerprint
-import app.revanced.patches.youtube.ad.video.fingerprints.ShowVideoAdsFingerprint
+import app.revanced.patches.youtube.ad.video.fingerprints.LoadAdsFingerprint
 import app.revanced.patches.youtube.misc.integrations.patch.IntegrationsPatch
 import app.revanced.patches.youtube.misc.settings.bytecode.patch.SettingsPatch
 import app.revanced.patches.youtube.misc.settings.framework.components.impl.StringResource
@@ -27,7 +27,7 @@ import app.revanced.patches.youtube.misc.settings.framework.components.impl.Swit
 @Version("0.0.1")
 class VideoAdsPatch : BytecodePatch(
     listOf(
-        ShowVideoAdsConstructorFingerprint
+        LoadAdsFingerprint
     )
 ) {
     override fun execute(context: BytecodeContext): PatchResult {
@@ -41,17 +41,19 @@ class VideoAdsPatch : BytecodePatch(
             )
         )
 
-        ShowVideoAdsFingerprint.resolve(
-            context, ShowVideoAdsConstructorFingerprint.result!!.classDef
-        )
-
-        // Override the parameter by calling shouldShowAds and setting the parameter to the result
-        ShowVideoAdsFingerprint.result!!.mutableMethod.addInstructions(
-            0, """
-                invoke-static { }, Lapp/revanced/integrations/patches/VideoAdsPatch;->shouldShowAds()Z
-                move-result v1
-            """
-        )
+        LoadAdsFingerprint.result!!.mutableMethod.let { method ->
+            method.addInstructions(
+                0,
+                """ 
+                    invoke-static { }, Lapp/revanced/integrations/patches/VideoAdsPatch;->shouldShowAds()Z
+                    move-result v1
+                    if-nez v1, :show_video_ads
+                    const/4 v1, 0x0
+                    return-object v1
+                """,
+                listOf(ExternalLabel("show_video_ads", method.instruction(0)))
+            )
+        }
 
         return PatchResultSuccess()
     }