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

Version Descriptor Set Terminators must be set to 1, not 0 #5

Open
sahlberg opened this issue Sep 8, 2022 · 3 comments
Open

Version Descriptor Set Terminators must be set to 1, not 0 #5

sahlberg opened this issue Sep 8, 2022 · 3 comments

Comments

@sahlberg
Copy link

sahlberg commented Sep 8, 2022

Makeps3iso currently creates ISO images with VDST==0, which is invalid. It has to be set to 1.

The python library pycdlib has a workaround for this issue: clalancette/pycdlib#99
but this should also be fixed in makeps3iso since other tools might also do this check to validate the ISO and break.

@bucanero
Copy link
Owner

bucanero commented Dec 19, 2022

I totally missed this issue. I'll take a look and see if I can update the VDST value.

Edit: also I'd check if PS3 backup managers work correctly if the ISOs have VDST=1 before doing any changes. Because in the end, the main goal is to use these ISOs on the PS3, and if managers expect the value to be zero, then it would be a headache.

@bucanero
Copy link
Owner

I'm not sure, but it seems that the value is actually set to 1 here?

isd->version[0]=1;

@sahlberg
Copy link
Author

sahlberg commented Dec 24, 2022

I'm not sure, but it seems that the value is actually set to 1 here?

isd->version[0]=1;

I don't know the internal structure of DVDs well enough to tell if that is the VDST that pycdlib complained about or if it is a different field.
Pycdlib has a workaround that solves the issues for me so I am fine if you just want to close this issue.
(and if no one else has ever reported an issue with this, it can't be all that many people that were affected in the first place.)
Since a workaround exists it might not be worth the effort to dig into this any deeper.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants