-
Notifications
You must be signed in to change notification settings - Fork 2
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 when patching on Windows #4
Comments
Are you sure you're using the exact same version of Lithium? |
Yes, I'm using Lithium 0.24.5. |
Seems to fix the issue, it is due to windows reading smali will have \r\n while the patch only search for \n. To temporary fix the issue just change line 152 of patch.go with the following. |
Thanks for tracking that down. I'll make a fix for that. On a side note, at some point, I'll get around to replacing the text-based smali patching with proper semantic pattern matching on the instructions themselves. |
The text was updated successfully, but these errors were encountered: