-
Notifications
You must be signed in to change notification settings - Fork 13.8k
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
Fix error when printing ticket contents from a kirbi file format #17468
Fix error when printing ticket contents from a kirbi file format #17468
Conversation
Looks like this fixes the crash - but I can't decrypt the ticket 👀
It looks like the AES_KEY is ignored, and it goes down this path the NTHASH path instead:
Breakpoint:
i.e. it returns an empty key at line 70 instead of continuing to line 73 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add a quick module test for this? 👍
Fixing the AES_KEY bug would be great too
Looks good to me; Kirbi no longer crashes, and I can decrypt kirbi files too with the aes_key after running a secrets dump. kiwi decryption steps
We should circle back round to supporting ul_type 12 at some point - but it's good for now 👍
|
Quick fix not sure how this slipped in, must have refactored some "real quick" before pushing and din't re-test in the original PR here #17390
Before:
After: