Skip to content
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

Cannot unpatch extern method patch #92

Open
alexhaffner opened this issue Nov 30, 2023 · 1 comment
Open

Cannot unpatch extern method patch #92

alexhaffner opened this issue Nov 30, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@alexhaffner
Copy link

alexhaffner commented Nov 30, 2023

Hey! I'm using HarmonyX with unity (in editor directly), trying to patch & unpatch in play mode.
After I call Harmony.UnpatchAll(); the methods still seems to be patched.

public class DestroyPatch
{
    [HarmonyPostfix]
    [HarmonyPatch(nameof(Object.Destroy), new Type[] { typeof(Object), typeof(float) })]
    public static void PrefixA(Object obj, float t)
    {
        Debug.Log("Destroy(Object obj, float t) called");
    }
    
    [HarmonyPostfix]
    [HarmonyPatch(nameof(Object.DestroyImmediate), new Type[] { typeof(Object), typeof(bool) })]
    public static void PrefixB(Object obj, bool allowDestroyingAssets)
    {
        Debug.Log("DestroyImmediate(Object obj, bool allowDestroyingAssets) called");
    }
}

Both are extern calls. I don't have issues with "normal" methods.

Looking into UnpatchConditional, it seems that HasMethodBody returns 0, and thus those methods are skipped and never unpatched.

Thanks!

@ManlyMarco ManlyMarco added the bug Something isn't working label Dec 3, 2023
@DaXcess
Copy link

DaXcess commented Jan 5, 2025

Having this issue as well.

As a workaround (and ironically), a patch can fix this, but this might have unforeseen consequences, so use at your own risk:

/// <summary>
/// This may or may not completely crash the game, here be dragons
/// </summary>
[HarmonyPatch(typeof(MethodBaseExtensions), nameof(MethodBaseExtensions.HasMethodBody))]
[HarmonyPrefix]
private static bool OnUnpatch(ref bool __result)
{
    // TODO: Perform some checks so this only targets the methods we want, instead of literally everything
    __result = true;

    return false;
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants