Skip to content
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

add a check for kiosk/idu pkgs #16730

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

FlexBy420
Copy link
Contributor

@FlexBy420 FlexBy420 commented Feb 19, 2025

Compared multiple idu/kiosk pkgs to normal ones and didnt really see a way to difference them based on metadata.
Usually kiosk/idu pkgs end with -TE in the name, so based on that we can use PKG_AES_KEY_IDU. Not really a great way to do that since if user renames the PKG they wont be able to install it.

compared multiple idu/kiosk pkgs to normal ones and didnt really see a way to difference them based on metadata
@AniLeo AniLeo requested a review from elad335 February 19, 2025 16:13
@elad335
Copy link
Contributor

elad335 commented Feb 20, 2025

If there is no difference in metadata, we can opt to data validity checks after decryption for different keys. Although it seems suspicious.

@@ -478,30 +478,37 @@ bool package_reader::read_metadata()
return true;
}


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this please

bool package_reader::decrypt_data()
{
if (!m_is_valid)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are you changing tabs for spaces?
Revert all of these changes please

@Megamouse Megamouse added the Loader Involving the load of PS3 file formats label Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Loader Involving the load of PS3 file formats
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants