Repository navigation
OpenCore-Patcher Developer Discussions not for general users #243
Replies: 101 comments 127 replies
|
Hi @gandolf243 Yes, you are right. I have been heavily working on my Medelcartelinc fork to fix several critical bugs in the T1 Experimental Support. Specifically, I fixed the os.execl restart crash, cleaned up the Mode Selector UI, and just added a bypass in subprocess_wrapper.py to fix the OCLP_PHT_ERROR_INVALID_CERTIFICATES error when the app is launched via sudo on unsigned builds. I am opening a Pull Request right now to bring all my fixes into your repo! |
Nope, it's not overreacting. Some pieces of code are completely missing, patches for T2 Macs have completely messed up and other stuff and managed to fix the messed up T2 patches. However, the code is currently highly unstable and not ready for being released. The last version that was released before this PR getting merged. |
|
So before releasing anything I test the application logic and also do some testing and research on Mac mini 2018 and MacBook Pro 2020 4 thunderbolt 3 ports. |
|
Hi @albert-mueller and @gandolf243! Don't panic, the code is not lost! 😅 I intentionally cleaned up the GUI and hid the T1 Experimental Mode (like the Mode Selector and T1 patches) because I wanted this PR to focus only on your critical bugs (the os.execl crash and the PHT sudo bypass) to keep it stable for your T2 users. Also, in my latest two commits, I just restored the missing UI buttons and icons in your standard mode that I accidentally removed during the cleanup. The T1 Experimental code is perfectly safe on my end. Test these fixes on your hardware, and once we merge this, I can easily open a separate PR with the Dual Mode Selector and the T1 Tahoe patches so we can work on them together! |
I didn’t got panicked fortunately. It happens. The world hasn’t ended because of broken source code 😀. |
|
Good! You did leave some bugs in the code though, the save openCore is broken and I took me a moment (after I realized that it wasn't building) to find the build opencore button, this is slower, especially since I was considering auto selecting the partition for the user if the the EFI or another partition was the only one available. I wasn't panicking, I was simply noticing that the code was missing and in it's place was some extra bugs, I wanted to know where it was and fix the issues, now that I know that my fixes won't be overridden, I will start fixing the bugs. |
We'll work together with @gandolf243 to fix bugs. I already fixed a couple inside Settings. |
|
Besides bugs, I found also exploitable vulnerabilities in gui_main_menu.py only - and most of them allow attackers to execute arbitary code or launch denial of service attacks. Except one that inside the Developer mode, an attacker could confuse the patcher into selecting Option 5, 6, ü, ю, у, й, ь, ö, œ, ß or anything to cause the application to crash out of confusion attack, which is a mix of confusion attack and denial of service again. All of them were easy to exploit - even an attacker with basic Python knowledge could exploit some of them. I fixed them in the PR. And one other vulnerability allowed an attacker to perform ClickFix attacks due to lack of error knowledge. If for some reason those kexts were to be missing, an attacker could trick the user into skipping the injection of these kexts by simply deleting the entire block of kext injection to cause denial of service attacks again. I fixed all of these vulnerabilities. The app doesn't crash, it closes but then fails to open. @Medelcartelinc @gandolf243 can you try to fix? |
The Save OpenCore is broken since a couple of releases, prior to this broken pull request. |
Will fix it. |
|
Hey, @albert-mueller I just added a apple provided authentication and did a minor design overhaul, I fixed the save opencore button, and added one or two features. Hope you enjoy the changes. |
Yes, if you run it locally say on my mac, if it was on High Sierra, the oldest version of chatgpt (which i don't recommend using because it outputs nonsense) would take 7min for a paragraph or two and in the mean time my fan would see if it could convert my mac into a airplane! but this is FREE, preset-up, cloud hosted AI models, the free version randomly selects between Gemini, ChatGPT and Claude to answer your request. that is why I like that site. No on-device models, all cloud hosted, 50 request/d, we can set in-app limits to help keep it open for others or see how much it would cost for more requests. |
|
what do you think of my update idea? |
|
is it OK, if I start setting up documentation in the wiki page? |
|
also, is OpenCore-Patcher T2 signed? if so, then I would want to setup notifications instead of Dialogs. (I would keep dialogs as the fall back if 1, the binary was not signed, 2, if the user didn't let OpenCore-Patcher T2 send notifications. |
|
Hey @albert-mueller, you missed it, some one got a macrumors account just to accuse us of stealing free software! (i.e, OCLP) |
|
@albert-mueller, Sorry, I unfortunately went too far over in mac rumors and got crystall1nedev to say "I genuinely have no words to say in response to you or anyone involved with the OCLP T2 fork if this is how you're going to act." |
|
@albert-mueller @gandolf243 Sorry, I accidentally deleted my previous message! I read your reply Albert ("Can we reintroduce these then?"), and the answer is absolutely YES. We have everything isolated in our fork and we can definitely do a clean rebase to reintroduce all those custom patches on top of the new base. Just to reiterate what I posted before, if we switch blindly right now, we will lose our Tahoe support. Here is the breakdown of the specific modifications we currently have in our fork that need to be preserved: Wi-Fi (Tahoe): We manually reintroduced wifip2pd, IO80211.framework, and WiFiPeerToPeer.framework to restore AWDL (Airdrop/Handoff) for the Broadcom chip on Tahoe. I spent 3 full days of my own time debugging and getting Tahoe to work perfectly on my Mac, fixing some missing things, and I also personally brought the T1 support. To be honest, I feel like they are attacking us now because we kept pushing forward while they left OpenCore development on standby for a long time. We didn't give up when things got hard. I do this entirely as a hobby, not for money. I do it mainly to keep my own old Mac alive and to help other people do the same. It might not work perfectly on every single Mac yet, but it works on many, and a lot of users have thanked us. I honestly do not understand what that person wants from us. We are just trying to keep these machines running, which should be the whole point of this open-source community. |
|
@albert-mueller can I (or you) move the contents of the docs folder to the wiki tab of this repo? |
|
@Medelcartelinc @albert-mueller hey! I am going to bed but I just want to say this: please help me boot tahoe here: macOS Tahoe 26 on unsupported Macs discussion post by Gandolf243 |
|
@albert-mueller, I am changing how I go a this. I am instead going to create a domain called "albert-mueller.github.io/OpenCore-Patcher-T2" and host the docs there. does that sound OK? |
|
@Medelcartelinc Breaking news! Now the prebuilt Priveleged Helper Tool binary is no longer there to mitigate a vulnerability where an attacker could abuse it for delivering malware by abusing my own signature. So from 190009 onwards, if you want the patcher to include a Priveleged Helper Tool, now you'll have to build the binary. |
|
@albert-mueller, the docs should be on the web after you do this: |
|
@gandolf243 I archived only temporarily the repo and unarchived it immediately to test an End of life warning if the repo ever gets archived to ensure if the repo gets archived or deleted, the users will immediately get notified: |
|
Hey @albert-mueller, @gandolf243 |
|
@gandolf243 Thanks for looking into that and sharing the link! Following the advice in that chat, I went ahead and submitted a ticket for manual review (#3487883) through this account just in case. |
|
@albert-mueller, I can't create a Announcement. Can you update the lock to include collaborators or find a way to promote me and any other person that you think should be promoted to maintainer? |
|
GUYS!!!! @Medelcartelinc account is unbanned!!! He is back!! |
|
@Medelcartelinc Hi @Medelcartelinc ! I pushed PR to your OpenCore Legacy Patcher T2 repo for the following reasons:
|
|
@albert-mueller, which one of these images do you want to be the background of the supported Models image on the website? don't worry if this looks bad, I just ran out of free Claude prompts for fixing up the shader. |



Uh oh!
There was an error while loading. Please reload this page.
@albert-mueller this code for some reason is not complete, there is code missing that I saw on @Medelcartelinc's fork. @Medelcartelinc don't delete your fork yet. Am I over reacting? Or is this real? all I know is that there are still bug in this code that I thought had gotten fixed. I want to work on this, but I need to know where the code went.
All reactions