Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update Wiimmfi Patch hack to v3 #274

Merged
merged 1 commit into from
Feb 28, 2021
Merged

Conversation

Leseratte10
Copy link
Contributor

Welp, PR #273 seems to be broken (I can no longer reopen it and it shows the wrong commits ...) so I'm making a new one to merge my change into the release branch.

Description of the original PR:

Two days ago, there was another update to the existing Wiimmfi patchers, because a security issue has been found in a bunch of other Wii games (not just Mario Kart Wii). This PR updates the Priiloader Hack to v3 which does contain these security fixes.

Marking this as a draft PR for now so a couple more people can test this before it's merged and "final" - I didn't really have the chance to test this with a whole bunch of different games. (So please don't merge this just yet)

Please let me know if this causes issues with any games.

@DacoTaco
Copy link
Owner

ill hear it when this is ready for merging :)

@InternalLoss
Copy link

Tested with Mario Kart Wii, PES 2009, Super Mario Galaxy (non-WFC), Super Smash Bros. Brawl and Animal Crossing: Let's Go To The City - no crashes or errors

@Leseratte10
Copy link
Contributor Author

Yeah, looks like that's fine. The logs on the server are looking correct, too - and the patching code itself is identical to the one in the other Wiimmfi patchers so it's unlikely that there are more issues with other games. I guess this is ready to be merged.

@Leseratte10 Leseratte10 marked this pull request as ready for review February 28, 2021 15:48
@DacoTaco DacoTaco merged commit d5f40e5 into DacoTaco:release Feb 28, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants