CoolDroid changes for verifySignatureHook

[cooldroid]

Working well for me. Please review and incorporate if you don't see any issues.

[pylerSM]

I will check this. If I understand correctly, system will just return true and original method will NOT be called. Right?

Maybe we can fully replace such method. (using replaceMethod). It would be same as this, just we would do it using Xposed API.

What do you think?

[pylerSM]

^Hm, we cant replace some methods like this since system needs return value when X option is disabled.

I will test your patch tomorrow.

[cooldroid]

Yes you correct, replacing the method would cause issue if the option is disabled.

[pylerSM]

And what about your patch? if option is disabled, I have no idea what will be returned. Just nothing or?

[cooldroid]

            if (isModuleEnabled() && verifySignature) {
                param.setResult(true);
                return;
            }

If the option is disabled the module will continue normally. the "return" statement is inside the module enabled/disabled check.

[pylerSM]

Right. Merged.

[cooldroid]

Seems that this merge has been overridden by a recent commit. :)

[pylerSM]

Fixed :)